Problem/Motivation
Varbase Total Control Dashboard grants role permissions via Vardot\Installer\ModuleInstallerFactory::addPermissions() (scanning config/permissions), which silently skips a missing role instead of validating. Following the Drupal core Recipe approach already used on Varbase API, Varbase Media, Varbase Blog, Varbase Bootstrap Paragraphs and Varbase Carousels makes this declarative and validated.
Steps to reproduce
Proposed resolution
Move the role permission grants into recipes/default/recipe.yml (config actions only, module installation stays as-is), drop config/permissions/, and call the recipe at the end of varbase_total_control_install() instead of addPermissions().
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
- ✅ Reviewed by a human
- ✅ Code review by maintainers
- ✅ Full testing and approval
- ✅ Credit contributors
- ✅ Review with the product owner
- ✅ Update Release Notes
- ✅ Release varbase-9.2.0, varbase_total_control-9.2.0
User interface changes
- N/A
API changes
- N/A
Data model changes
- N/A
Release notes snippet
task: #3608640 Switch to a Default Varbase Total Control Dashboard recipe and drop the use of the Module Installer Factory class for permissions
Issue fork varbase_total_control-3608640
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 natshah✅ Released varbase_total_control-9.2.0
Comment #6
rajab natshah✅ Released varbase-9.2.0