Merge pull request #298 from henriquemoody/related

AbstractRelated is accepts '' as valid by default
Respect · Feb 13, 2015 · 11be936 · 11be936
2 parents d1af494 + 1b1689b
commit 11be936
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 1 deletion.
diff --git a/library/Rules/AbstractRelated.php b/library/Rules/AbstractRelated.php
@@ -66,10 +66,11 @@ public function check($input)
     public function validate($input)
     {
         $hasReference = $this->hasReference($input);
+
         if ($this->mandatory && !$hasReference) {
             return false;
         }
 
-        return $this->decision('validate', $hasReference, $input);
+        return $this->decision('validate', $hasReference, $input) || $this->getReferenceValue($input) === '';
     }
 }
diff --git a/tests/library/Respect/Validation/Rules/KeyTest.php b/tests/library/Respect/Validation/Rules/KeyTest.php
@@ -13,6 +13,16 @@ public function testArrayWithPresentKeyShouldReturnTrue()
         $this->assertTrue($validator->validate($obj));
     }
 
+    public function testArrayWithEmptyKeyShouldReturnTrue()
+    {
+        $validator = new Key('someEmptyKey');
+        $obj = array();
+        $obj['someEmptyKey'] = '';
+        $this->assertTrue($validator->assert($obj));
+        $this->assertTrue($validator->check($obj));
+        $this->assertTrue($validator->validate($obj));
+    }
+
     /**
      * @expectedException Respect\Validation\Exceptions\KeyException
      */