Problem/Motivation

\Drupal\Core\Hook\HookCollectorPass

 * Finally, a hook_implementations_map container parameter is added. This
 * contains a mapping from [hook,class,method] to the module name.

This is no longer true since #3506930: Separate hooks from events

Proposed resolution

Rewrite it.

Remaining tasks

User interface changes

Introduced terminology

API changes

Data model changes

Release notes snippet

Issue fork drupal-3556289

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:

About issue forks

Comments

alexpott created an issue. See original summary.

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

nicxvan opened merge request !13707

nicxvan’s picture

nicxvan commented
Status: Active » Needs review
berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Status: Needs review » Reviewed & tested by the community

Not sure if we should document the includes bits as it's just for the BC layer and we'll need to remember to remove them again, but I think this addresses the issue.

In the CR, I also mentioned that these structures are considered internal, including keyvalue entries. Maybe we could also put this in code somewhere?

nicxvan’s picture

nicxvan commented

CompilerPasses are considered internal by default, but we can add it back if we want to.

nicxvan’s picture

nicxvan commented

That should be added here probably: https://git.drupalcode.org/project/drupal/-/blob/11.x/core/lib/Drupal/Co...

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Version: 11.x-dev » 11.3.x-dev
Status: Reviewed & tested by the community » Fixed

Committed and pushed 312d4a52365 to 11.x and efbcaa175e4 to 11.3.x. Thanks!

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.

  • alexpott committed efbcaa17 on 11.3.x 
    Issue #3556289 by alexpott, nicxvan, berdir: \Drupal\Core\Hook\...

  • alexpott committed 312d4a52 on 11.x 
    Issue #3556289 by alexpott, nicxvan, berdir: \Drupal\Core\Hook\...

nicxvan closed merge request !13707

nicxvan’s picture

nicxvan commented
Component: base system » extension system

Thanks! Just updating the metadata.

Status: Fixed » Closed (fixed)

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