|
[task] comment module uses standard _link() hook: msg#00522php.drupal.devel
Project: Drupal Version: cvs Component: comment.module Category: tasks Priority: normal Assigned to: weitzman@xxxxxxxxxxxx Reported by: weitzman@xxxxxxxxxxxx Updated by: TDobes Status: patch 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 Previous comments: ------------------------------------------------------------------------ December 29, 2003 - 22:44 : weitzman@xxxxxxxxxxxx Attachment: http://drupal.org/files/issues/comment_links_use_link_hook.patch (2.7 KB) 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. ------------------------------------------------------------------------ January 31, 2004 - 08:43 : Goba 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... -- View: http://drupal.org/node/view/4861 Edit: http://drupal.org/project/comments/add/4861 |
|
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| Previous by Date: | [feature] Vocab and Terms in Overview should be 'localizable': 00522, mic |
|---|---|
| Next by Date: | [task] comment module uses standard _link() hook: 00522, TDobes |
| Previous by Thread: | [feature] Vocab and Terms in Overview should be 'localizable'i: 00522, mic |
| Next by Thread: | [task] comment module uses standard _link() hook: 00522, TDobes |
| Indexes: | [Date] [Thread] [Top] [All Lists] |
| News | FAQ | advertise |