- Patch #683814 by yched: remove unneeded call to batch_process().
parent
c1d2d11f92
commit
3fc8b8b3f3
|
@ -165,13 +165,7 @@ function simpletest_run_tests($test_list, $reporter = 'drupal') {
|
|||
|
||||
module_invoke_all('test_group_started');
|
||||
|
||||
// Normally, the forms portion of the batch API takes care of calling
|
||||
// batch_process(), but in the process it saves the whole $form into the
|
||||
// database (which is huge for the test selection form).
|
||||
// By calling batch_process() directly, we skip that behavior and ensure
|
||||
// that we don't exceed the size of data that can be sent to the database
|
||||
// (max_allowed_packet on MySQL).
|
||||
batch_process('admin/config/development/testing/results/' . $test_id);
|
||||
return $test_id;
|
||||
}
|
||||
|
||||
/**
|
||||
|
|
|
@ -190,7 +190,8 @@ function simpletest_test_form_submit($form, &$form_state) {
|
|||
}
|
||||
}
|
||||
if (count($tests_list) > 0 ) {
|
||||
simpletest_run_tests($tests_list, 'drupal');
|
||||
$test_id = simpletest_run_tests($tests_list, 'drupal');
|
||||
$form_state['redirect'] = 'admin/config/development/testing/results/' . $test_id;
|
||||
}
|
||||
else {
|
||||
drupal_set_message(t('No test(s) selected.'), 'error');
|
||||
|
|
Loading…
Reference in New Issue