Use !isset($context[$Property->getName()]) for evaluating a nullabl…

…e property. (#23) Co-authored-by: david_smith <[email protected]>
zero-to-prod · Nov 20, 2024 · 2451ff1 · 2451ff1
1 parent 076e226
commit 2451ff1
Showing 1 changed file with 7 additions and 7 deletions.
diff --git a/src/DataModelHelper.php b/src/DataModelHelper.php
@@ -62,7 +62,7 @@ public static function mapOf(mixed $value, array $context, ?ReflectionAttribute
             throw new PropertyRequiredException("Property `\${$Property->getName()}` is required.");
         }
 
-        if (!$value && $Property->getType()?->allowsNull()) {
+        if (!isset($context[$Property->getName()]) && $Property->getType()?->allowsNull()) {
             return null;
         }
 
@@ -122,7 +122,7 @@ public static function pregReplace(mixed $value, array $context, ?ReflectionAttr
             throw new PropertyRequiredException("Property `\${$Property->getName()}` is required.");
         }
 
-        if (!$value) {
+        if (!isset($context[$Property->getName()])) {
             return $Property->getType()?->allowsNull()
                 ? null
                 : '';
@@ -156,7 +156,7 @@ public static function pregMatch(mixed $value, array $context, ?ReflectionAttrib
             throw new PropertyRequiredException("Property `\${$Property->getName()}` is required.");
         }
 
-        if (!$value && $Property->getType()?->allowsNull()) {
+        if (!isset($context[$Property->getName()]) && $Property->getType()?->allowsNull()) {
             return null;
         }
 
@@ -196,7 +196,7 @@ public static function isUrl(mixed $value, array $context, ?ReflectionAttribute
             throw new PropertyRequiredException("Property `\${$Property->getName()}` is required.");
         }
 
-        if (!$value && $Property->getType()?->allowsNull()) {
+        if (!isset($context[$Property->getName()]) && $Property->getType()?->allowsNull()) {
             return null;
         }
 
@@ -242,7 +242,7 @@ public static function isEmail(mixed $value, array $context, ?ReflectionAttribut
             throw new PropertyRequiredException("Property `\${$Property->getName()}` is required.");
         }
 
-        if (!$value && $Property->getType()?->allowsNull()) {
+        if (!isset($context[$Property->getName()]) && $Property->getType()?->allowsNull()) {
             return null;
         }
 
@@ -289,7 +289,7 @@ public static function isMultiple(mixed $value, array $context, ?ReflectionAttri
             throw new PropertyRequiredException("Property `\${$Property->getName()}` is required.");
         }
 
-        if (!$value && $Property->getType()?->allowsNull()) {
+        if (!isset($context[$Property->getName()]) && $Property->getType()?->allowsNull()) {
             return null;
         }
 
@@ -336,7 +336,7 @@ public static function when(mixed $value, array $context, ?ReflectionAttribute $
             throw new PropertyRequiredException("Property `\${$Property->getName()}` is required.");
         }
 
-        if (!$value && $Property->getType()?->allowsNull()) {
+        if (!isset($context[$Property->getName()]) && $Property->getType()?->allowsNull()) {
             return null;
         }