task: #3608373 Convert the remaining procedural helpers/tokens to the OOP hook class and remove the procedural files (9.2.x)

Rajab Natshahrequested to merge
issue/varbase_media-3608373:3608373-9.2.x into 9.2.x

Moves the remaining procedural code into the existing \Drupal\varbase_media\Hook\VarbaseMediaHooks class: the shared helper functions (includes/helpers.inc), the tokens (varbase_media.tokens.inc → hook_token_info/hook_tokens) and the bulk-upload submit callback (now [$this, 'bulkUploadFormSubmitHandler'] with DependencySerializationTrait). Adds file URL generator, theme manager and config manager to the constructor. Removes the #[LegacyHook] varbase_media.module, includes/helpers.inc and varbase_media.tokens.inc; the update-hook caller in includes/updates/v8.inc is rewired to the service (change record).

Related issue: #3608373

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