Deprecate Model::drop(), use Model::unset() instead

Author: GromNaN

Diff for: src/Eloquent/Model.php

@@ -20,9 +20,6 @@
 use MongoDB\BSON\UTCDateTime;
 use function uniqid;
 
-/**
- * @method void unset(string|string[] $columns) Remove one or more fields.
- */
 abstract class Model extends BaseModel
 {
     use HybridRelations, EmbedsRelations;
@@ -341,8 +338,21 @@ public function offsetSet($offset, $value): void
      *
      * @param  string|string[]  $columns
      * @return void
+     *
+     * @deprecated Use unset() instead.
      */
     public function drop($columns)
+    {
+        $this->unset($columns);
+    }
+
+    /**
+     * Remove one or more fields.
+     *
+     * @param  string|string[]  $columns
+     * @return void
+     */
+    public function unset($columns)
     {
         $columns = Arr::wrap($columns);
 
@@ -560,19 +570,6 @@ protected function isGuardableColumn($key)
         return true;
     }
 
-    /**
-     * @inheritdoc
-     */
-    public function __call($method, $parameters)
-    {
-        // Unset method
-        if ($method == 'unset') {
-            return $this->drop(...$parameters);
-        }
-
-        return parent::__call($method, $parameters);
-    }
-
     /**
      * @inheritdoc
      */

Diff for: tests/ModelTest.php

@@ -511,7 +511,7 @@ public function testUnsetAndSet(): void
         $this->assertTrue($user->originalIsEquivalent('note1'));
 
         // Unset the value
-        unset($user['note1']);
+        $user->unset('note1');
         $this->assertFalse(isset($user->note1));
         $this->assertNull($user['note1']);
         $this->assertFalse($user->originalIsEquivalent('note1'));