If a module provides default config for another module, like node module shipping a view (#1806334: Replace the node listing at /node with a view), the config will be loaded regardless of whether or not node is enabled.
Currently, Views works around this by manually storing a key in the config and calling module_exists().

However, this will apply to all config entity types, and this tries to genericize and automate this.

I need to write some tests to provide this is broken.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

tim.plunkett’s picture

Status: Active » Needs review
FileSize
4.86 KB

I'm curious to see if this breaks non-ConfigEntity config.

tim.plunkett’s picture

Here's a test.

Status: Needs review » Needs work
Issue tags: -Configuration system, -VDC, -Configurables

The last submitted patch, drupal-1830716-2-PASS.patch, failed testing.

dawehner’s picture

Status: Needs work » Needs review

#2: drupal-1830716-2-PASS.patch queued for re-testing.

Status: Needs review » Needs work

The last submitted patch, drupal-1830716-2-PASS.patch, failed testing.

tim.plunkett’s picture

Status: Needs work » Needs review

#2: drupal-1830716-2-PASS.patch queued for re-testing.

Status: Needs review » Needs work
Issue tags: +Configuration system, +VDC, +Configurables

The last submitted patch, drupal-1830716-2-PASS.patch, failed testing.

tim.plunkett’s picture

Status: Needs work » Needs review
FileSize
7.75 KB

Status: Needs review » Needs work

The last submitted patch, vdc-1830716-7.patch, failed testing.

tim.plunkett’s picture

tim.plunkett’s picture

Assigned: tim.plunkett » Unassigned
Status: Postponed » Closed (duplicate)