remove/protect unused parts of ImportConfigManipulator

Created on 19 August 2025, 22 days ago

The ImportConfigManipulator could be considerably simplified.

The following aren't used:

> public function getImportProcessor(ImportConfigInterface $import_config, $processor_id);

The following parameters aren't used, and could be removed:

public function getImportProcessorsByStages(ImportConfigInterface $import_config, UNUSED: array $overrides = []) {

The following is only called by the same class, so could be made protected:

> public function getImportProcessorsByStage(ImportConfigInterface $import_config, $stage, array $overrides = [], $processors = []) {

📌 Task

Status

Active

Version

4.0

Component

Code

Created by

🇬🇧United Kingdom joachim

Live updates comments and jobs are added and updated live.

Merge Requests

Issue created by @joachim
Comment 14 days ago →
🇮🇳India kulpratap2002
Working on this.
Merge request !135Issue #3542033: Clean up unused parameters and remove unused method in... → (Open) created by Unnamed author
Pipeline finished with Failed
14 days ago
Total: 382s
#582926
Pipeline finished with Success
14 days ago
Total: 419s
#582931
Comment 14 days ago →
🇮🇳India kulpratap2002

I have cleaned up the ImportConfigManipulator, and based on this, I also changed the ImportConfigManipulatorInterface

Methods can’t be made protected since they are defined in the interface. Implementations must keep them public; otherwise, it would break the contract and cause fatal errors.

Please review.

Production build 0.71.5 2024