From c7a275a7a680bf4c0210a1ee478b5e591c3d3bbf Mon Sep 17 00:00:00 2001
From: Rob Ingram <rob@headcrash.org>
Date: Thu, 29 Jun 2017 18:33:22 +1200
Subject: [PATCH] Make delete handler respect `canDelete` (#142)

---
 bulkManager/code/GridFieldBulkActionDeleteHandler.php | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/bulkManager/code/GridFieldBulkActionDeleteHandler.php b/bulkManager/code/GridFieldBulkActionDeleteHandler.php
index b6f5f06..cd9292e 100644
--- a/bulkManager/code/GridFieldBulkActionDeleteHandler.php
+++ b/bulkManager/code/GridFieldBulkActionDeleteHandler.php
@@ -1,7 +1,7 @@
 <?php
 /**
  * Bulk action handler for deleting records.
- * 
+ *
  * @author colymba
  */
 class GridFieldBulkActionDeleteHandler extends GridFieldBulkActionHandler
@@ -24,7 +24,7 @@ class GridFieldBulkActionDeleteHandler extends GridFieldBulkActionHandler
 
     /**
      * Delete the selected records passed from the delete bulk action.
-     * 
+     *
      * @param SS_HTTPRequest $request
      *
      * @return SS_HTTPResponse List of deleted records ID
@@ -34,8 +34,10 @@ class GridFieldBulkActionDeleteHandler extends GridFieldBulkActionHandler
         $ids = array();
 
         foreach ($this->getRecords() as $record) {
-            array_push($ids, $record->ID);
-            $record->delete();
+            if ($record->canDelete()) {
+                array_push($ids, $record->ID);
+                $record->delete();
+            }
         }
 
         $response = new SS_HTTPResponse(Convert::raw2json(array(