Closed (fixed)
Project:
Documentation
Component:
Documentation in CVS
Priority:
Minor
Category:
Bug report
Assigned:
Reporter:
Created:
5 Jan 2007 at 19:05 UTC
Updated:
20 Jan 2007 at 02:16 UTC
Jump to comment: Most recent file
The documentation on hook_link doesn't mention that having a title without an href will be themed without running through a l() call. This is a useful fact to know, and is contrary to the current documentation.
| Comment | File | Size | Author |
|---|---|---|---|
| #8 | hook_link_clarification_1.patch | 2.62 KB | bdragon |
| #6 | hook_link_clarification_0.patch | 2.63 KB | bdragon |
| hook_link_clarification.patch | 2.17 KB | bdragon |
Comments
Comment #1
fax8 commented+1 Original documentation is really confused and incorrect.
Fabio Varesano
Comment #2
Caleb G2 commentedFor us following along at home - why is this useful to know? (example?)
Comment #3
bdragon commentedPeople porting custom 4.7 modules that used non-link links to show summary statistics and such.
*slaps forehead*
Doh! Forgot to provide an example....
Comment #4
fax8 commentedmy video module add video size and lenght to links in 4.7
(example: http://www.windsmpeg.tk/video+type/full+video)
they are only simple text and reading hook_link original
documentation I was not able to understand that I can also add simple text.
Comment #5
bdragon commentedC'mon man, leave the status alone for a few minutes ;). I'll CNR it when I have the updated patch ready . ;)
I'm using your example for the code section, btw.
Comment #6
bdragon commentedOk, this one has an example too...
Comment #7
RobRoy commentedI'd change and to &&. Besides that looks pretty good.
Comment #8
bdragon commentedWhoops, agree there, && is the way to go there... Shoulda read it better when copy/pasting.
Comment #9
bdragon commentedGoing once, going twice, SOLD!
Comment #10
(not verified) commented