Both \Drupal\layout_builder\Form\DefaultsEntityForm and \Drupal\layout_builder\Form\OverridesEntityForm provide a base form ID to allow hook_form_BASE_FORM_ID_alter() implementations to target both of them, in addition to their individual form IDs.
However, DefaultsEntityForm uses the wrong method when generating the base form ID.
Given nodes and users, the respective base form IDs should be:
node_layout_builder_form
node_layout_builder_form
user_layout_builder_form
user_layout_builder_form
They actually are:
node_layout_builder_form
entity_view_display_layout_builder_form
user_layout_builder_form
entity_view_display_layout_builder_form
Fix DefaultsEntityForm::getBaseFormId()
Write tests
N/A
N/A
N/A
N/A
Needs work
11.0 🔥
Last updated
Blocks and Layouts Initiative. See the #2811175 Add layouts to Drupal issue.
The change is currently missing an automated test that fails when run with the original code, and succeeds when the bug has been fixed.
Not all content is available!
It's likely this issue predates Contrib.social: some issue and comment data are missing.
This one might be a little disruptive, quite a few instances of the existing ID being used in contrib and likely many more in client projects.
I suppose thats what CRs are for, I wonder if there is a good way to deprecate the old ID before we change this?