fix comparisons with null values at property paths

Author: xabbuh

Constraints/GreaterThanOrEqualValidator.php

@@ -24,7 +24,7 @@ class GreaterThanOrEqualValidator extends AbstractComparisonValidator
      */
     protected function compareValues($value1, $value2)
     {
-        return $value1 >= $value2;
+        return null === $value2 || $value1 >= $value2;
     }
 
     /**

Constraints/GreaterThanValidator.php

@@ -24,7 +24,7 @@ class GreaterThanValidator extends AbstractComparisonValidator
      */
     protected function compareValues($value1, $value2)
     {
-        return $value1 > $value2;
+        return null === $value2 || $value1 > $value2;
     }
 
     /**

Constraints/LessThanOrEqualValidator.php

@@ -24,7 +24,7 @@ class LessThanOrEqualValidator extends AbstractComparisonValidator
      */
     protected function compareValues($value1, $value2)
     {
-        return $value1 <= $value2;
+        return null === $value2 || $value1 <= $value2;
     }
 
     /**

Constraints/LessThanValidator.php

@@ -24,7 +24,7 @@ class LessThanValidator extends AbstractComparisonValidator
      */
     protected function compareValues($value1, $value2)
     {
-        return $value1 < $value2;
+        return null === $value2 || $value1 < $value2;
     }
 
     /**

Tests/Constraints/AbstractComparisonValidatorTestCase.php

@@ -237,6 +237,31 @@ public function throwsOnInvalidStringDatesProvider()
         ];
     }
 
+    /**
+     * @dataProvider provideComparisonsToNullValueAtPropertyPath
+     */
+    public function testCompareWithNullValueAtPropertyAt($dirtyValue, $dirtyValueAsString, $isValid)
+    {
+        $constraint = $this->createConstraint(['propertyPath' => 'value']);
+        $constraint->message = 'Constraint Message';
+
+        $object = new ComparisonTest_Class(null);
+        $this->setObject($object);
+
+        $this->validator->validate($dirtyValue, $constraint);
+
+        if ($isValid) {
+            $this->assertNoViolation();
+        } else {
+            $this->buildViolation('Constraint Message')
+                ->setParameter('{{ value }}', $dirtyValueAsString)
+                ->setParameter('{{ compared_value }}', 'null')
+                ->setParameter('{{ compared_value_type }}', 'NULL')
+                ->setCode($this->getErrorCode())
+                ->assertRaised();
+        }
+    }
+
     /**
      * @return array
      */
@@ -258,6 +283,8 @@ public function provideAllInvalidComparisons()
      */
     abstract public function provideInvalidComparisons();
 
+    abstract public function provideComparisonsToNullValueAtPropertyPath();
+
     /**
      * @param array|null $options Options for the constraint
      *

Tests/Constraints/EqualToValidatorTest.php

@@ -75,4 +75,11 @@ public function provideInvalidComparisons()
             [new ComparisonTest_Class(4), '4', new ComparisonTest_Class(5), '5', __NAMESPACE__.'\ComparisonTest_Class'],
         ];
     }
+
+    public function provideComparisonsToNullValueAtPropertyPath()
+    {
+        return [
+            [5, '5', false],
+        ];
+    }
 }

Tests/Constraints/GreaterThanOrEqualValidatorTest.php

@@ -78,4 +78,11 @@ public function provideInvalidComparisons()
             ['b', '"b"', 'c', '"c"', 'string'],
         ];
     }
+
+    public function provideComparisonsToNullValueAtPropertyPath()
+    {
+        return [
+            [5, '5', true],
+        ];
+    }
 }

Tests/Constraints/GreaterThanValidatorTest.php

@@ -80,4 +80,11 @@ public function provideInvalidComparisons()
             ['22', '"22"', '22', '"22"', 'string'],
         ];
     }
+
+    public function provideComparisonsToNullValueAtPropertyPath()
+    {
+        return [
+            [5, '5', true],
+        ];
+    }
 }

Tests/Constraints/IdenticalToValidatorTest.php

@@ -93,4 +93,11 @@ public function provideInvalidComparisons()
             [new ComparisonTest_Class(4), '4', new ComparisonTest_Class(5), '5', __NAMESPACE__.'\ComparisonTest_Class'],
         ];
     }
+
+    public function provideComparisonsToNullValueAtPropertyPath()
+    {
+        return [
+            [5, '5', false],
+        ];
+    }
 }

Tests/Constraints/LessThanOrEqualValidatorTest.php

@@ -81,4 +81,11 @@ public function provideInvalidComparisons()
             ['c', '"c"', 'b', '"b"', 'string'],
         ];
     }
+
+    public function provideComparisonsToNullValueAtPropertyPath()
+    {
+        return [
+            [5, '5', true],
+        ];
+    }
 }