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
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

moshe weitzman’s picture

moshe weitzman CreditAttribution: moshe weitzman commented
Assigned: Unassigned » moshe weitzman
Gábor Hojtsy’s picture

Gábor Hojtsy
he/him
Primary language Hungarian
Location Hungary
CreditAttribution: Gábor Hojtsy 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 CreditAttribution: 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 CreditAttribution: TDobes commented
FileSize
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 CreditAttribution: Dries commented

Committed to HEAD. Thanks.

Anonymous’s picture

(not verified) CreditAttribution: commented