Batch operations example needs redirect

Good idea. I think the redirect should actually be put into the batch_set array though, as it's clearer what's going on then.

batch_set does not seem to have a redirect parameter, although it would be good to have it.
Should it be set in batch_process($redirect) ?

Hm. If you look at the code in batch_process(), you'll see that it adds the redirect to the $batch that was set with batch_set(), using array addition. So it can actually be added in batch_set()....

But it doesn't actually look like that is done in any of the core functions that call batch_set(). Probably my comment in #1 should be ignored. :)

It looks like you have two issue fixes in the same patch? Please limit the changes to this issue. I think that the other fix is in a different patch that hasn't been committed to Drupal 6 yet, but I'm not sure...

And can you also add a comment to the batch_process() line to indicate that it's being redirected? And preferably replace the comment that is there with a properly-formatted comment. :)
http://drupal.org/node/1354#inline

Yes, that diff was from a earlier patch. Sorry about that.
I pulled from the repo again, added the comment and re-rolled the patch.

Looks OK to me, thanks!

Oops, sorry I didn't notice this before, but this is a Drupal 6 issue, but it's the same doc in D7/8. So we should fix this there first, then go back to the patch in #6 for Drupal 6.

Find patch attached for D8.
Comment 8 is for D8 and comment 6 is for D6.

#8 looks good for Drupal 8/7 thanks! Then please set to RTBC/Drupal 6, where we already have a patch (#6). Thanks!

Committed and pushed to 8.x and 7.x. Thanks!

Bleh. Sorry!

Re-attaching barbi's D6 patch from #6 above, which is RTBC for Drupal 6.

cross-post! :)

Thanks, committed, pushed.