MigrateBatchExecutable "progressMessage" incorrect visibility

Created on 21 February 2023, about 2 years ago

Updated 20 April 2023, almost 2 years ago

Problem/Motivation

In 🐛 MigrateBatchExecutable doesn't need additional progress messages Fixed , a new progressMessage method was added to MigrateBatchExecutable.php.

However, it was added with the protected visibility which causes issues as MigrateExecutable has it as public visibility. This causes a PHP fatal error.

Steps to reproduce

Install Migrate Tools 6.x
Configure a migration to use MigrateBatchExecutable
Notice a fatal error is thrown with the progressMessage method being noted

Proposed resolution

Make MigrateExecutable's progressMessage public visibility.

Remaining tasks

User interface changes

API changes

Data model changes

🐛 Bug report

Status

Needs review

Version

6.0

Component

Code

Created by

🇺🇸United States Nuuou Lincoln, NE

Live updates comments and jobs are added and updated live.

Comments & Activities

Issue created by @Nuuou
Comment about 2 years ago →
🇳🇿New Zealand xurizaemon Ōtepoti, Aotearoa 🏝
Status changed to Needs review almost 2 years ago3:22pm 20 April 2023
Open in Jenkins → Open on Drupal.org →
Core: 10.1.x + Environment: PHP 8.1 & MySQL 8
last update almost 2 years ago
36 pass
Comment almost 2 years ago →
🇧🇪Belgium gilles_webstanz
Hello,

We have noticed that the signature of the function is different.

Here is a patch to correct this.

Have a nice day.

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024