Problem/Motivation

The comments in ConfigInstaller::installOptionalConfig() say that optional config is ignored if it exists already:

      // Only list configuration that:
      // - does not already exist
      // - is a configuration entity (this also excludes config that has an
      //   implicit dependency on modules that are not yet installed)

However, this is only documented in this inline comment, and not in the 'config_api' docs topic in core.api.php.

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

CommentFileSizeAuthor
#2 3368277-2.patch821 bytessidharth_soman

Comments

joachim created an issue. See original summary.

sidharth_soman’s picture

StatusFileSize
new821 bytes

Hi, I've updated the 'config_api' docs topic as needed. Please review the patch.

joachim’s picture

Status: Active » Reviewed & tested by the community

LGTM! Thanks!

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 2: 3368277-2.patch, failed testing. View results

joachim’s picture

Status: Needs work » Reviewed & tested by the community

Unrelated test failure - docs patch only.

  • catch committed a5c30498 on 10.1.x
    Issue #3368277 by sidharth_soman, joachim: document that config/optional...
catch’s picture

Status: Reviewed & tested by the community » Fixed

Commited/pushed to 11.x and cherry-picked to 10.1.x, thanks!

  • catch committed 6c028114 on 11.x
    Issue #3368277 by sidharth_soman, joachim: document that config/optional...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.