Config storage transformers do not get applied when installing Drupal from configuration

Open on Drupal.org →

Created on 28 February 2024, 9 months ago

Updated 7 March 2024, 8 months ago

Problem/Motivation

Note: This could be argued to be a bugfix or a feature request, depending on your viewpoint, so going with task as middle ground, to hopefully avoid any contention.

When building a site with a module that uses a storage transfomer (such as newer versions of Config Split → , Config Ignore → , etc.), it is not possible to (correctly) use the the Install from existing configuration (or drush si --existing-config) feature, as the configuration transfomers will not be called in that case.

Steps to reproduce

Build a site from Minimal profile and enable Config Split and split off some configuration
Export all configuration. The split configuration should not be in the sync directory, but in its own directory
Attempt to install from configuration

At this point the split configuration will not have been installed/imported, even though you would expect that it has been.

Proposed resolution

When installing from configuration we already call install_config_revert_install_changes() after all module have been installed. Simply invoking the storage transfomer at this point fixes the issue.

Remaining tasks

Write tests
Review patch

User interface changes

None.

API changes

None.

Data model changes

None.

Release notes snippet

-

📌 Task

Status

Closed: duplicate

Version

11.0 🔥

Component

Configuration →

Last updated 2 days ago

Maintained by
🇬🇧United Kingdom @alexpott
🇺🇸United States @tim.plunkett

Created by

🇩🇪Germany tstoeckler Essen, Germany

Live updates comments and jobs are added and updated live.

Needs tests
The change is currently missing an automated test that fails when run with the original code, and succeeds when the bug has been fixed.

CMI 2.0 candidate

Sign in to follow issues

Merge Requests

!6806Config storage transformers do not get applied when installing Drupal from configuration
Open
🇩🇪Germany tstoeckler
updated 9 months ago

Comments & Activities

Issue created by @tstoeckler
Status changed to Needs work 9 months ago7:25am 28 February 2024
Comment 9 months ago →
🇩🇪Germany tstoeckler Essen, Germany
Note that I found this in Config Overlay → , because we actually have tests for installing from configuration that broke when we ported to the storage transformer system there. That also verifies that the simple change (will create draft MR in a minute) is sufficient, as we actually apply a core patch in the CI there and that makes the respective tests pass.

Will need an actual test here, though, I presume.
Merge request !6806Invoke the storage transfomer for install from config → (Open) created by tstoeckler
Pipeline finished with Failed
9 months ago
Total: 184s
#105645
Pipeline finished with Success
9 months ago
Total: 470s
#106050
Comment 9 months ago →
🇦🇺Australia thursday_bw
Add the CMI 2.0 candidate tag, and adding the CMI 2 initiative proposal as a related issue.
Status changed to Closed: duplicate 8 months ago10:36am 7 March 2024
Comment 8 months ago →
🇩🇪Germany tstoeckler Essen, Germany
Closed as duplicate of 🐛 Dispatch config transformation event during site install from configuration Active

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024