Problem/Motivation

See #2910353: Prevent saving config entities when configuration overrides are applied

Config entities should not be loaded with overrides and then saved again as this may result in writing back things like translation or other overrides into the default version.

Steps to reproduce

Proposed resolution

use loadOverrideFree()

Remaining tasks

User interface changes

API changes

Data model changes

Issue fork block_class-3613577

Command icon 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:

About issue forks

Comments

berdir created an issue. See original summary.

berdir opened merge request !86

berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Status: Active » Needs review

Merge request created.

csakiistvan’s picture

csakiistvan
Primary language Hungarian
Location Budapest, Hungary
commented
Assigned: Unassigned » csakiistvan
csakiistvan’s picture

csakiistvan
Primary language Hungarian
Location Budapest, Hungary
commented
Assigned: csakiistvan » Unassigned

✅ Tested and works — applied MR !86, ran ddev drush cr and re-ran drush updatedb on a block carrying a settings.php override: before the fix the raw stored config came back with label: 'OVERRIDDEN LABEL', after it the stored label stayed Breadcrumbs while the invalid dependency was still removed, so the override no longer leaks into the default config.

csakiistvan’s picture

csakiistvan
Primary language Hungarian
Location Budapest, Hungary
commented
Status: Needs review » Reviewed & tested by the community