fix: Create deep copy before checking each sub schema in anyOf

#711

Author: DannyvdSluijs

src/JsonSchema/Constraints/UndefinedConstraint.php

@@ -333,11 +333,15 @@ protected function validateOfProperties(&$value, $schema, JsonPointer $path, $i
         if (isset($schema->anyOf)) {
             $isValid = false;
             $startErrors = $this->getErrors();
+            $coerceOrDefaults = $this->factory->getConfig(self::CHECK_MODE_COERCE_TYPES | self::CHECK_MODE_APPLY_DEFAULTS);
+
             foreach ($schema->anyOf as $anyOf) {
                 $initErrors = $this->getErrors();
                 try {
-                    $this->checkUndefined($value, $anyOf, $path, $i);
-                    if ($isValid = (count($this->getErrors()) == count($initErrors))) {
+                    $anyOfValue = $coerceOrDefaults ? DeepCopy::copyOf($value) : $value;
+                    $this->checkUndefined($anyOfValue, $anyOf, $path, $i);
+                    if ($isValid = (count($this->getErrors()) === count($initErrors))) {
+                        $value = $anyOfValue;
                         break;
                     }
                 } catch (ValidationException $e) {

tests/Constraints/UndefinedConstraintTest.php

@@ -69,6 +69,50 @@ public function getValidTests(): array
 JSON
                 ,
                 'checkMode' => Constraint::CHECK_MODE_COERCE_TYPES
+            ],
+            'anyOf with apply defaults should not affect value passed to each sub schema (#711)' => [
+                'input' => <<<JSON
+                    {
+                        "b": 2
+                    }
+JSON
+                ,
+                'schema' => <<<JSON
+                    {
+                        "anyOf": [
+                            {
+                                "required": [ "a" ],
+                      "properties": {
+                          "a": {
+                              "type": "integer"
+                          },
+                          "aDefault": {
+                              "type": "integer",
+                              "default": 1
+                          }
+                      },
+                      "type": "object",
+                      "additionalProperties": false
+                    },
+                    {
+                      "required": [ "b" ],
+                      "properties": {
+                          "b": {
+                              "type": "integer"
+                          },
+                          "bDefault": {
+                              "type": "integer",
+                              "default": 2
+                          }
+                      },
+                      "type": "object",
+                      "additionalProperties": false
+                    }
+                  ]
+                }
+JSON
+                ,
+                'checkMode' => Constraint::CHECK_MODE_APPLY_DEFAULTS
             ]
         ];
     }