FIX #579 BUG Ambiguous column RecordID when doing batch actions

silverstripe · Jul 26, 2018 · fea9ef7 · fea9ef7
1 parent 8e5905f
commit fea9ef7
Showing 1 changed file with 1 addition and 3 deletions.
diff --git a/code/CMSBatchActionHandler.php b/code/CMSBatchActionHandler.php
@@ -327,9 +327,7 @@ protected function getPages($ids)
         if ($recordClass::has_extension(Versioned::class)) {
             // Workaround for get_including_deleted not supporting byIDs filter very well
             // Ensure we select both stage / live records
-            $pages = Versioned::get_including_deleted($recordClass, array(
-                '"RecordID" IN ('.DB::placeholders($ids).')' => $ids
-            ));
+            $pages = Versioned::get_including_deleted($recordClass)->byIDs($ids);
         } else {
             $pages = DataObject::get($recordClass)->byIDs($ids);
         }