MINOR rearranged FormResponse::add's to get the right message out to …

…the browser
silverstripe-archive · Nov 1, 2009 · ca5f921 · ca5f921
1 parent 06724ff
commit ca5f921
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 5 deletions.
diff --git a/code/ThreeStep/batchactions/BatchApprovePages.php b/code/ThreeStep/batchactions/BatchApprovePages.php
@@ -10,12 +10,12 @@ function getDoingText() {
 
 	function run(DataObjectSet $pages) {
 		$pageIDs = $pages->column('ID');
+		foreach($pageIDs as $pageID) FormResponse::add("$('Form_EditForm').reloadIfSetTo($pageID);");
+
 		$this->batchaction($pages, 'batchApprove',
 			_t('BatchApprovePages.APPROVED_PAGES', 'Approved %d pages, %d failures')
 		);
 
-		foreach($pageIDs as $pageID) FormResponse::add("$('Form_EditForm').reloadIfSetTo($pageID);");
-
 		return FormResponse::respond();
 	}
 }
diff --git a/code/ThreeStep/batchactions/BatchPublishPages.php b/code/ThreeStep/batchactions/BatchPublishPages.php
@@ -10,12 +10,13 @@ function getDoingText() {
 
 	function run(DataObjectSet $pages) {
 		$pageIDs = $pages->column('ID');
-		$this->batchaction($pages, 'batchPublish',
-			_t('BatchPublishPages.PUBLISHED_PAGES', 'Published %d pages, %d failures')
-		);
 
 		foreach($pageIDs as $pageID) FormResponse::add("$('Form_EditForm').reloadIfSetTo($pageID);");
 
+		$this->batchaction($pages, 'batchPublish',
+			_t('BatchPublishPages.PUBLISHED_PAGES', 'Published %d pages, %d failures')
+		);
+
 		return FormResponse::respond();
 	}
 }