Jump to:
| Project: | Related links |
| Version: | 6.x-1.x-dev |
| Component: | Code |
| Category: | bug report |
| Priority: | critical |
| Assigned: | Unassigned |
| Status: | closed (fixed) |
Issue Summary
The regular expression on line 238 in relatedlinks.module seems to grab everything inside the anchor tag, including other HTML tags like
. As the comments in the module point out, the regular expression used here is from the links module, which suffers from the same problem. Since images with hypertext links tend to have text that has the same link, I've modified the regex to ignore anchor tags with HTML tags within them. I posted the new regex for the links module on #156592: img/image links in text being rendered very oddly and replaced the one on line 238 here. If someone wants better handling of this, someone else is going to have to come up with the regex since I'm not really good at it.
See also the patch on issue #300470: Discovered Links do not respect node access which does a similar thing.
Answers part of issue #237352: Add more documentation.
See also issue #125486: No Picture links supported.
Comments
#1
For the time being, we are just stripping any tags within the anchor. Ideally, we would be parsing this and the href itself for title attributes etc. But this'll do for now.
-K
#2
Automatically closed -- issue fixed for 2 weeks with no activity.