Problem/Motivation
Follow up issue after #3252219: Add Container wrapper ID field to the advanced section settings tab
Warning: Undefined array key "container_wrapper_id" in Drupal\varbase_layout_builder\Plugin\Layout\VarbaseLayoutBuilderBootstrapLayout->buildConfigurationForm() (line 504 of /app/docroot/modules/contrib/varbase_layout_builder/src/Plugin/Layout/VarbaseLayoutBuilderBootstrapLayout.php) #0 /app/docroot/core/includes/bootstrap.inc(346): _drupal_error_handler_real(2, 'Undefined array...', '/app/docroot/mo...', 504)
Proposed resolution
Use the parent::defaultConfiguration and the defaultConfiguration to reset the value for the Container wrapper ID
Remaining tasks
- ✅ File an issue about this project
- ✅ Addition/Change/Update/Fix to this project
- ✅ Testing to ensure no regression
- ➖ Automated unit/functional testing coverage
- ➖ Developer Documentation support on feature change/addition
- ➖ User Guide Documentation support on feature change/addition
- ✅ Code review from 1 Varbase core team member
- ✅ Full testing and approval
- ✅ Credit contributors
- ✅ Review with the product owner
- ✅ Update Release Notes and Update Helper on new feature change/addition
- ✅ Release Varbase 9.0.9, varbase_layout_builder-10.0.20
Varbase update type:
- ✅ No Update
- ➖ Optional Update
- ➖ Forced Update
- ➖ Forced Update if Unchanged
User interface changes
- N/A
API changes
- N/A
Data model changes
- Issue #3266114 by abu-zakham: Fixed warnings as a follow up issue after added Container wrapper ID field to the advanced section settings tab
Issue fork varbase_layout_builder-3266114
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
Comments
Comment #4
rajab natshahComment #5
rajab natshahComment #6
rajab natshahComment #7
ahmad abbad commentedPatch #2 worked for me
Comment #8
rajab natshahComment #11
rajab natshahThanks @abu-zakham
Your logic is better on this
Using the
parent::defaultConfigurationand thedefaultConfigurationSome of the default Configurations could be moved to the defaultConfiguration method
Comment #13
rajab natshahComment #14
rajab natshahComment #15
rajab natshahComment #16
rajab natshah✅ Released varbase_layout_builder-10.0.20
Comment #18
rajab natshah✅ Released Varbase 9.0.9