- Issue created by @trickfun
This completely prevents a user from changing module settings in the admin interface, short of modifying the config YAML.
Please fix this.
Even if I add an IF that skips using the entity_type_id of
menu_link_content
(this is the one that causes the issue for me, since the settings form is picking the first one using the layout builder, this is what it picks for me), I still get an error:Drupal\Component\Plugin\Exception\PluginNotFoundException: The "layout_twocol_section" plugin does not exist. Valid plugin IDs for Drupal\Core\Layout\LayoutPluginManager are: layout_onecol, with_sidebar in Drupal\Core\Plugin\DefaultPluginManager->doGetDefinition() (line 53 of core/lib/Drupal/Component/Plugin/Discovery/DiscoveryTrait.php). Drupal\Core\Plugin\DefaultPluginManager->getDefinition() (Line: 16) Drupal\Core\Plugin\Factory\ContainerFactory->createInstance() (Line: 83) Drupal\Component\Plugin\PluginManagerBase->createInstance() (Line: 116) Drupal\layout_builder\Section->getLayout() (Line: 218) Drupal\lb_plus\Element\LayoutBuilderPlus->buildAdministrativeSection() (Line: 127) Drupal\lb_plus\Element\LayoutBuilderPlus->layout() (Line: 99) Drupal\lb_plus\Element\LayoutBuilderPlus->preRender() call_user_func_array() (Line: 113)
- 🇺🇸United States tim bozeman
I'm sorry that's happening. I made a wrong assumption that every site would have those layouts when I hard coded those two layout_id's on line 110 and 113 of SettingsForm.php.
Maybe that part should look up what layout ID's are available? Or maybe even just make that a static page that just shows some example markup for the colors if anything at all. As a work around until we get a chance to fix it, I think it's pretty safe to just comment out lines 77 through 116 😓
Yes, I found out I can view the form that way. Another workaround could be to add a try/catch.
A proper solution would be to use some static example markup that showcases some blocks in a layout, I think.
- 🇺🇸United States tim bozeman
Yes, I agree. Trying to put a random Layout Builder element on the settings page was a bad idea. Sorry about that 😅
I guess priority should be "Major" since not everyone is affected.
This patch is just a workaround that removes the layout form element.