task: #3608370 Remove the #[LegacyHook] shims and procedural files for full OOP hooks in the Varbase Auth module

Rajab Natshahrequested to merge
issue/varbase_auth-3608370:3608370-remove-the-legacyhook into 10.1.x

Removes the redundant backward-compat shims now that all Varbase Auth hooks live in Drupal\varbase_auth\Hook\VarbaseAuthHooks. Deletes varbase_auth.module (the #[LegacyHook] preprocess_page shim) and includes/helpers.inc (the @internal legacy wrapper). The logic remains on the VarbaseAuthHooks service (preprocessPage(), addTemplateVariable()).

Related issue: #3608370

This changeset was produced with AI assistance (Claude Code). Per the Drupal AI-generated contribution policy, it has been reviewed by a human before submission.

Checkpoints

  • 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
Edited by Rajab Natshah

Merge request reports