Comment Page 6.x-1.1 not working?
JohnNoc - August 26, 2008 - 00:56
| Project: | Comment Page |
| Version: | 6.x-1.1 |
| Component: | Code |
| Category: | bug report |
| Priority: | critical |
| Assigned: | Unassigned |
| Status: | needs review |
Jump to:
Description
Installed and tested on two different scenarios:
- Major bug on a fresh install of Drupal 6.4 with Comment Page 6.x-1.1
- Working great on Drupal 5.10 with Comment Page 5.x-1.1 but after upgrade to Drupal 6.4 with Comment Page 6.x-1.1, the same problem as above occurs.
- The problem
- All comment pages go to just one comment page - the first published comment. The url on the address bar is correct, but the page is still the first published comment. This occurs on all comment page links, may it be configured with url
.../comment/* or .../comment/*/node-title. All the same.
#1
My problem is a bit different ... but comment_page still not working
I've got the link to the comment page on the comments ... but:
/comment/x <= not found 404
Any idea ?
#2
I find and fix this bug.
it is in "function comment_page_display($comment_id)" in file comment_page.module -
variable $comment_id is not cid in Drupal 6
I change "$comment = _comment_load($comment_id);" at line 174
to "$comment = _comment_load($comment_id->cid);"
and all work fine
you can see how it is work now at:
http://dig.by/comment/4 and
http://dig.by/comment/3
#3
#2 worked for me.
#4
I am not a PHP or CVS guru. My knowlege of PHP is "How to make Hello-Word application".
If my patch work fine, maybe anyone add it to new release?
#5
I'm using Drupal 6.6, and I'm definitely experiencing this bug. All linked comment pages show up as "404 - page not found" instead of displaying any actual content.
The suggested patch at #2 seems to solve the problem. Any chance we could get a patched release?
#6
Yup, this one-line change did it for me :
NEW : $comment = _comment_load($comment_id->cid);
Thanks,
David
Social Security Help
#7
ditto, #299933-2: Comment Page 6.x-1.1 not working? fixed it for me
#8
Works on D 6.13.
Commit this please, it is already late for months.