- 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 bozemanI'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 bozemanYes, 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.