[drupal-devel] [bug] Comment pager not working in cvs?

tostinni drupal-devel at drupal.org
Mon Sep 19 17:21:40 UTC 2005

Issue status update for 
Post a follow up: 

 Project:      Drupal
 Version:      cvs
 Component:    comment.module
 Category:     bug reports
 Priority:     critical
 Assigned to:  leafish_dylan
 Reported by:  leafish_dylan
 Updated by:   tostinni
 Status:       patch (code needs review)

I also posted a patch there [1].
But there's still some major bug with comments...
[1] http://drupal.org/node/26966#comment-41012


Previous comments:

Wed, 29 Jun 2005 23:43:10 +0000 : leafish_dylan

I've just upgraded a 4.6.1 site to CVS and there's no comment pager on
nodes with more than $comments_per_page. Is this a bug, or it it one of
my modules? I've tested the site with various other themes, including
the defaults.


Mon, 04 Jul 2005 23:41:03 +0000 : leafish_dylan

Can anybody confirm this? I can't find the problem.


Fri, 15 Jul 2005 12:35:14 +0000 : leafish_dylan



Tue, 23 Aug 2005 14:12:16 +0000 : Uwe Hermann

Confirmed in HEAD. The problem is here (comment.module,

      if ($pager = theme('pager', NULL, $comments_per_page, 0,
array('comments_per_page' => $comments_per_page))) {
        $output .= $pager;

The $pager variable is empty (for whatever reason), hence no pager is
generated. I didn't debug this any further...


Fri, 09 Sep 2005 21:11:26 +0000 : leafish_paul

Confirmed here too - can any one take a look at this?


Mon, 19 Sep 2005 15:52:20 +0000 : leafish_dylan

Attachment: http://drupal.org/files/issues/comment.module_30.patch (1.02 KB)

Comment paging has been broken in CVS for at least 3 months, yet only
one other person has noticed this? 

Here's a patch to fix the bug...

More information about the drupal-devel mailing list