[Serializer] Fix and improve constraintViolationListNormalizer's RFC7807 compliance

dunglas · dunglas · commit 4b657c242f72 · 2018-05-17T08:03:24.000+02:00
diff --git a/src/Symfony/Component/Serializer/Normalizer/ConstraintViolationListNormalizer.php b/src/Symfony/Component/Serializer/Normalizer/ConstraintViolationListNormalizer.php
@@ -34,19 +34,29 @@ public function normalize($object, $format = null, array $context = array())
         foreach ($object as $violation) {
             $violations[] = array(
                 'propertyPath' => $violation->getPropertyPath(),
-                'message' => $violation->getMessage(),
-                'code' => $violation->getCode(),
+                'type' => sprintf('urn:uuid:%s', $violation->getCode()),
+                'title' => $violation->getMessage(),
             );
             $propertyPath = $violation->getPropertyPath();
             $prefix = $propertyPath ? sprintf('%s: ', $propertyPath) : '';
             $messages[] = $prefix.$violation->getMessage();
         }
 
-        return array(
-            'title' => isset($context['title']) ? $context['title'] : 'An error occurred',
-            'detail' => $messages ? implode("\n", $messages) : '',
-            'violations' => $violations,
+        $result = array(
+            'type' => $context['type'] ?? 'https://symfony.com/doc/current/validation.html',
+            'title' => $context['title'] ?? 'Validation Failed',
         );
+        if (isset($context['status'])) {
+            $result['status'] = $context['status'];
+        }
+        if ($messages) {
+            $result['detail'] = implode("\n", $messages);
+        }
+        if (isset($context['instance'])) {
+            $result['instance'] = $context['instance'];
+        }
+
+        return $result + ['violations' => $violations];
     }
 
     /**
diff --git a/src/Symfony/Component/Serializer/Tests/Normalizer/ConstraintViolationListNormalizerTest.php b/src/Symfony/Component/Serializer/Tests/Normalizer/ConstraintViolationListNormalizerTest.php
@@ -43,19 +43,20 @@ public function testNormalize()
         ));
 
         $expected = array(
-            'title' => 'An error occurred',
+            'type' => 'https://symfony.com/doc/current/validation.html',
+            'title' => 'Validation Failed',
             'detail' => 'd: a
 4: 1',
             'violations' => array(
                     array(
                         'propertyPath' => 'd',
-                        'message' => 'a',
-                        'code' => 'f',
+                        'type' => 'urn:uuid:f',
+                        'title' => 'a',
                     ),
                     array(
                         'propertyPath' => '4',
-                        'message' => '1',
-                        'code' => '6',
+                        'type' => 'urn:uuid:6',
+                        'title' => '1',
                     ),
                 ),
         );
