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 {

CommentFileSizeAuthor
#3 diff24.84 KBjik
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jik’s picture

Dries’s picture

I 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.

jik’s picture

FileSize
4.84 KB

You'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.

jik’s picture

Note 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.

jik’s picture

Woops, forgot to delete the $Id$ line at the top of the patch. Please ignore it.

Dries’s picture

I 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.

Dries’s picture

Category: bug » task
alexandreracine’s picture

Version: » 4.5.0

Was 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.

forngren’s picture

Status: Active » Closed (fixed)

Fixed in later versions