Closed (duplicate)
Project:
Drupal core
Version:
8.0.x-dev
Component:
configuration system
Priority:
Major
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
16 Feb 2013 at 13:58 UTC
Updated:
29 Jul 2014 at 21:54 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
tim.plunkettIf this is truly a major bug, why is nothing breaking?
Comment #2
sunThings will start to break when the most important Configurables are finally converted; e.g., node types, fields, comment node type settings, etc.
This issue is borderline critical. The only reason for why the concrete problem outlined in the OP about filter formats currently works is pure coincidence.
Comment #3
tayzlor commentedSome of these issues are surfacing in #1808248: Add a separate module install/uninstall step to the config import process. A specific example of this can be recreated by trying to do a config import of the standard profiles YAML files over the top of a minimal installation.
One example of this is -
The minimal profile does not ship with a 'administrator' role, but standard does.
the config export from standard includes the following files -
action.action.user_remove_role_action.administrator.yml
action.action.user_add_role_action.administrator.yml
user.role.administrator.yml
The config will also be imported in the above order. When we come to import the administrator role, an insert hook is fired on role creation - user_user_role_insert(), which goes and again creates the 2 actions (which have already just been imported).
Additionally, after the full config import has run, action.action.user_remove_role_action.administrator and action.action.user_add_role_action.administrator still show up on the config import UI as having changes that need to be synced, because the UUIDs still differ between the staging and active directories.
Comment #4
mtiftWhen I try to import standard profiles YAML files over the top of a minimal installation, I get the following error message: "Drupal\Component\Plugin\Exception\PluginException: The plugin (comment_publish_action) did not specify an instance class. in Drupal\Component\Plugin\Factory\DefaultFactory::getPluginClass() (line 62 of /Users/mtift/Sites/d83/core/lib/Drupal/Component/Plugin/Factory/DefaultFactory.php)"
Comment #5
tim.plunkettThat's not a supported operation though, you could never reuse parts of an install profile over another installation.
Now that we require full config imports, isn't this won't fix? Because the full config would have the modules required.
Comment #6
mtiftThis this issue is about module dependencies, I posted a follow-up comment regarding install profiles in #2029771-1: Having installation profiles in system.module.yml causes config import to fail
It would seem that the OP regarding order of module dependencies is still up for debate, and that we need a better alternative to importing alphabetically. Or is this issue solved by full config imports?
Related: #1944368: React to / force order of imports
Comment #6.0
mtiftAdd related issue
Comment #7
alexpottDuplicate of #2080823: Create API to discover config entities' soft dependencies and use this to present a confirm form on module uninstall - and the current patch actually fixes this.