Problem/Motivation
Follow up issue after
#3400506: Refactor updates for Varbase Layout Builder by moving hook updates to include/updates with load Include based on major branches
The \Drupal::moduleHandler()->loadInclude function operates slowly and necessitates the installation of the associated module to ensure proper formatting and logical functioning.
Proposed resolution
Change to
// Include all helpers and updates.
include_once __DIR__ . '/includes/helpers.inc';
include_once __DIR__ . '/includes/updates.inc';
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
- ➖ UX/UI designer responsibilities
- ➖ Accessibility and Readability
- ✅ 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.17, varbase-9.1.0, varbase_layout_builder-9.0.40, varbase_layout_builder-10.0.45, varbase-10.0.0-rc1, varbase_layout_builder-10.1.0-beta12
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
- N/A
Release notes snippet
- Issue #3406182: Switched from Drupal module load Include to native PHP include once for Varbase Layout Builder to include all
helpersandupdates
Comments
Comment #6
rajab natshahComment #8
rajab natshah✅ Released varbase_layout_builder-9.0.40
Comment #9
rajab natshah✅ Released varbase_layout_builder-10.0.45
Comment #10
rajab natshah✅ Released varbase_layout_builder-10.1.0-beta12
Comment #11
rajab natshahComment #12
rajab natshahComment #13
rajab natshah✅ Released varbase-9.0.17
Comment #14
rajab natshah✅ Released varbase-10.0.0-rc1