- Issue created by @Harlor
- Status changed to Needs review
over 1 year ago 10:53am 14 September 2023 - 🇩🇪Germany Harlor Berlin
I'm not sure if one should use the same sandbox parameter for all entity types. Using a different sandbox parameter seems to fix the issue: https://git.drupalcode.org/issue/entity_type_behaviors-3387281/-/commit/...
- Status changed to Closed: works as designed
over 1 year ago 12:05pm 14 September 2023 - 🇩🇪Germany Harlor Berlin
Sorry it seems the issue was caused by a config for an entity type that does not have bundles. Which is not supported - or is it?
- Status changed to Active
over 1 year ago 1:54pm 18 September 2023 - 🇩🇪Germany Hydra
I could reproduce this issue in a vainlla D10 setup, activating entity_type_behaviors for nodes and terms and importing them at the same time in an other environment via drush config-import.
Your patch seems to solve this issue. Of course it would be nice to have some kind of UI integration for entity_types without bundles, but the default issue here is kinda critical and makes this module unusable.
- Merge request !4Issue #3387281 Config import fails when importing multiple entity_type_behaviors → (Merged) created by Harlor
- Status changed to Needs review
over 1 year ago 2:23pm 18 September 2023 - 🇩🇪Germany Hydra
Alright, the patch looked good to me. I added some tests to make sure it won't fail again.
Since there are no tests activated on the project, we don't see anything :( But I could reproduce the described error message with the test, and the commit from @Harlor did indeed fix it.Maybe @Harlor Could give me a quick review on the test before setting this to RTBC.
- Status changed to RTBC
over 1 year ago 8:06am 19 September 2023 - First commit to issue fork.
Automatically closed - issue fixed for 2 weeks with no activity.