Section ID is lost when layout_bulder_ids is used in conjunction with other modules like bootstrap_layouts or varbase_layout_builder.

Created on 17 September 2020, over 4 years ago
Updated 11 August 2023, over 1 year ago

Problem/Motivation

This module provides a configuration, stored temporary on layout_settings at form_state, but it is the same form_state storage key used by other modules like bootstrap_layouts or varbase_layout_builder.

As consequence, when layout_builder_ids is used together with one of those modules, the section id is not saved properly.

Steps to reproduce

  • Install layout_builder_ids + bootstrap_layouts or layout_builder_ids + varbase_layout_builder.
  • Configure a content type to use layout builder
  • Add a section and set a section ID. Save the section.
  • Configure the section again. The section ID has been lost

Proposed resolution

Store the form_state in a key different from layout_settings. For example, can be used directly layout_builder_id (in level array)

Remaining tasks

Review and test

User interface changes

None

API changes

None

Data model changes

layout_builder_id is saved on the first level of the form_state storage array.

🐛 Bug report
Status

Needs work

Version

2.0

Component

Code

Created by

🇪🇸Spain akalam

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

  • 🇨🇦Canada ebremner

    Sorry I have not been able to look at this in awhile but I am finally getting back to it. I agree that some of these changes can be made, if someone would like to a crack at again using all the latest 2.0.x, then I will have a look too. I think it it is going to need an update hook for any existing ids for sections and blocks.

Production build 0.71.5 2024