Starting with Drupal 11.1.0, core introduced support for object oriented hook implementations and future versions are expected to phase out traditional procedural hooks.

To prepare for this change, it’s important to update the datalayer module by migrating its existing procedural hook implementations to the new class-based approach.

CommentFileSizeAuthor
#8 datalayer-object-oriented-hooks-3538208-8.patch19.48 KBwaropd

Issue fork datalayer-3538208

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

diegodz created an issue. See original summary.

diegodz’s picture

diegodz commented

I’m happy to work on this and start converting the existing hooks. Any feedback or suggestions are welcome!

diegodz opened merge request !12

diegodz’s picture

diegodz commented
Assigned: diegodz » Unassigned
Status: Active » Needs review

diegodz changed the visibility of the branch 3538208-support-for-object to hidden.

diegodz changed the visibility of the branch 3538208-support-for-object to active.

diegodz’s picture

diegodz commented
waropd’s picture

waropd commented
StatusFileSize
new datalayer-object-oriented-hooks-3538208-8.patch19.48 KB

Static patch of the MR
Any update on a new release?

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

  • eojthebrave committed 36a4840a on 2.1.x authored by diegodz 
    feat: #3538208 Support for object oriented hook implementations

By:...
eojthebrave’s picture

eojthebrave
he/him
Primary language English
Location Minneapolis, MN
commented
Status: Needs review » Fixed

Thanks for working on this. I just merged the MR.

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.