Problem/Motivation

OOP hooks have landed in core. Let's rewrite the .module file to use the new hooks.

Issue fork alternative_user_emails-3481917

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

ptmkenny created an issue. See original summary.

ptmkenny opened merge request !26

ptmkenny’s picture

ptmkenny commented
Status: Active » Postponed

This will be added, but let's see what the phpstan best practice ends up being for "hook not defined" before committing this.

ptmkenny’s picture

ptmkenny commented

The best practice is that the attribute will be backported to Drupal 10. So we'll commit this after core releases support.

ptmkenny opened merge request !29

ptmkenny closed merge request !26

ptmkenny changed the visibility of the branch oo_hooks2 to hidden.

ptmkenny closed merge request !29

ptmkenny opened merge request !32

ptmkenny’s picture

ptmkenny commented
Title: Use OOP hooks » Require 11.2, use OOP hooks and constraint attribute
ptmkenny’s picture

ptmkenny commented
Status: Postponed » Active

  • ptmkenny committed 0fdf6112 on 1.0.x 
    task: #3481917 Require 11.2, use OOP hooks and constraint attribute

By...
ptmkenny’s picture

ptmkenny commented
Status: Active » Fixed

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.