Commit 8b9dfcef authored by xjm's avatar xjm

Issue #2400143 by geertvd, pcambra, koence, pjonckiere: Bulk form actions...

Issue #2400143 by geertvd, pcambra, koence, pjonckiere: Bulk form actions display action message in the confirmation form
parent 37e278b4
......@@ -148,7 +148,12 @@ public function testBulkForm() {
}
$edit += array('action' => 'node_delete_action');
$this->drupalPostForm(NULL, $edit, t('Apply'));
// Make sure we don't show an action message while we are still on the
// confirmation page.
$errors = $this->xpath('//div[contains(@class, "messages--status")]');
$this->assertFalse($errors, 'No action message shown.');
$this->drupalPostForm(NULL, array(), t('Delete'));
$this->assertText(t('Deleted 5 posts.'));
// Check if we got redirected to the original page.
$this->assertUrl('test_bulk_form');
}
......
......@@ -291,13 +291,16 @@ public function viewsFormSubmit(&$form, FormStateInterface $form_state) {
);
$form_state->setRedirect($operation_definition['confirm_form_route_name'], array(), $options);
}
if ($count) {
drupal_set_message($this->formatPlural($count, '%action was applied to @count item.', '%action was applied to @count items.', array(
'%action' => $action->label(),
)));
else {
// Don't display the message unless there are some elements affected and
// there is no confirmation form.
$count = count(array_filter($form_state->getValue($this->options['id'])));
if ($count) {
drupal_set_message($this->formatPlural($count, '%action was applied to @count item.', '%action was applied to @count items.', array(
'%action' => $action->label(),
)));
}
}
}
}
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment