Permalink
Browse files

BUGFIX: Removed regression introduced in 9741d1. With the former in p…

…lace, CMS authors can no longer perform batch publish actions
  • Loading branch information...
1 parent 41faaec commit b154c0f0e7a0abe519e2dc3879951f3618d8652c Russell Michell committed May 30, 2013
Showing with 3 additions and 7 deletions.
  1. +3 −7 admin/code/CMSBatchAction.php
@@ -82,13 +82,9 @@ public function batchaction(SS_List $objs, $helperMethod, $successMessage, $argu
foreach($objs as $obj) {
- // Don't run doPublish() if $obj isn't in a fit state be published
- // @todo Implementation logic _is_ better in CMSBatchAction_Publish#run() but at least objects not matching, still get the batchaction run over them
- if ($helperMethod == 'isPublish' && !$obj->getIsDeletedFromStage()) {
- // Perform the action
- if (!call_user_func_array(array($obj, $helperMethod), $arguments)) {
- $status['error'][$obj->ID] = '';
- }
+ // Perform the action
+ if (!call_user_func_array(array($obj, $helperMethod), $arguments)) {
+ $status['error'][$obj->ID] = '';
}
// Now make sure the tree title is appropriately updated

0 comments on commit b154c0f

Please sign in to comment.