Error: Cannot use object of type Closure as array in Drupal\migrate_tools\MigrateExecutable->resetListeners()

Created on 14 September 2023, about 2 years ago

Problem/Motivation

Executing migration fails with PHP error Cannot use object of type Closure as array in Drupal\migrate_tools\MigrateExecutable->resetListeners() (line 294 of /var/www/html/web/modules/contrib/migrate_tools/src/MigrateExecutable.php)

foreach ($this->getEventDispatcher()->getListeners($event_name) as $registered_listener) {
  if ($registered_listener[0] instanceof self) {
    $this->getEventDispatcher()->removeListener($event_name, $registered_listener);
  }
}

getListeners returns array<callable[]|callable>, so $registered_listener will not always be an array.

Steps to reproduce

To be listed

Proposed resolution

I added an additional check to make sure $registered_listener is an array

Remaining tasks

🐛 Bug report

Status

Active

Version

6.0

Component

Code

Created by

grietdk

Live updates comments and jobs are added and updated live.

Comments & Activities

Issue created by @grietdk
First commit to issue fork.
Open in Jenkins → Open on Drupal.org →
Core: 10.1.x + Environment: PHP 8.1 & MySQL 8
last update about 2 years ago
36 pass
@sarwan_verma opened merge request.
Status changed to Needs review about 2 years ago11:08am 16 October 2023
Comment about 2 years ago →
🇮🇳India sarwan_verma
Hi @grietdk, drupalspoons

I have fixed the issue in MR, please review.
Status changed to Needs work 4 months ago7:16pm 4 July 2025
Comment 4 months ago →
heddn Nicaragua
Shouldn't we handle this differently? The code in question should handle both closures and arrays. Not just arrays.

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024