- Issue created by @phenaproxima
- Status changed to Needs review
7 months ago 2:17pm 17 April 2024 - Status changed to Needs work
7 months ago 10:57pm 17 April 2024 - πΊπΈUnited States thejimbirch Cape Cod, Massachusetts
I followed the validation steps, but was unable to apply the second recipe.
My process:
# Checkout the repo. β― git clone git@git.drupal.org:project/distributions_recipes.git β― cd distributions_recipes # Get to the issue code. β― git remote add distributions_recipes-3441557 git@git.drupal.org:issue/distributions_recipes-3441557.git β― git fetch distributions_recipes-3441557 β― git checkout -b 'default-content-duplicate' --track distributions_recipes-3441557/'default-content-duplicate' # Install dependencies. β― composer install # Install Drupal β― php core/scripts/drupal install # Start a web server. β― php core/scripts/drupal quick-start # I then changed 2 recipes to require the basic_shortcuts recipe. # Apply the first. β― php core/scripts/drupal recipe core/recipes/editorial_workflow [OK] Editorial workflow applied successfully # Apply the second. β― php core/scripts/drupal recipe core/recipes/image_media_type In EntityMethodDeriver.php line 59: Class "Drupal\shortcut\Entity\ShortcutSet" does not exist recipe <path>
- πΊπΈUnited States phenaproxima Massachusetts
Try installing Shortcut first?
- Status changed to Needs review
7 months ago 12:53pm 18 April 2024 - πΊπΈUnited States phenaproxima Massachusetts
It worked okay for me.
I did it this way:
drush si minimal -y # Apply the basic_shortcuts recipe directly php core/scripts/drupal recipe core/recipes/basic_shortcuts # Now apply the Standard recipe, which also runs basic_shortcuts php core/scripts/drupal recipe core/recipes/standard
That worked fine.
Conversely, if I check out 11.x and do the same command sequence, I get the expected SQL error:
SQLSTATE[23000]: Integrity constraint violation: 19 UNIQUE constraint faile d: shortcut.uuid: INSERT INTO "shortcut" ("shortcut_set", "uuid", "langcode ") VALUES (?, ?, ?); Array ( [0] => default [1] => 478b3170-1dfd-49d8-8eb3-f1b285445ab7 [2] => en )
So, as far as I can tell, this MR fixes the problem.
- πΊπΈUnited States thejimbirch Cape Cod, Massachusetts
Following the validation steps in #6 I was successfully able to install basic_shortcuts, then standard which proves the content configs are ignored when they have already been applied.
Marking RTBC.
- Status changed to RTBC
7 months ago 2:17pm 18 April 2024 - π¬π§United Kingdom alexpott πͺπΊπ
Committed and pushed fa163524cd4 to 11.x and 28cf67bda8f to 10.3.x. Thanks!
-
alexpott β
committed 28cf67bd on 10.3.x
Issue #3441557 by phenaproxima, thejimbirch: Recipes with default...
-
alexpott β
committed 28cf67bd on 10.3.x
- Status changed to Fixed
7 months ago 8:51am 19 April 2024 -
alexpott β
committed fa163524 on 11.x
Issue #3441557 by phenaproxima, thejimbirch: Recipes with default...
-
alexpott β
committed fa163524 on 11.x
- 296e10a2 committed on patch
Update recipe 10.3.x patch 28cf67bd Issue #3441557 by phenaproxima,...
- 296e10a2 committed on patch
- 85a9d784 committed on patch
Update recipe 11.x patch fa163524 Issue #3441557 by phenaproxima,...
- 85a9d784 committed on patch
Automatically closed - issue fixed for 2 weeks with no activity.