Problem/Motivation

Our entity_operation_alter (one in each integration sub-module: display_builder_entity_view, display_builder_page_layout, display_builder_views) are not needed since we have harmonized the discovery and the interface of "buildables" as plugins in #3549266: Move DisplayBuildableInterface to a new plugin type.

And they will be even less relevant once we will be able to execute Instance::get('buildable')::getUrlFromInstanceId() and Instance::get('buildable')::getDisplayUrlFromInstanceId() once #3562989: Implements RevisionLogInterface for Instance entity will be merged.

Because we don't need to delegate to each sub-module a logic which is already the same each time.

Proposed resolution

Remove the hooks.

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

Assigned: pdureau » mogtofu33
Status: Active » Needs review

Pipeline is green

  • mogtofu33 committed a742d7e7 on 1.0.x authored by pdureau
    task: #3593112 Remove entity_operation_alter hooks
    
    By: pdureau
    
mogtofu33’s picture

Assigned: mogtofu33 » Unassigned
Status: Needs review » Fixed

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.