Batch system error on redirection [#617420]

I have not found a suitable category for 'possibly' bugs in the batch system.

Line 450 of the batch.inc file is:

      $function($_batch['source_url'], array('query' => array('op' => 'finish', 'id' => $_batch['id'])));

This is something that should happen when the origin form contains a redirection. The problem is that I have not found the origin of this variable, but I've found "source_page", used in this batch.inc file, and created in form.inc, in the function batch_process().

Is source_url an error and should be source_page? of course I'm getting a nasty red error message.

Comment	File	Size	Author
#3	617420-batchapi-source_url.patch	921 bytes	ksenzee
#3

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

ksenzee

she/her

Seattle area

CreditAttribution: ksenzee commented 28 October 2009 at 21:18

Category:

support

» bug

Looks like a bug to me, introduced in #555762: Changes to batch API:

-    drupal_goto($_batch['source_page']);
+    $function = $_batch['redirect_callback'];
+    if (function_exists($function)) {
+      $function($_batch['source_url'], array('op' => 'finish', 'id' => $_batch['id']));
+    }