[Form] removed a constraint in PropertyPath as the path can definitel…

…y be an empty string for errors attached on the main form (when using a constraint defined with the 'validation_constraint' option)
symfony · Jul 15, 2011 · 5182a0c · 5182a0c · webmozart · May 18, 2012
1 parent c045120
commit 5182a0c
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 8 deletions.
diff --git a/src/Symfony/Component/Form/Util/PropertyPath.php b/src/Symfony/Component/Form/Util/PropertyPath.php
@@ -55,7 +55,7 @@ class PropertyPath implements \IteratorAggregate
      */
     public function __construct($propertyPath)
     {
-        if ('' === $propertyPath || null === $propertyPath) {
+        if (null === $propertyPath) {
             throw new InvalidPropertyPathException('The property path must not be empty');
         }
 

diff --git a/tests/Symfony/Tests/Component/Form/PropertyPathTest.php b/tests/Symfony/Tests/Component/Form/PropertyPathTest.php
@@ -385,13 +385,6 @@ public function testInvalidPropertyPath_unexpectedCharacters()
         new PropertyPath('property.$form');
     }
 
-    public function testInvalidPropertyPath_empty()
-    {
-        $this->setExpectedException('Symfony\Component\Form\Exception\InvalidPropertyPathException');
-
-        new PropertyPath('');
-    }
-
     public function testInvalidPropertyPath_null()
     {
         $this->setExpectedException('Symfony\Component\Form\Exception\InvalidPropertyPathException');