From 84decc9a666dccef5e7d8241622898a2023368bc Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?S=C5=82awomir=20Chrobak?= <slawomir.chrobak@gmail.com>
Date: Fri, 29 Apr 2016 17:04:39 +0200
Subject: [PATCH] Skips min/max properties validation for non objects

---
 .../Constraints/ObjectConstraint.php          |   9 +-
 .../Constraints/MinMaxPropertiesTest.php      | 108 ++++++++++++++++++
 2 files changed, 113 insertions(+), 4 deletions(-)
 create mode 100644 tests/JsonSchema/Tests/Constraints/MinMaxPropertiesTest.php

diff --git a/src/JsonSchema/Constraints/ObjectConstraint.php b/src/JsonSchema/Constraints/ObjectConstraint.php
index 0cfb59a3..f40e1202 100644
--- a/src/JsonSchema/Constraints/ObjectConstraint.php
+++ b/src/JsonSchema/Constraints/ObjectConstraint.php
@@ -81,12 +81,8 @@ public function validateElement($element, $matches, $objectDefinition = null, $p
     {
         $this->validateMinMaxConstraint($element, $objectDefinition, $path);
         foreach ($element as $i => $value) {
-
-            $property = $this->getProperty($element, $i, new UndefinedConstraint());
             $definition = $this->getProperty($objectDefinition, $i);
 
-            $this->validateMinMaxConstraint(!($property instanceof UndefinedConstraint) ? $property : $element, $definition, $path);
-
             // no additional properties allowed
             if (!in_array($i, $matches) && $additionalProp === false && $this->inlineSchemaProperty !== $i && !$definition) {
                 $this->addError($path, "The property " . $i . " is not defined and the definition does not allow additional properties", 'additionalProp');
@@ -111,6 +107,11 @@ public function validateElement($element, $matches, $objectDefinition = null, $p
                 // normal property verification
                 $this->checkUndefined($value, new \stdClass(), $path, $i);
             }
+
+            $property = $this->getProperty($element, $i, new UndefinedConstraint());
+            if (is_object($property)) {
+                $this->validateMinMaxConstraint(!($property instanceof UndefinedConstraint) ? $property : $element, $definition, $path);
+            }
         }
     }
 
diff --git a/tests/JsonSchema/Tests/Constraints/MinMaxPropertiesTest.php b/tests/JsonSchema/Tests/Constraints/MinMaxPropertiesTest.php
new file mode 100644
index 00000000..8c3a641d
--- /dev/null
+++ b/tests/JsonSchema/Tests/Constraints/MinMaxPropertiesTest.php
@@ -0,0 +1,108 @@
+<?php
+
+/*
+ * This file is part of the JsonSchema package.
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace JsonSchema\Tests\Constraints;
+
+class MinMaxPropertiesTest extends BaseTestCase
+{
+    /**
+     * {@inheritdoc}
+     */
+    public function getValidTests()
+    {
+        return array(
+            array(
+                '{
+                  "value": {}
+                }',
+                '{
+                  "type": "object",
+                  "properties": {
+                    "value": {"type": "object", "minProperties": 0}
+                  }
+                }'
+            ),
+            array(
+                '{
+                  "value": {}
+                }',
+                '{
+                  "type": "object",
+                  "properties": {
+                    "value": {"type": "object", "maxProperties": 1}
+                  }
+                }'
+            ),
+            array(
+                '{
+                  "value": {}
+                }',
+                '{
+                  "type": "object",
+                  "properties": {
+                    "value": {"type": "object", "minProperties": 0,"maxProperties": 1}
+                  }
+                }'
+            ),
+            array(
+                '{
+                  "value": {"foo": 1, "bar": 2}
+                }',
+                '{
+                  "type": "object",
+                  "properties": {
+                    "value": {"type": "object", "minProperties": 1,"maxProperties": 2}
+                  }
+                }'
+            ),
+        );
+    }
+
+    /**
+     * {@inheritdoc}
+     */
+    public function getInvalidTests()
+    {
+        return array(
+            array(
+                '{
+                  "value": 1
+                }',
+                '{
+                  "type": "object",
+                  "properties": {
+                    "value": {"type": "object", "minProperties": 1}
+                  }
+                }'
+            ),
+            array(
+                '{
+                  "value": 1
+                }',
+                '{
+                  "type": "object",
+                  "properties": {
+                    "value": {"type": "object", "maxProperties": 1}
+                  }
+                }'
+            ),
+            array(
+                '{
+                  "value": {"foo": 1, "bar": 2, "baz": 3}
+                }',
+                '{
+                  "type": "object",
+                  "properties": {
+                    "value": {"type": "object", "minProperties": 1,"maxProperties": 2}
+                  }
+                }'
+            ),
+        );
+    }
+}