Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.Problem/Motivation
While working on recipes reverting using a new checkpoint storage that listened to config CRUD events I discovered that when we delete simple configuration with the config importer we don't set the initial values correctly. This is very similar to #3396197: Config saved during import does not have correct initial values set
Steps to reproduce
Proposed resolution
- Fix \Drupal\Core\Config\ConfigImporter::importConfig() to call ->initWithData() with the old data in the correct place
- Add test coverage
Remaining tasks
User interface changes
None
API changes
None
Data model changes
None
Release notes snippet
N/a
Issue fork drupal-3405798
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
changes
Comments
Comment #3
alexpottComment #6
alexpottThe test only change has failed as expected - https://git.drupalcode.org/project/drupal/-/pipelines/58557
Comment #7
bircheryes, that is totally reasonable. Amazing that nobody has found this until now.
I have reviewed it and have no complaints about the code change.
Comment #8
borisson_I agree with @bircher, looks great, rtbc++
Comment #12
catchCommitted/pushed to 11.x and cherry-picked to 10.2.x and 10.1.x, thanks!