Problem/Motivation

The LdapUserLoginEvent is dispatched without the event name:

$this->eventDispatcher->dispatch($event);

This change happened in https://git.drupalcode.org/project/ldap/-/commit/6aaf0848d6165c2b1d84b0b...

This change causes that event subscribers that subscribe to the EVENT_NAME are no longer triggered. And this change seems to be unintentional to me.

Proposed resolution

Use the EVENT_NAME again.

Issue fork ldap-3534935

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

harlor created an issue. See original summary.

harlor opened merge request !108

harlor’s picture

harlor commented
Status: Active » Needs review
hydra’s picture

hydra commented
Status: Needs review » Reviewed & tested by the community

This looks good to me

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

bluegeek9 changed the visibility of the branch 8.x-4.x to hidden.

bluegeek9’s picture

bluegeek9 commented
Status: Reviewed & tested by the community » 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.