remove implicit empty string to null conversion in sanitize() (#223)

techmahedy · web-flow · commit 6b5a3db2ea56 · 2026-03-11T19:21:30.000+06:00
diff --git a/src/Phaseolies/Database/Entity/Model.php b/src/Phaseolies/Database/Entity/Model.php
@@ -502,10 +502,6 @@ protected function sanitize($value)
     {
         if (is_string($value)) {
             $value = trim($value);
-
-            if ($value === '') {
-                $value = null;
-            }
         }
 
         return $value;
diff --git a/tests/Model/ModelTest.php b/tests/Model/ModelTest.php
@@ -93,7 +93,7 @@ public function testAttributeMutation()
 
         // Test empty string becomes null
         $model->email = '   ';
-        $this->assertNull($model->email);
+        $this->assertNotNull($model->email);
     }
 
     public function testMassAssignment()

Original file line number	Diff line number	Diff line change
`@@ -502,10 +502,6 @@ protected function sanitize($value)`
`502`	`502`	`{`
`503`	`503`	`if (is_string($value)) {`
`504`	`504`	`$value = trim($value);`
`505`		`-`
`506`		`- if ($value === '') {`
`507`		`- $value = null;`
`508`		`- }`
`509`	`505`	`}`
`510`	`506`
`511`	`507`	`return $value;`
Original file line number	Diff line number	Diff line change
`@@ -93,7 +93,7 @@ public function testAttributeMutation()`
`93`	`93`
`94`	`94`	`// Test empty string becomes null`
`95`	`95`	`$model->email = ' ';`
`96`		`- $this->assertNull($model->email);`
	`96`	`+ $this->assertNotNull($model->email);`
`97`	`97`	`}`
`98`	`98`
`99`	`99`	`public function testMassAssignment()`