Problem/Motivation

Drupal 11 introduced object-oriented hook implementations via the #[Hook] attribute (change record). Converting the module's procedural hooks to an OOP hook class improves structure and aligns with Drupal core. This applies across the supported branches (an MR is opened per branch as needed).

Steps to reproduce

Proposed resolution

Move hook_preprocess_node__varbase_heroslider() from varbase_heroslider.module into src/Hook/VarbaseHerosliderHooks.php with a #[Hook] attribute. No #[LegacyHook] wrapper (Drupal ~11.4-only branches).

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

User interface changes

  • N/A

API changes

  • N/A

Data model changes

  • N/A

Release notes snippet

  • task: convert the procedural hooks to OOP hook classes
Command icon 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

rajab natshah created an issue. See original summary.

  • rajab natshah committed a6c5ece1 on 1.1.x
    task: #3608318 Convert the procedural hooks to OOP hook classes in the...
rajab natshah’s picture

Status: Active » Fixed
Issue tags: +varbase_heroslider-1.1.3

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

rajab natshah’s picture

Issue tags: +varbase-10.1.2
rajab natshah’s picture

Issue summary: View changes

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.