Skip to content
This repository has been archived by the owner on Aug 18, 2024. It is now read-only.

Fix event dispatcher BC #725

Open
MPParsley opened this issue Jul 16, 2021 · 1 comment
Open

Fix event dispatcher BC #725

MPParsley opened this issue Jul 16, 2021 · 1 comment

Comments

@MPParsley
Copy link
Collaborator

The arguments to the event dispatcher are backwards compared to expected (some BC code in Symfony is allowing it for now):

$this->eventDispatcher->dispatch(GroupCreationEventInterface::EVENT_NAME, $event);

3:55
that’s in several places (the reversed arguments) (edited)

@pfrenssen
Copy link
Contributor

Seems we have a lot of deprecated calls :)

  2196x: Calling the Symfony\Component\EventDispatcher\EventDispatcherInterface::dispatch() method with a string event name as the first argument is deprecated in drupal:9.1.0, an Event object will be required instead in drupal:10.0.0. See https://www.drupal.org/node/3154407

pfrenssen added a commit that referenced this issue Dec 17, 2021
In Drupal 9.1 the order of the arguments to this method has been changed.
Ref. https://www.drupal.org/node/3154407

Fixes #725.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants