Logic issue with updateStatusOverride callback

Created on 3 February 2025, 2 months ago

Problem/Motivation

updateStatusOverride function, located in config_split/src/Form/ConfigImportFormTrait.php has some logic issue.

It is being called only once from the launchImport function

Below are the code:

        if (!$override !== NULL) {
          $batch_builder->addOperation([$this, 'updateStatusOverride'], [$this->getSplit()->getName(), $override]);
        }

This code will go the updateStatusOverride callback only in case of $override === NULL.

But the callback itself expects a string. Having NULL as a parameter leads to an error.

So basically, I think this is just a small mistyping issue, and instead of
if (!$override !== NULL) {

should be

if ($override !== NULL) {

🐛 Bug report
Status

Active

Version

2.0

Component

Code

Created by

🇵🇱Poland driverok

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

Production build 0.71.5 2024