Problem/Motivation
Follow up issue after
#3400109: Refactor updates for Varbase API 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_api-9.0.13, varbase-9.1.0, varbase_api-9.1.1, varbase-10.0.0-rc1, varbase_api-10.0.2
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 #3405344: Switched from Drupal module load Include to native PHP include once for Varbase API to include all
helpersandupdates
Comments
Comment #2
rajab natshahComment #7
rajab natshahComment #8
rajab natshah✅ Released varbase_api-9.0.13
Comment #9
rajab natshah✅ Released varbase_api-9.1.1
Comment #10
rajab natshah✅ Released varbase_api-10.0.2
Comment #11
rajab natshahComment #12
rajab natshah✅ Released varbase-9.0.17
Comment #13
rajab natshah✅ Released varbase-10.0.0-rc1