Fix Model::unset with dot field name

Author: GromNaN

src/Eloquent/Model.php

@@ -315,11 +315,16 @@ public function originalIsEquivalent($key)
      */
     public function offsetUnset($offset): void
     {
-        parent::offsetUnset($offset);
-
-        // Force unsetting even if the attribute is not set.
-        // End user can optimize DB calls by checking if the attribute is set before unsetting it.
-        $this->unset[$offset] = true;
+        if (str_contains($offset, '.')) {
+            // Update the field in the subdocument
+            Arr::forget($this->attributes, $offset);
+        } else {
+            parent::offsetUnset($offset);
+
+            // Force unsetting even if the attribute is not set.
+            // End user can optimize DB calls by checking if the attribute is set before unsetting it.
+            $this->unset[$offset] = true;
+        }
     }
 
     /**

tests/ModelTest.php

@@ -504,6 +504,43 @@ public function testUnset(): void
         $this->assertFalse(isset($user2->note2));
     }
 
+    public function testUnsetDotAttributes(): void
+    {
+        $user = User::create(['name' => 'John Doe', 'notes' => ['note1' => 'ABC', 'note2' => 'DEF']]);
+
+        $user->unset('notes.note1');
+
+        $this->assertFalse(isset($user->notes['note1']));
+        $this->assertTrue(isset($user->notes['note2']));
+
+        $user->save();
+
+        $this->assertFalse(isset($user->notes['note1']));
+        $this->assertTrue(isset($user->notes['note2']));
+
+        // Re-fetch to be sure
+        $user = User::find($user->_id);
+
+        $this->assertFalse(isset($user->notes['note1']));
+        $this->assertTrue(isset($user->notes['note2']));
+
+        // Unset the parent key
+        $user->unset('notes');
+
+        $this->assertFalse(isset($user->notes['note1']));
+        $this->assertFalse(isset($user->notes['note2']));
+        $this->assertFalse(isset($user->notes));
+
+        $user->save();
+
+        $this->assertFalse(isset($user->notes));
+
+        // Re-fetch to be sure
+        $user = User::find($user->_id);
+
+        $this->assertFalse(isset($user->notes));
+    }
+
     public function testUnsetAndSet(): void
     {
         $user = User::create(['name' => 'John Doe', 'note1' => 'ABC', 'note2' => 'DEF']);