Problem/Motivation

According to core change record: https://www.drupal.org/node/3159012

Symfony\Component\EventDispatcher\Event is deprecated, and EventDispatcher::dispatch() argument order changed from

$event_dispatcher->dispatch($event_name, $event);
to
$event_dispatcher->dispatch($event, $event_name);

And I believe this change will have a lot impact on the hook_event_dispatcher module.

Steps to reproduce

Proposed resolution

It's actually going to be tricky to support old style (before D9.1) and new style at the same time, so maybe create a patch first, so people using older version can use old style, people using D9.1 and apply the patch.

Or, could create 2 versions before / after the change?

Remaining tasks

User interface changes

API changes

Data model changes

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

yang_yi_cn created an issue. See original summary.

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

bradjones1’s picture

Status: Active » Needs review

I think indeed the easiest way to handle this is a new version that is only compatible with ^9.1. The MR does this; a maintainer would have to fork off a new major version branch though. (And this could be an opportunity to open a new 3.x without a core major in front.

bradjones1’s picture

Title: Drupal 9.1 / Symphony 5 breaking changes » Drupal 9.1 / Symfony 5 breaking changes
el7cosmos’s picture

Version: 8.x-2.x-dev » 3.x-dev

el7cosmos’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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