Merge pull request #94 from Seldaek/additionalProp

Add support for additionalProperties set to true

Author: justinrainbow

src/JsonSchema/Constraints/Object.php

@@ -82,7 +82,11 @@ public function validateElement($element, $matches, $objectDefinition = null, $p
 
             // additional properties defined
             if (!in_array($i, $matches) && $additionalProp && !$definition) {
-                $this->checkUndefined($value, $additionalProp, $path, $i);
+                if ($additionalProp === true) {
+                    $this->checkUndefined($value, null, $path, $i);
+                } else {
+                    $this->checkUndefined($value, $additionalProp, $path, $i);
+                }
             }
 
             // property requires presence of another

tests/JsonSchema/Tests/Constraints/AdditionalPropertiesTest.php

@@ -124,6 +124,19 @@ public function getValidTests()
                   },
                   "additionalProperties": {"type":"string"}
                 }'
+            ),
+            array(
+                '{
+                  "prop":"1",
+                  "additionalProp":[]
+                }',
+                '{
+                  "type":"object",
+                  "properties":{
+                    "prop":{"type":"string"}
+                  },
+                  "additionalProperties": true
+                }'
             )
         );
     }