Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.There is no hook_link() in D7; it was removed.
Therefore documentation that says 'the same format as hook_link()' is not very helpful now.
The mentions of 'hook_link' in the docs hook_flag_link() for should be updated, and the actual format explained in full detail.
AFAICT this only returns an array with title and href, at least in our implementation of it. Would be good to check what the module code does with what it gets from the invocation of that hook, just to be sure what its effective spec is!
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | replace_mention_of_hook_link_with_proper_description-1886512_5.patch | 1.23 KB | emartoni |
| #2 | replace_mention_of_hook_link_with_proper_description-1886512.patch | 863 bytes | pbose_drupal |
Comments
Comment #1
laughnanIt seems like this isn't every called for any files in 7.x-3.x. I could write a patch to update the API documentation if you'd like.
Comment #2
pbose_drupal commentedI just went through the module and updated the description. See the patch attached.
Comment #3
rajeevkChanging status to NR to test patch.
Comment #4
emartoni commentedHi @pbose_drupal, how are you?
Patch works, but I found some problems:
* Typo:
Query Parameters which includes desitnation , token etc.* Line break issue:
Comment #5
emartoni commentedPatch attached!
Best,
Eduardo
Comment #6
ivnishDrupal 7 is EOL. Issue will be closed, but patches are still here