Permalink
Browse files

BUGFIX: Add batch handler status messages (fixes #7427)

7427 was mostly fixed by Ingos previous patch. But two batch actions, delete from draft site and delete from published site werent returning
status messages. Abstracted out the status preperation code that the batch actions that were returning status messages were using, and
used that to add status messages to the problem two
  • Loading branch information...
1 parent 211ce61 commit 8c05f3554fe80bd6fbf1aca47ad4fabaaeeb30b8 Hamish Friedlander committed Jun 22, 2012
Showing with 2 additions and 2 deletions.
  1. +2 −2 code/batchactions/CMSBatchActions.php
@@ -78,7 +78,7 @@ function run(SS_List $pages) {
}
- return Convert::raw2json($status);
+ return $this->response(_t('CMSBatchActions.DELETED_DRAFT_PAGES', 'Deleted %d pages from draft site, %d failures'), $status);
}
function applicablePages($ids) {
@@ -123,7 +123,7 @@ function run(SS_List $pages) {
}
- return Convert::raw2json($status);
+ return $this->response(_t('CMSBatchActions.DELETED_PAGES', 'Deleted %d pages from published site, %d failures'), $status);
}
function applicablePages($ids) {

0 comments on commit 8c05f35

Please sign in to comment.