tonyair/silverstripe-reports
1
0
Fork 0
mirror of https://github.com/silverstripe/silverstripe-reports synced 2024-10-22 11:05:53 +02:00
Code Issues Projects Releases Wiki Activity

MINOR Making controller/response optional in CMSBatchAction->batchaction() in order to make the class more testable in isolation

Browse Source 
git-svn-id: svn://svn.silverstripe.com/silverstripe/open/modules/cms/trunk@115128 467b73ca-7a2a-4603-9d3b-597d59a354a9
This commit is contained in:
Ingo Schommer 2010-12-16 08:33:06 +00:00
parent 2afc2e99dd
commit 2d3bb7de3e
1 changed files with 7 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
code/CMSBatchAction.php
View File

@ -64,10 +64,13 @@ abstract class CMSBatchAction extends Object {
unset($page);
}
Controller::curr()->getResponse()->setStatusCode(
200,
sprintf($successMessage, $pages->Count(), count($status['error']))
);
$response = Controller::curr()->getResponse();
if($response) {
$response->setStatusCode(
200,
sprintf($successMessage, $pages->Count(), count($status['error']))
);
}
return Convert::raw2json($status);
}