- 🇮🇳India prashant.c Dharamshala
We are also experiencing this issue on our site which was built on Drupal 8.6 and now upgraded to Drupal 10.0.8. Some of the configs like admin toolbar, advagg, etc.do not have UUIDs, I also doubt it is happening due to those.
Changing to Major as it is a blocker during config export/import.
- Status changed to Needs review
about 1 year ago 4:29pm 30 October 2023 - 🇺🇸United States neclimdul Houston, TX
Webform also triggers this with its default contact form when installing from config.
Don't think a notice qualifies for major. If someone could help me recreate the fatal from the load failures addressed in #4 maybe we can bump it back but I suspect that might be a different maybe related issue. Until then it will also be difficult to create a realistic test for it.
Here's an updated patch for Drupal 10 with just the uuid handling (since I don't know how to recreate and test the load failure) and a test.
- last update
about 1 year ago 30,482 pass - 🇺🇸United States neclimdul Houston, TX
-
+++ b/core/tests/Drupal/Tests/Core/Config/StorageComparerTest.php @@ -284,6 +284,50 @@ public function testDifferentCollections() { + $recreate_expected = [ + 'field.storage.node.body', + // TODO Should this be recreated? + 'views.view.test_view', + ];
This asserts the existing behavior with the undefined property evaluating to null. Is recreating actually correct though? That seems kinda odd.
-
+++ b/core/tests/Drupal/Tests/Core/Config/StorageComparerTest.php @@ -284,6 +284,50 @@ public function testDifferentCollections() { + // TODO does asserting the order matter here? + $this->assertEqualsCanonicalizing($recreate_expected, $this->storageComparer->getChangelist('create')); + $this->assertEqualsCanonicalizing($recreate_expected, $this->storageComparer->getChangelist('delete'));
I don't know if the order matters here. Or if we need to assert it since its probably better tested else where. I would appreciate someone more familiar with the methods chiming in here.
-
- Status changed to Needs work
about 1 year ago 11:32pm 10 November 2023 The Needs Review Queue Bot → tested this issue.
While you are making the above changes, we recommend that you convert this patch to a merge request → . Merge requests are preferred over patches. Be sure to hide the old patch files as well. (Converting an issue to a merge request without other contributions to the issue will not receive credit.)
- 🇺🇦Ukraine teodorakis
teodorakis → changed the visibility of the branch 2863785-config-import-creates to hidden.
- 🇺🇦Ukraine teodorakis
teodorakis → changed the visibility of the branch 2863785-config-import-creates to active.
- 🇬🇧United Kingdom alexpott 🇪🇺🌍
@neclimdul if webform is creating entities without a UUID then that's a bug in webform - no?
- 🇬🇧United Kingdom therobyouknow
I got this doing a drush cex
A drush cr
- clear cacheappeared to resolve it
My Drupal version 10.0.9 (need to upgrade to latest)
- 🇬🇧United Kingdom therobyouknow
this has come back
when developing drupal code to share with others, I would want to remove the uuid
https://drupal.stackexchange.com/questions/295320/best-practice-remove-u...however, I believe in doing so, it causes the "Undefined index: uuid". This post appears to explain: https://drupal.stackexchange.com/questions/204859/site-uuid-in-source-st...