I just found that the Simpletest 2.10 breaks tests invoking the batch API. Many exceptions on undefined variables in maintenance-page.tpl.php occur.

Reproduce:

- download latest Feeds for Drupal 6
- apply #600584: Use Batch API
- run "CSV import to users"

Digging around issues I found #321828-16: maintenance_page not registered correctly in drupal_common_theme() which fixes the problem. I will post a patch to Drupal 6 asap, but I actually wonder whether this couldn't be fixed on a Simpletest level.

CommentFileSizeAuthor
#1 709646-1_batch.patch9.45 KBalex_b

Comments

alex_b’s picture

alex_b commented
Status: Active » Needs review
StatusFileSize
new 709646-1_batch.patch9.45 KB

Status: Needs review » Needs work

The last submitted patch, 709646-1_batch.patch, failed testing.

alex_b’s picture

alex_b commented
Status: Needs work » Needs review

Um, of course this breaks because it's a D6 patch. NR by maintainer nevertheless.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Needs review » Closed (duplicate)

Please review this core patch: #867722: PHP notices when batch is used without JavaScript.