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.
If I go to administer > comments > overview and click on "comment approval queue", I don't see the comment approval queue -- I see already approved comments.
I think this needs to happen:
--- comment.module 2004/03/11 06:15:01 1.4
+++ comment.module 2004/03/11 07:20:26
@@ -1245,7 +1255,7 @@
$output .= comment_admin_overview(0);
break;
default:
- if (arg(3) == 1) {
+ if ($op == 1) {
$output = comment_admin_overview(1);
}
else {
Comments
Comment #1
jik CreditAttribution: jik commentedComment #2
Dries CreditAttribution: Dries commentedI can't reproduce this problem with either Drupal 4.3.2, Drupal 4.4.0-RC or Drupal HEAD: using
arg(3)
looks valid to me.Comment #3
jik CreditAttribution: jik commentedYou're right, the arg(3) is working in 4.4.0-rc, but only because the layout of the URLs changed. However, in investigating this I found three other related problems in 4.4.0-rc, for which a patch is attached (I don't think this patch is dependent on the others I'm submitting tonight, but I suppose it may be):
The link for the comment approval queue should be
admin/comment/comments/1, not admin/comment/moderation. Making it
admin/comment/moderation is confusing community-based moderation with
the original intent of this link, which was to allow administrators to
approve unpublished comments.
Fix a typo ("it's" should be "its").
After editing a comment, return to the comments list we were on
before. That is, if we were viewing the list of unpublished comments,
return to that list. This makes it easier to approve unpublished
comments one by one from the list of unpublished comments.
Previously, the code would always display a list of already published
comments after editing a comment.
Comment #4
jik CreditAttribution: jik commentedNote that this patch is against 4.4.0-rc, but I can't select that in the version drop-down, so I'm selectin cvs, which is the closest match.
Comment #5
jik CreditAttribution: jik commentedWoops, forgot to delete the $Id$ line at the top of the patch. Please ignore it.
Comment #6
Dries CreditAttribution: Dries commentedI fixed the broken URL and the typo in both HEAD and DRUPAL-4-4 but I'm not happy with the 'afterstatus' stuff. We need to think of a better (more generic) way to remember the page we came from.
Comment #7
Dries CreditAttribution: Dries commentedComment #8
alexandreracine CreditAttribution: alexandreracine commentedWas for version x.y.z.
Is this still valid?
Drupal.org could use the adminblock module...
I'll close this next time I'll come by.
Comment #9
forngren CreditAttribution: forngren commentedFixed in later versions