Problem/Motivation

There is not more hook_user_insert() in Drupal 8 or greatest versions. The alternative is hook_ENTITY_TYPE_insert().

Steps to reproduce

See in notify.module file, line 113 is referencing a wrong hook.

Proposed resolution

Change hook_user_insert for hook_ENTITY_TYPE_insert().

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#2 3221929-2.patch377 byteslarisse

Comments

larisse created an issue. See original summary.

larisse’s picture

larisse commented
Assigned: larisse » Unassigned
Status: Active » Needs review
StatusFileSize
new 3221929-2.patch377 bytes

I created this patch to reference the right hook.

vipin.mittal18’s picture

vipin.mittal18
He/Him
Primary language English
Location Greater Noida, India
commented
Assigned: Unassigned » vipin.mittal18
Status: Needs review » Reviewed & tested by the community
vipin.mittal18’s picture

vipin.mittal18
He/Him
Primary language English
Location Greater Noida, India
commented
Status: Reviewed & tested by the community » Patch (to be ported)

RTBC +1

gisle’s picture

gisle
he/him
Primary language Norwegian Bokmål
Location Norway
commented
Status: Patch (to be ported) » Reviewed & tested by the community

There is no need to port this patch. It applies directly to the version cited. Setting the correct status.

  • larisse authored 71d4756 on 2.0.x 
    Issue #3221929 by larisse: Fixed reference to wrong hook
gisle’s picture

gisle
he/him
Primary language Norwegian Bokmål
Location Norway
commented
Assigned: vipin.mittal18 » Unassigned
Status: Reviewed & tested by the community » Fixed

Pushed to the latest snapshot of 2.0.x-dev. Thanks for the patch.

larisse’s picture

larisse commented

Thanks!

Status: Fixed » Closed (fixed)

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