[Form] Disallowed infinity in NumberToLocalizedStringTransformer

symfony · Feb 2, 2012 · 9a4e22e · 9a4e22e
1 parent baa63b8
commit 9a4e22e
Show file tree

Hide file tree

Showing 3 changed files with 54 additions and 0 deletions.
diff --git a/...fony/Component/Form/Extension/Core/DataTransformer/NumberToLocalizedStringTransformer.php b/...fony/Component/Form/Extension/Core/DataTransformer/NumberToLocalizedStringTransformer.php
@@ -114,6 +114,10 @@ public function reverseTransform($value)
             throw new TransformationFailedException($formatter->getErrorMessage());
         }
 
+        if ($value >= INF || $value <= -INF) {
+            throw new TransformationFailedException('I don\'t have a clear idea what infinity looks like');
+        }
+
         return $value;
     }
 

diff --git a/...Component/Form/Extension/Core/DataTransformer/IntegerToLocalizedStringTransformerTest.php b/...Component/Form/Extension/Core/DataTransformer/IntegerToLocalizedStringTransformerTest.php
@@ -90,4 +90,24 @@ public function testReverseTransformDisallowsNaN2()
 
         $transformer->reverseTransform('nan');
     }
+
+    /**
+     * @expectedException Symfony\Component\Form\Exception\TransformationFailedException
+     */
+    public function testReverseTransformDisallowsInfinity()
+    {
+        $transformer = new IntegerToLocalizedStringTransformer();
+
+        $transformer->reverseTransform('∞');
+    }
+
+    /**
+     * @expectedException Symfony\Component\Form\Exception\TransformationFailedException
+     */
+    public function testReverseTransformDisallowsNegativeInfinity()
+    {
+        $transformer = new IntegerToLocalizedStringTransformer();
+
+        $transformer->reverseTransform('-∞');
+    }
 }
diff --git a/.../Component/Form/Extension/Core/DataTransformer/NumberToLocalizedStringTransformerTest.php b/.../Component/Form/Extension/Core/DataTransformer/NumberToLocalizedStringTransformerTest.php
@@ -144,4 +144,34 @@ public function testReverseTransformDisallowsNaN2()
 
         $transformer->reverseTransform('nan');
     }
+
+    /**
+     * @expectedException Symfony\Component\Form\Exception\TransformationFailedException
+     */
+    public function testReverseTransformDisallowsInfinity()
+    {
+        $transformer = new NumberToLocalizedStringTransformer();
+
+        $transformer->reverseTransform('∞');
+    }
+
+    /**
+     * @expectedException Symfony\Component\Form\Exception\TransformationFailedException
+     */
+    public function testReverseTransformDisallowsInfinity2()
+    {
+        $transformer = new NumberToLocalizedStringTransformer();
+
+        $transformer->reverseTransform('∞,123');
+    }
+
+    /**
+     * @expectedException Symfony\Component\Form\Exception\TransformationFailedException
+     */
+    public function testReverseTransformDisallowsNegativeInfinity()
+    {
+        $transformer = new NumberToLocalizedStringTransformer();
+
+        $transformer->reverseTransform('-∞');
+    }
 }