Contrary to #89181: Use queue API for node and comment, user, node multiple deletes, this patch needs to be backported to D6.

CommentFileSizeAuthor
#2 drupal.comment-notice-redirect-d6.patch2.76 KBsun
drupal.comment-notice-redirect.0.patch3.26 KBsun

Comments

dries’s picture

dries commented
Version: 7.x-dev » 6.x-dev
Status: Needs review » Reviewed & tested by the community

Committed to CVS HEAD. Moving to Drupal 6.

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
StatusFileSize
new drupal.comment-notice-redirect-d6.patch2.76 KB

Proper patch for D6.

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented

Why is that a problematic redirect?

sun’s picture

sun
Primary language German
Location Karlsruhe
commented 
+++ modules/comment/comment.admin.inc	5 Jan 2010 16:23:45 -0000
@@ -84,7 +84,6 @@ function comment_admin_overview_validate
   if (count($form_state['values']['comments']) == 0) {
     form_set_error('', t('Please select one or more comments to perform the update on.'));
-    drupal_goto('admin/content/comment');
   }

In case of the form validation error, this redirects to a hard-coded path (which is entirely incorrect for the form on admin/content/comment/approval) and which also means that you lose all your form values! :)

Powered by Dreditor.

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented
Status: Reviewed & tested by the community » Fixed

Ok, committed, thanks.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.