Closed (fixed)
Project:
link node
Version:
7.x-1.0
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
31 Dec 2011 at 20:47 UTC
Updated:
24 Nov 2015 at 19:58 UTC
Jump to comment: Most recent
case 1:
The following text:
This is a link to the [node:61] node
This is a link to the [node:61,title="foo"] with an alternate title
This is a link to the [node:61,title="bar"] with an alternate titleProduces the following output:
This is a link to the Service and Outreach node
This is a link to the Service and Outreach with an alternate title
This is a link to the Service and Outreach with an alternate titlecase 2:
The following text:
This is a link to the [node:61,title="foo"] with an alternate title
This is a link to the [node:61,title="bar"] with an alternate title
This is a link to the [node:61] nodeProduces the following output:
This is a link to the foo with an alternate title
This is a link to the bar with an alternate title
This is a link to the bar node
Comments
Comment #1
reikel commentedCase 1 is caused by a bug at line 99 of link_node.module, where $meta_tag_params_regexp is changed to $meta_tag_params_regexp2 inside the foreach( match[1] ) loop. Thus, after $meta_tag_params_regexp fails to match once, it is never tried again.
Here are the diffs to fix the bug:
Still looking at Case 2. I'm guessing it's caused by some internal drupal node caching, so that node_load() at line 87 does not actually get a fresh copy of the node each time.
Comment #2
reikel commentedCase 2 is indeed caused by internal node caching, which apparently is new in Drupal 7, at least according to this guy: http://www.computerminds.co.uk/articles/drupal-7-nodeload-references
Per the suggestion in his comment thread, here are the diffs to fix the bug:
Comment #4
TomChiverton commentedWill be in next release
Comment #5
TomChiverton commented