Closed (fixed)
Project:
UI Patterns (SDC in Drupal UI)
Version:
2.0.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
9 Jan 2025 at 12:42 UTC
Updated:
24 Jul 2025 at 10:40 UTC
Jump to comment: Most recent, Most recent file

Comments
Comment #2
pdureau commentedFor reference: https://git.drupalcode.org/project/metatag/-/tree/2.1.x/src/Plugin/Field...
Comment #3
pdureau commentedComment #4
christian.wiedemann commentedComment #5
christian.wiedemann commentedHi,
I can't reproduce this bug. I installed metatag and checked the field storage for field_metatag and the computed metatag. Any hint how I can that reproduce
Comment #6
christian.wiedemann commentedComment #7
pdureau commentedComment #8
grimreaperI have the warnings on the MR of #3485615: [10.1.x] Upgrade to UIP2.
But currently the installation is blocked by #3498496: [2.0.0-rc2] Context error during installation , so I will try to reproduce on a standard install.
Comment #9
grimreaperGot a clue.
It is for some specific entity_type.
Related to:
- core.base_field_override.shortcut.default.metatag
- core.base_field_override.simple_mega_menu.default.metatag
- core.base_field_override.user.user.metatag
These 3 config twice at each cache clear.
I tried to reproduce on a fresh install. See screenshot of /admin/config/regional/content-language.
But I don't know why on my fresh install, the config is saved, but when I export the config, there is no core.base_field_override.shortcut.default.metatag file.
I will try to go deeper tomorrow.
But then maybe it is legacy Metatag stuff in Sobki's config.
Comment #10
grimreaperI think I found it.
On my fresh install I had been able to get a core.base_field_override.user.user.metatag
Whereas on Sobki it looks like:
The field type is not the same.
So I guess it is a Metatag update that had not been correctly applied or a missing update in the Metatag module.
Comment #11
grimreaperNope.
I have deleted my config. Resaved /admin/config/regional/content-language, so it had been recreated for simple_mega_menu and user (not for shortcut like on my other fresh install website).
Recreated with "field_type: metatag_computed" for both user and simple_mega_menu but now the warning is in the previous foreach.
Comment #12
grimreaperOn my fresh install website I don't pass in app/core/lib/Drupal/Component/Plugin/Discovery/DerivativeDiscoveryDecorator.php for field property derivative.
Comment #13
pdureau commentedComment #14
pdureau commentedComment #15
grimreaperComment #17
grimreaperSo, happening not only with Metatag, but with other computed fields like path, menu_link, default_revision and customer_profiles (commerce).
Not sure if this is the proper fix, if there is a better place to act, or better to change the foreach loop directly. But at least, I don't have warnings anymore.
And for the warnings to happen, I had to use a component as a layout in my display and trigger this display. The component can be left empty of slots/props.
Comment #18
christian.wiedemann commentedHi, now I fixed the problems during installation which are related to UI Patterns. This also fixes:
https://www.drupal.org/project/ui_patterns/issues/3498496
I also add these two composer patches which fixes core issues.
We had two issues:
Two problems still exists.
Comment #19
christian.wiedemann commentedComment #20
pdureau commentedLet's talk about that tomorrow
Comment #21
grimreaperThanks for the fixes!
Once pipeline is green. Ok to merge on my opinion.
Comment #22
pdureau commentedPipeline fixes
Comment #23
christian.wiedemann commentedComment #24
pdureau commentedAuto merge.
Comment #26
christian.wiedemann commentedComment #28
christian.wiedemann commented