Merge pull request jsonrainbow#22 from paranoiq/master

justinrainbow · justinrainbow · commit bf1998956e12 · 2012-11-29T08:11:18.000-08:00
wrong handling of "required": false (closes jsonrainbow#21)
diff --git a/src/JsonSchema/Constraints/Object.php b/src/JsonSchema/Constraints/Object.php
@@ -22,6 +22,10 @@ class Object extends Constraint
      */
     function check($element, $definition = null, $path = null, $additionalProp = null, $patternProperties = null)
     {
+        if ($element instanceof Undefined) {
+            return;
+        }
+
         if ($patternProperties) {
             $this->validatePatternProperties($element, $path, $additionalProp, $patternProperties);
         }

Original file line number	Diff line number	Diff line change
`@@ -22,6 +22,10 @@ class Object extends Constraint`
`22`	`22`	`*/`
`23`	`23`	`function check($element, $definition = null, $path = null, $additionalProp = null, $patternProperties = null)`
`24`	`24`	`{`
	`25`	`+ if ($element instanceof Undefined) {`
	`26`	`+ return;`
	`27`	`+ }`
	`28`	`+`
`25`	`29`	`if ($patternProperties) {`
`26`	`30`	`$this->validatePatternProperties($element, $path, $additionalProp, $patternProperties);`
`27`	`31`	`}`