Fix incorrect Implements hook_*_* docblock in token module.

  1. Find all the docblock those are in format of Implements hook_*_* with command: grep -rn 'Implements hook_.*_.*\(\).' modules/contrib/token
  2. Fix each docblock which is not mentioning a valid hook.
CommentFileSizeAuthor
#2 fix-incorrect-docblock-implements-3005173.patch1.04 KBmsankhala
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

msankhala created an issue. See original summary.

msankhala’s picture

msankhala CreditAttribution: msankhala at Srijan | A Material+ Company commented
Status: Active » Needs review
FileSize
fix-incorrect-docblock-implements-3005173.patch1.04 KB

Here is the patch.

longwave’s picture

longwave
he/him
Primary language English
Location UK
CreditAttribution: longwave as a volunteer commented
Status: Needs review » Reviewed & tested by the community

Nice cleanup considering three of these were entirely wrong!

catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch at Third and Grove commented
Status: Reviewed & tested by the community » Closed (duplicate)

See comments at #3005001: [Meta] Fix incorrect Implements hook_*_* docblock in core. Marking this as duplicate for now.

Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir at MD Systems GmbH commented
Status: Closed (duplicate) » Reviewed & tested by the community

@catch this isn't core so it can't be combined into a single patch ;)

idebr’s picture

idebr CreditAttribution: idebr at ezCompany commented

Patch still applies cleanly to 8.x-1.x HEAD

  • Berdir committed d3474fb on 8.x-1.x authored by msankhala 
    Issue #3005173 by msankhala: Fix incorrect Implements hook_*_* docblock...
Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir at MD Systems GmbH commented
Status: Reviewed & tested by the community » Fixed

Committed.

Status: Fixed » Closed (fixed)

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