Problem/Motivation

Deprecation error surfaces when running any migrations:

Calling Sql::__construct() without the $migration_manager argument is deprecated in drupal:9.5.0 and the $migration_manager argument will be required in drupal:11.0.0. See https://www.drupal.org/node/3277306

See: https://www.drupal.org/node/3277306

Steps to reproduce

  1. Run any migration.
  2. Observe the indicated error message in Drupal watchdog.

Proposed resolution

  1. Inject the "plugin.manager.migration" service in to class Drupal\acquia_migrate\Plugin\migrate\id_map\SqlWithCentralizedMessageStorage and pass it to parent constructor.

Remaining tasks

  1. MR the resolution.

User interface changes

N/A

API changes

N/A

Data model changes

N/A

Issue fork acquia_migrate-3425906

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

jienckebd created an issue. See original summary.

jienckebd opened merge request !54

wim leers’s picture

wim leers
Location Ghent 🇧🇪🇪🇺
commented
Status: Active » Reviewed & tested by the community

LGTM!

(Tests are green, except for the usual "DB not available"-related failures. 😬 See #3408200: Recent GitLab CI changes made database-heavy tests fail increasingly often, but randomly for that.)

wim leers’s picture

wim leers
Location Ghent 🇧🇪🇪🇺
commented
Assigned: jienckebd » Unassigned
Status: Reviewed & tested by the community » Fixed

  • Wim Leers committed 7eca4200 on 1.9.x authored by jienckebd 
    Issue #3425906 by jienckebd: Fix deprecation error: Calling Sql::...

Status: Fixed » Closed (fixed)

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