Explore alternatives to extending internal core ConfigSync form class

Created on 25 June 2024, 6 months ago

Updated 27 June 2024, 6 months ago

Problem/Motivation

Output from PHPStan (see 📌 GitLabCI: Fix existing PHPStan issues Fixed ):

 ------ ---------------------------------------------------------------- 
  Line   src/Form/ConfigDistroImportForm.php                             
 ------ ---------------------------------------------------------------- 
  18     Class Drupal\config_distro\Form\ConfigDistroImportForm extends  
         @internal class Drupal\config\Form\ConfigSync.

We should explore alternative implementations for ConfigDistroImportForm that don't require extending an internal class.

Complicating considerations:

The config_sync module extends the config distro import form class (see https://git.drupalcode.org/project/config_sync/-/blob/3.0.x/src/Form/Con...)
The config_distro_ignore submodule alters the config distro import form: https://git.drupalcode.org/project/config_distro/-/blob/2.0.x/modules/co...

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

📌 Task

Status

Active

Version

2.0

Component

Code

Created by

🇺🇸United States joegraduate Arizona, USA

Live updates comments and jobs are added and updated live.

Merge Requests

!18Explore alternatives to extending internal core ConfigSync form class
Open
🇺🇸United States trackleft2
updated 6 months ago

Comments & Activities

Issue created by @joegraduate
Comment 6 months ago →
🇺🇸United States joegraduate Arizona, USA
Comment 6 months ago →
🇺🇸United States joegraduate Arizona, USA
First commit to issue fork.
Merge request !18Draft: Try replacing overridden class with hook_form_alter implementation. → (Open) created by trackleft2
Pipeline finished with Success
6 months ago
Total: 139s
#210245
Comment 6 months ago →
🇺🇸United States joegraduate Arizona, USA

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024