Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
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.
Comment | File | Size | Author |
---|---|---|---|
#5 | comment.module_30.patch | 1.02 KB | ixis.dylan |
Comments
Comment #1
ixis.dylan CreditAttribution: ixis.dylan commentedCan anybody confirm this? I can't find the problem.
Comment #2
ixis.dylan CreditAttribution: ixis.dylan commentedHello?
Comment #3
Uwe Hermann CreditAttribution: Uwe Hermann commentedConfirmed in HEAD. The problem is here (comment.module, comment_render()):
The
$pager
variable is empty (for whatever reason), hence no pager is generated. I didn't debug this any further...Comment #4
pfaocleConfirmed here too - can any one take a look at this?
Comment #5
ixis.dylan CreditAttribution: ixis.dylan commentedComment 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...
Comment #6
tostinni CreditAttribution: tostinni commentedI also posted a patch there.
But there's still some major bug with comments...
Comment #7
ixis.dylan CreditAttribution: ixis.dylan commentedOh, okay. I wish I'd seen your patch before I hunted this bug down myself, but thanks anyway.
You're right, the comment module is very broken right now. Most of the problems are related to the pager or the comment options box, but there are limitations with this module that are really quite suprising. Can we still not delete a single comment in a thread, for example? This is evil, and wrong.
Comment #8
pfaocleAgreed on the deleting one comment only ting, here's some links:
http://drupal.org/node/11877
http://www.settingtheworldtorights.com/node/417 (search for 'delete a single comment')
Comment #9
pfaocleWhoops - wrong version. Maybe this'll get noticed now... ;)
Comment #10
Robin Monks CreditAttribution: Robin Monks commented+1, looks good, style seems fine...it's just a typo, get this in HEAD!!!
Comment #11
pfaocle+1
Aye!
Comment #12
Dries CreditAttribution: Dries commentedLooks like this has been committed on 07-Oct-05?
Comment #13
(not verified) CreditAttribution: commented