Don't cast empty values (null or '') to value objects. Instead, return null.

Author: szainmehdi

src/CastsValueObjects.php

@@ -23,6 +23,11 @@ public function getAttribute($key)
             // Allow other mutators and such to do their work first.
             $value = parent::getAttribute($key);
 
+            // Don't cast empty $value.
+            if ($value === null || $value === '') {
+                return null;
+            }
+
             // Cache the instantiated value for future access.
             // This allows tests such as ($model->casted === $model->casted) to be true.
             if (!$this->isValueObjectCached($key)) {

tests/CastsValueObjectTest.php

@@ -123,4 +123,20 @@ public function testModelToJsonWithValueObjects()
         $this->assertJson($json);
         $this->assertJsonStringEqualsJsonString(json_encode(['email' => $email]), $json);
     }
+
+    public function testNullValuesDontGetCast()
+    {
+        $user = new UserModel();
+
+        $this->assertNull($user->email);
+    }
+
+    public function testEmptyStringValuesDontGetCast()
+    {
+        $user = new UserModel();
+
+        $user->setInternalAttributes(['email' => '']);
+
+        $this->assertNull($user->email);
+    }
 }