@gisle I just used this great module on a project. I found it very useful and well done. Thanks for your work.

api.php document is incorrect given the following in .module file

      drupal_alter('freelink', $link, $data);

It's plain to see only two parameters are passed to hook_freelink_alter(). However the api.php signature does not match.
Same is true for hook_freelinking_alter().

See patch for fix.

Please provide credit on any commit.

CommentFileSizeAuthor
#4 freelinking-api-fix_4.patch1.54 KBgisle
freelinking-api-fix.patch1.54 KBAnonymous (not verified)
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented

pferlito created an issue. See original summary.

gisle’s picture

gisle
he/him
Primary language Norwegian Bokmål
Location Norway
CreditAttribution: gisle commented
Issue summary: View changes
gisle’s picture

gisle
he/him
Primary language Norwegian Bokmål
Location Norway
CreditAttribution: gisle commented
FileSize
freelinking-api-fix_4.patch1.54 KB

Same patch as before - just to trigger the automated test.

  • gisle committed 17e0240 on 7.x-3.x authored by pferlito 
    Issue #2615054 by pferlito: Fixed API Doc error
gisle’s picture

gisle
he/him
Primary language Norwegian Bokmål
Location Norway
CreditAttribution: gisle commented
Status: Needs review » Fixed

Pushed to the most recent snapshot of 7.x-3.x-dev.

Status: Fixed » Closed (fixed)

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