EventDispatcher::dispatch() argument order changed 9.1

Open on Drupal.org →

Created on 13 July 2023, over 1 year ago

Updated 25 March 2024, 9 months ago

Problem/Motivation

When updating to Drupal 10 I get the following error:

TypeError: Drupal\Component\EventDispatcher\ContainerAwareEventDispatcher::dispatch(): Argument #1 ($event) must be of type object, string given, called in /web/modules/contrib/paragraphs/src/Entity/Paragraph.php on line 558 in Drupal\Component\EventDispatcher\ContainerAwareEventDispatcher->dispatch() (line 89 of /web/core/lib/Drupal/Component/EventDispatcher/ContainerAwareEventDispatcher.php).

Steps to reproduce

Add a new paragraph to a content and check the browser console / report logs.

Proposed resolution

The EventDispatcher::dispatch() argument order changed so we need update it.

Remaining tasks

User interface changes

API changes

Data model changes

🐛 Bug report

Status

Closed: duplicate

Version

1.16

Component

Code

Created by

🇧🇪Belgium gilmar.lima Brussels

Live updates comments and jobs are added and updated live.

event dispatcher

Sign in to follow issues

Comments & Activities

Issue created by @gilmar.lima
Comment 9 months ago →
🇭🇺Hungary boobaa
The problem is also present with 8.x-1.16.
Comment 9 months ago →
🇭🇺Hungary boobaa
Okay, I think I (also) missed the fact I was using the patch from #2868252-32: Consider adding a hook to Paragraphs::getSummary → . If I upgrade the module to 1.17 and the patch to #2868252-46: Consider adding a hook to Paragraphs::getSummary → , my errors are gone.

So, is this possible that this issue could be closed as a duplicate of that other one? Because the 1.17 codebase doesn't even have event dispatching in it at all…
Issue was unassigned.
Status changed to Closed: duplicate 9 months ago4:03pm 25 March 2024
Comment 9 months ago →
🇩🇪Germany tobiasb Berlin
I would say yes, lets closed it.

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024