Problem/Motivation

Theme hooks have been implemented in a procedural way rather than in an object-oriented approach. It's recommended to use OO hooks, as they have been supported since Drupal 11.3

Proposed resolution

Update theme hooks to be OO instead of procedural to follow current best practices

Issue fork eureka-3577511

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:

About issue forks

Comments

dabodia created an issue. See original summary.

nexusnovaz made their first commit to this issue’s fork.

nexusnovaz opened merge request !6

nexusnovaz’s picture

nexusnovaz commented
Status: Needs work » Needs review

MR !6 is ready for review!

dabodia’s picture

dabodia commented

Reviewed it and checked locally. I had some issues but pushed another commit to fix them, let me know if anything in my commit looks odd

nexusnovaz’s picture

nexusnovaz commented
Status: Needs review » Reviewed & tested by the community

LGTM - ready to merge

  • dabodia committed d493e661 on 1.0.x authored by nexusnovaz 
    feat: #3577511 Make theme hooks follow D11 OOP approach

By: dabodia
By...
dabodia’s picture

dabodia commented
Status: Reviewed & tested by the community » Fixed

Merged - thanks for the work!

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.

Status: Fixed » Closed (fixed)

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