Hello and thanks for this module,

Problem/Motivation

When I try to install ECA Helper module, I get the error message below:

TypeError: Drupal\multiple_registration\Controller\MultipleRegistrationController::__construct(): Argument #5 ($messengerService) must be of type Drupal\Core\Messenger\Messenger, Drupal\eca_helper\Decorate\Messenger given, called in /var/www/html/web/modules/contrib/multiple_registration/src/Controller/MultipleRegistrationController.php on line 101 in Drupal\multiple_registration\Controller\MultipleRegistrationController->__construct() (line 89 of /var/www/html/web/modules/contrib/multiple_registration/src/Controller/MultipleRegistrationController.php).

How can I resolve it?

I'm also connecting the related issues too.

Best,
Orkut

Issue fork multiple_registration-3545635

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

orkut murat yılmaz created an issue. See original summary.

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

phily opened merge request !23

phily’s picture

phily
he/him
Primary language French
Location 🇪🇺🇫🇷 Paris, France
commented
Status: Active » Needs review

Faced the same and fixed it using Claude code:
The MultipleRegistrationController constructor has a strict type hint for the concrete Drupal\Core\Messenger\Messenger class instead of the Drupal\Core\Messenger\MessengerInterface interface.
The eca_helper module decorates the messenger service with Drupal\eca_helper\Decorate\Messenger, which implements MessengerInterface but is not an instance of the concrete Messenger class.

MR!23 has been tested with the following combo: Drupal 11.1.9, multiple_registration 3.3.2 and eca_helper 2.10.0
I'm planning checking it with Drupal 11.3 later today.

Later edit: seems ok with Drupal 11.3.2

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

ysamoylenko’s picture

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

Thank you for the patch. It lloks good to me!

ysamoylenko’s picture

ysamoylenko commented
Status: Reviewed & tested by the community » Fixed

The changes have been merged.

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.

Status: Fixed » Closed (fixed)

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