Problem/Motivation
Installing more recipes on the same instance, sometimes gets to a brick wall. Getting: The configuration 'block.block.gin_messages' exists already and does not match the recipe's configuration error.
I installed a recipe that has dependencies to drupal_cms_starter, drupal_cms_news, then I installed another one with dependency to drupal_cms_blog. I am getting the error mentioned above.
The only way I could get the error to go away was by adding
config:
strict: false
in the drupal_cms_admin_ui.
Steps to reproduce
Proposed resolution
Add config: strict: false in drupal_cms_admin_ui.
I don't think the configs added by this recipes are not supposed to be overwritten.
Issue fork drupal_cms-3523920
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:
Comments
Comment #3
ciprian.stavovei commentedComment #4
phenaproximaI agree; there's no particular reason for this recipe to compare config strictly. RTBC if tests pass.
Comment #5
phenaproximaOn second thought...although I approve of the fix, I am kind of wondering how this is happening. The drupal_cms_admin_ui recipe, like all our recipes, has a test which proves it can be applied twice. So something else must be modifying the block and making it deviate from our shipped config.
Again - I think that setting
strict: falseis completely fine, and the MR is RTBC. But I want to more thoroughly understand the problem before I merge this, just in case it points to a larger bug that needs a more comprehensive fix.Can you provide steps to reproduce, starting from a bare Drupal CMS site -- or whatever it is you started with?
Comment #6
arthur_lorenz commentedThanks, we are currently working on creating recipes based on Drupal CMS for our Lupus Decoupled stack. To provide a streamlined editing experience we base all our recipes on the
drupal_cms_starterrecipe. That's how we noticed that issue.To reproduce it using only
drupal_cmsrecipes:* Fresh Drupal install
* Apply
drupal_cms_starterrecipe* Apply
drupal_cms_newsrecipe* Apply
drupal_cms_starterrecipe againIt fails because the
gin_messages-block's weight got altered.Comment #7
phenaproximaSo this is the one recipe in our stack which is not meant to be applied more than once. I would suggest not using this recipe, and instead compose your recipe using the smaller parts of Drupal CMS (such as
drupal_cms_admin_ui, or any of the other recipes we have, except fordrupal_cms_starter).Comment #8
phenaproximaBut, having said that, I think I'm still okay merging this patch. There's no harm in making
drupal_cms_admin_uinon-strict.Comment #9
phenaproximaComment #12
phenaproximaMerged into 1.x and cherry-picked to 1.1.x. Thanks!