feat!: use mongodb function to check for dirtiness

Co-Authored-By: Hamid Alaei Varnosfaderani <[email protected]>

Author: divine

composer.json

@@ -23,7 +23,8 @@
         "illuminate/container": "^10.0",
         "illuminate/database": "^10.0",
         "illuminate/events": "^10.0",
-        "mongodb/mongodb": "^1.15"
+        "mongodb/mongodb": "^1.15",
+        "ext-mongodb": "*"
     },
     "require-dev": {
         "phpunit/phpunit": "^9.5.10",

src/Eloquent/Model.php

@@ -18,6 +18,8 @@
 use MongoDB\BSON\Binary;
 use MongoDB\BSON\ObjectID;
 use MongoDB\BSON\UTCDateTime;
+use function MongoDB\BSON\fromPHP;
+use MongoDB\Driver\Exception\UnexpectedValueException;
 use function uniqid;
 
 abstract class Model extends BaseModel
@@ -259,20 +261,11 @@ public function originalIsEquivalent($key)
             return false;
         }
 
-        if ($this->isDateAttribute($key)) {
-            $attribute = $attribute instanceof UTCDateTime ? $this->asDateTime($attribute) : $attribute;
-            $original = $original instanceof UTCDateTime ? $this->asDateTime($original) : $original;
-
-            return $attribute == $original;
-        }
-
-        if ($this->hasCast($key, static::$primitiveCastTypes)) {
-            return $this->castAttribute($key, $attribute) ===
-                $this->castAttribute($key, $original);
+        try {
+            return (fromPHP([$attribute]) === fromPHP([$original]));
+        } catch (UnexpectedValueException $e) {
+            return false;
         }
-
-        return is_numeric($attribute) && is_numeric($original)
-            && strcmp((string) $attribute, (string) $original) === 0;
     }
 
     /**

tests/ModelTest.php

@@ -715,13 +715,48 @@ public function testMultipleLevelDotNotation(): void
     public function testGetDirtyDates(): void
     {
         $user = new User();
-        $user->setRawAttributes(['name' => 'John Doe', 'birthday' => new DateTime('19 august 1989')], true);
+        $user->name = 'John Doe';
+        $user->birthday = new DateTime('19 august 1989');
+        $user->syncOriginal();
         $this->assertEmpty($user->getDirty());
 
         $user->birthday = new DateTime('19 august 1989');
         $this->assertEmpty($user->getDirty());
     }
 
+    public function testGetDirty()
+    {
+        $user = new User([
+            'name' => 'John Doe',
+            'email' => 'john.doe@example.com',
+            'phone' => '123456789',
+        ]);
+
+        $user->save();
+
+        $this->assertFalse($user->isDirty());
+
+        $user->phone = '1234555555';
+        $this->assertTrue($user->isDirty());
+
+        $dirty = $user->getDirty();
+        $this->assertArrayHasKey('phone', $dirty);
+        $this->assertEquals('1234555555', $dirty['phone']);
+
+        $user->email = 'jane.doe@example.com';
+        $this->assertTrue($user->isDirty());
+        $dirty = $user->getDirty();
+        $this->assertArrayHasKey('phone', $dirty);
+        $this->assertArrayHasKey('email', $dirty);
+        $this->assertEquals('1234555555', $dirty['phone']);
+        $this->assertEquals('jane.doe@example.com', $dirty['email']);
+
+        $user->save();
+
+        $this->assertFalse($user->isDirty());
+        $this->assertEmpty($user->getDirty());
+    }
+
     public function testChunkById(): void
     {
         User::create(['name' => 'fork', 'tags' => ['sharp', 'pointy']]);