Rename FieldPropertySourceProp

All our source plugins follow a consistent naming pattern, expect one

ui_patterns:

  • UrlWidget.php: * id = "url",
  • FieldPropertySourceProp.php: * id = "field_properties_source_prop" ⚠️
  • BreadcrumbSource.php: * id = "breadcrumb",
  • PathSource.php: * id = "path",
  • TokenSource.php: * id = "token",
  • NumberWidget.php: * id = "number",
  • CheckboxesWidget.php: * id = "checkboxes",
  • ListTextareaWidget.php: * id = "list_textarea",
  • MenuSource.php: * id = "menu",
  • SelectWidget.php: * id = "select",
  • BlockSource.php: * id = "block",
  • CheckboxWidget.php: * id = "checkbox",
  • TextfieldWidget.php: * id = "textfield",
  • WysiwygWidget.php: * id = "wysiwyg",
  • AttributesWidget.php: * id = "attributes",

ui_patterns_field_formatters

  • FieldFormatterSource.php: * id = "field_formater",
  • FieldLabelSource.php: * id = "field_label",

ui_patterns_views

  • ViewFieldSource.php: * id = "view_field",
  • ViewGroupTitleSource.php: * id = "view_group_title",
  • ViewRowsSource.php: * id = "view_rows",

So, is it relevant to rename:

  • FieldPropertySourceProp.php >> FieldPropertySource.php ?
  • field_properties_source_prop >> field_property ?
  • FieldPropertiesSourcePropDeriver >> FieldPropertySourceDeriver ?

Rename FieldPropertySourceBase

Now, lets have a look on class hierarchy of FieldPropertySourceBase:

SourcePluginBase
  ui_patterns / FieldSourceBase
    ui_patterns_field_formatters / FieldLabelSource
    ui_patterns / FieldPropertySourceBase ⚠️
      ui_patterns / FieldPropertySourceProp
      ui_patterns_field_formatters / FieldFormatterSource

It is weird to have "FieldPropertySourceBase" as a parent of "FieldFormatterSource". Can we rename it? How?

Rename EntityLinksUrlSource

  • EntityLinksUrlSource.php >> EntityLinkSource.php.php
  • "Entity Link URL" >> "Entity link" (plugin names are always sentence case)
  • entity_link_url >> entity_link
Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

pdureau created an issue. See original summary.

pdureau’s picture

Title: [2.0.0-alpha3] Renaming of FieldPropertySourceProp and FieldPropertySourceBase » [2.0.0-beta1] Renaming of FieldPropertySourceProp and FieldPropertySourceBase
pdureau’s picture

Title: [2.0.0-beta1] Renaming of FieldPropertySourceProp and FieldPropertySourceBase » [2.0.0-beta1] Rename source plugins
Issue summary: View changes
pdureau’s picture

Title: [2.0.0-beta1] Rename source plugins » [2.0.0-alpha3] Rename source plugins

just_like_good_vibes made their first commit to this issue’s fork.

just_like_good_vibes’s picture

Hello i started to push WIP and i have addressed all comments here.

i started a refactor of the plugin derivers, but i haven't finished yet..

just_like_good_vibes’s picture

Status: Active » Needs review

ready for review please :)

pdureau’s picture

Assigned: Unassigned » pdureau
just_like_good_vibes’s picture

As we exchanged, i am pushing bufixes in this issue.
currently :
- Fix bug: CKEDITOR not shown in WysiwygWidget source widget
- Fix bug: ui_patterns_views style plugins broken (not saving and behaving correctly)

pdureau’s picture

Assigned: pdureau » Unassigned
Status: Needs review » Fixed
pdureau’s picture

Status: Fixed » Closed (fixed)