Problem/Motivation
Events Subscriber are a mess:
display_builder_entity_view.event_subscriber:
class: '\Drupal\display_builder_entity_view\Event\DisplayBuilderEntityViewEventsSubscriber
display_builder_page_layout.page_variant_subscriber:
class: Drupal\display_builder_page_layout\EventSubscriber\DisplayBuilderPageDisplayVariantSubscriber
display_builder_page_layout.page_layout_event_subscriber:
class: Drupal\display_builder_page_layout\EventSubscriber\DisplayBuilderPageLayoutEventsSubscriber
display_builder_views.page_layout_event_subscriber:
class: Drupal\display_builder_views\EventSubscriber\DisplayBuilderViewsLayoutEventsSubscriber
Proposed resolution
Align them with a consistent naming.
For example:
For subscribing to Drupal\display_builder\Event\DisplayBuilderEvents:
display_builder_entity_view.display_builder_subscriber:
class: '\Drupal\display_builder_entity_view\EventSubscriber\DisplayBuilderSubscriber
display_builder_page_layout.display_builder_subscriber:
class: Drupal\display_builder_page_layout\EventSubscriber\DisplayBuilderSubscriber
display_builder_views.display_builder_subscriber:
class: Drupal\display_builder_views\EventSubscriber\DisplayBuilderSubscriber
Other subscriptions:
display_builder_page_layout.page_variant_subscriber:
class: Drupal\display_builder_page_layout\EventSubscriber\PageVariantSubscriber
Issue fork display_builder-3534566
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
Comment #3
pdureau commentedComment #4
pdureau commentedI have tested by navigating the site and play with the 3 sub modules, everything seems to still work OK
Comment #6
mogtofu33 commentedComment #7
pdureau commented