Fix #4

Author: hkarlstrom

src/OpenApiValidation.php

@@ -366,21 +366,26 @@ private function validateObject(array $schema, array $value) : array
         }
         $validator = new Validator();
         $validator->setFormats($this->formatContainer);
-        try {
-            $value  = json_decode(json_encode($value));
-            $schema = json_decode(json_encode($schema));
-            $result = $validator->dataValidation($value, $schema, 99);
-        } catch (Exception $e) {
-            return [[
-                'name'    => 'server',
-                'code'    => 'error_server',
-                'message' => $e->getMessage(),
-            ]];
-            return [$e->getMessage()];
-        }
-        if (!$result->isValid()) {
-            foreach ($result->getErrors() as $error) {
-                $errors = array_merge($errors, $this->parseErrors($error));
+
+        // TODO support for anyOf, oneOf, not, discriminator property
+        $schemas = isset($schema['allOf']) ? $schema['allOf'] : [$schema];
+        foreach ($schemas as $schema) {
+            try {
+                $value  = json_decode(json_encode($value));
+                $schema = json_decode(json_encode($schema));
+                $result = $validator->dataValidation($value, $schema, 99);
+            } catch (Exception $e) {
+                return [[
+                    'name'    => 'server',
+                    'code'    => 'error_server',
+                    'message' => $e->getMessage(),
+                ]];
+                return [$e->getMessage()];
+            }
+            if (!$result->isValid()) {
+                foreach ($result->getErrors() as $error) {
+                    $errors = array_merge($errors, $this->parseErrors($error));
+                }
             }
         }
         return $errors;

tests/RequestBodyTest.php

@@ -77,4 +77,25 @@ public function testEmptyRequestBody()
         $this->assertTrue($json['ok']);
         $this->assertSame(200, $response->getStatusCode());
     }
+
+    public function testAllOfComposition()
+    {
+        $response = $this->response('put', '/all/of', [
+            'body' => [
+                'id'          => 'a',
+                'first_name'  => 'Jane',
+                'last_name'   => 'Doe',
+                'phone'       => '3333-11111111',
+                'nationality' => 'IE',
+            ],
+        ]);
+        $json = $this->json($response);
+        $this->assertSame('e_mail', $json['errors'][0]['name']);
+        $this->assertSame('error_required', $json['errors'][0]['code']);
+        $this->assertSame(400, $response->getStatusCode());
+        $this->assertSame('id', $json['errors'][1]['name']);
+        $this->assertSame('string', $json['errors'][1]['used']);
+        $this->assertSame('integer', $json['errors'][1]['expected']);
+        $this->assertSame('error_type', $json['errors'][1]['code']);
+    }
 }