[5.7] Allow Model destroy method to accept a collection of ids (#25878)

* Allow Model destroy method to accept a collection of ids

* add destroy with collection test

* coding style

* refactor nested ternary

* import Collection as BaseCollection

src/Illuminate/Database/Eloquent/Model.php

@@ -14,6 +14,7 @@
 use Illuminate\Contracts\Queue\QueueableEntity;
 use Illuminate\Database\Eloquent\Relations\Pivot;
 use Illuminate\Contracts\Queue\QueueableCollection;
+use Illuminate\Support\Collection as BaseCollection;
 use Illuminate\Database\Query\Builder as QueryBuilder;
 use Illuminate\Database\ConnectionResolverInterface as Resolver;
 
@@ -825,7 +826,7 @@ protected function insertAndSetId(Builder $query, $attributes)
     /**
      * Destroy the models for the given IDs.
      *
-     * @param  array|int  $ids
+     * @param  \Illuminate\Support\Collection|array|int  $ids
      * @return int
      */
     public static function destroy($ids)
@@ -835,6 +836,10 @@ public static function destroy($ids)
         // type value or get this total count of records deleted for logging, etc.
         $count = 0;
 
+        if ($ids instanceof BaseCollection) {
+            $ids = $ids->all();
+        }
+
         $ids = is_array($ids) ? $ids : func_get_args();
 
         // We will actually pull the models from the database table and call delete on

tests/Database/DatabaseEloquentModelTest.php

@@ -193,6 +193,11 @@ public function testDestroyMethodCallsQueryBuilderCorrectly()
         $result = EloquentModelDestroyStub::destroy(1, 2, 3);
     }
 
+    public function testDestroyMethodCallsQueryBuilderCorrectlyWithCollection()
+    {
+        $result = EloquentModelDestroyStub::destroy(new \Illuminate\Database\Eloquent\Collection([1, 2, 3]));
+    }
+
     public function testWithMethodCallsQueryBuilderCorrectly()
     {
         $result = EloquentModelWithStub::with('foo', 'bar');