Import error

Problem/Motivation

Batch import error when importing via the UI:

An AJAX HTTP error occurred.
HTTP Result Code: 500
Debugging information follows.
Path: /batch?id=6&op=do_nojs&op=do
StatusText: error
ResponseText: The website encountered an unexpected error. Try again later.TypeError: Drupal\config_split\Form\ConfigSplitImportForm::updateStatusOverride(): Argument #2 ($status) must be of type string, null given, called in /var/www/html/web/core/includes/batch.inc on line 297 in Drupal\config_split\Form\ConfigSplitImportForm->updateStatusOverride() (line 258 of modules/contrib/config_split/src/Form/ConfigImportFormTrait.php). _batch_process() (Line: 139)
_batch_do() (Line: 95)
_batch_page(Object) (Line: 52)
Drupal\system\Controller\BatchController->batchPage(Object)
call_user_func_array(Array, Array) (Line: 123)
Drupal\Core\EventSubscriber\EarlyRenderingControllerWrapperSubscriber->Drupal\Core\EventSubscriber\{closure}() (Line: 638)
Drupal\Core\Render\Renderer->executeInRenderContext(Object, Object) (Line: 121)
Drupal\Core\EventSubscriber\EarlyRenderingControllerWrapperSubscriber->wrapControllerExecutionInRenderContext(Array, Array) (Line: 97)
Drupal\Core\EventSubscriber\EarlyRenderingControllerWrapperSubscriber->Drupal\Core\EventSubscriber\{closure}() (Line: 181)
Symfony\Component\HttpKernel\HttpKernel->handleRaw(Object, 1) (Line: 76)
Symfony\Component\HttpKernel\HttpKernel->handle(Object, 1, 1) (Line: 53)
Drupal\Core\StackMiddleware\Session->handle(Object, 1, 1) (Line: 48)
Drupal\Core\StackMiddleware\KernelPreHandle->handle(Object, 1, 1) (Line: 28)
Drupal\Core\StackMiddleware\ContentLength->handle(Object, 1, 1) (Line: 32)
Drupal\big_pipe\StackMiddleware\ContentLength->handle(Object, 1, 1) (Line: 116)
Drupal\page_cache\StackMiddleware\PageCache->pass(Object, 1, 1) (Line: 90)
Drupal\page_cache\StackMiddleware\PageCache->handle(Object, 1, 1) (Line: 50)
Drupal\ban\BanMiddleware->handle(Object, 1, 1) (Line: 48)
Drupal\Core\StackMiddleware\ReverseProxyMiddleware->handle(Object, 1, 1) (Line: 51)
Drupal\Core\StackMiddleware\NegotiationMiddleware->handle(Object, 1, 1) (Line: 36)
Drupal\Core\StackMiddleware\AjaxPageState->handle(Object, 1, 1) (Line: 51)
Drupal\Core\StackMiddleware\StackedHttpKernel->handle(Object, 1, 1) (Line: 741)
Drupal\Core\DrupalKernel->handle(Object) (Line: 19)

Steps to reproduce

Create a config-split profile
- Select a disabled module (or any change that requires an import)
- Save
Go to Configuration Split setting page (/admin/config/development/configuration/config-split)
Activate the new config split profile
Import the new config split profile
Click Import all button
Observe error (see above)

Proposed resolution

I'm not really sure of the purpose of this logic, but the issue appears to be Here:
config_split/src/Form/ConfigImportFormTrait.php::222-224

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

That leading ! means this logic results in ONLY calling updateStatusOverride when $override is NULL, but the 2nd argument of updateStatusOverride is cast as a string requirement. I think the leading ! should be removed, but again I'm not sure of the purpose of this logic.

I'm using 2.0.1, but I see it the current version as well.

Remaining tasks

Update the code (see attached).

User interface changes

n/a

API changes

n/a

Data model changes

n/a