- Issue created by @Anybody
- 🇮🇳India mukhtarm
After removing the
theme_display: null
from the config (layout_paragraphs.modal_settings
), import it again (drush cim -y
), run theconfig-inspector
report it worked for me. Attaching the screenshots.Actually there is no way to provide a patch a MR for this, as the particular configuration is over rided after making the changes in Layout Paragraphs Modal Settings from
layout_paragraphs
Configuration settings. - last update
about 1 year ago 55 pass - Status changed to Needs review
about 1 year ago 12:37pm 27 March 2024 - 🇩🇪Germany Anybody Porta Westfalica
I don't think this is not fixable... Don't really understand your explanation, sorry.
The key isn't used anywhere in code, but here:
https://git.drupalcode.org/search?search=theme_display&nav_source=navbar...So I think it was just forgotten. @justin2pin what do you say?
- Status changed to RTBC
about 1 year ago 12:49pm 27 March 2024 - 🇩🇪Germany Grevil
I don't understand @MukhtarM concerns either.
The MR LGTM! I think, that the "theme_display" set call was simply unintentionally created in the original implementation of the modal settings form and simply forgotten. See https://git.drupalcode.org/project/layout_paragraphs/-/commit/104917d3c5....
- 🇮🇳India mukhtarm
sorry about that @anybody I was trying to fix the config error, rather not checked the usage in the code. +1 for MR 👍
- Status changed to Fixed
12 months ago 4:08pm 19 April 2024 - 🇩🇪Germany Grevil
Whoops, we forgot to create an update hook cleansing the old config. I'll quickly create a follow-up issue.
- 🇩🇪Germany Grevil
Done: 📌 Cleanse old layout_paragraphs.modal_settings "theme_display" config Needs review .
Automatically closed - issue fixed for 2 weeks with no activity.