after applying the attached patch, the links below each comment use our standard link hook. this will make it easier for modules to add a link there.

CommentFileSizeAuthor
#4 20040212-comment_links_use_link_hook.patch2.54 KBTDobes
comment_links_use_link_hook.patch2.7 KBmoshe weitzman

Comments

moshe weitzman’s picture

moshe weitzman commented
Assigned: Unassigned » moshe weitzman
gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented

There is no such distinction as main/not main for comments, so the $main parameter should always be zero IMHO. The comments are only listed on the node page, and not on the frontpage. Besides this small change, I am +1 for this patch, since it maintains consistency...

TDobes’s picture

TDobes commented

I'd like to see this patch land before the 4.4 code freeze. It would allow quote.module to work for both nodes and comments, which would greatly improve its usability. +1 from me

TDobes’s picture

TDobes commented
StatusFileSize
new 20040212-comment_links_use_link_hook.patch2.54 KB

oops... forgot to mention that I had to update the patch a bit to get it to apply to present-day CVS... the new patch is attached. Sorry for bugspam.

dries’s picture

dries commented

Committed to HEAD. Thanks.

Anonymous’s picture

(not verified) commented