self review

src/ApiResource/Error.php

@@ -30,7 +30,7 @@
     operations: [
         new Get(name: '_api_errors_hydra', outputFormats: ['jsonld' => ['application/ld+json']], normalizationContext: ['groups' => ['jsonld'], 'skip_null_values' => true]),
         new Get(name: '_api_errors_problem', outputFormats: ['json' => ['application/problem+json']], normalizationContext: ['groups' => ['jsonproblem'], 'skip_null_values' => true]),
-        new Get(name: '_api_errors_jsonapi', outputFormats: ['jsonapi' => ['application/vnd.api+json']], normalizationContext: ['groups' => ['jsonapi'], 'skip_null_values' => true]),
+        new Get(name: '_api_errors_jsonapi', outputFormats: ['jsonapi' => ['application/vnd.api+json']], normalizationContext: ['groups' => ['jsonapi'], 'skip_null_values' => true], provider: 'api_platform.state_provider.json_api.default_error'),
     ]
 )]
 class Error extends \Exception implements ProblemExceptionInterface, HttpExceptionInterface

src/Metadata/Resource/Factory/ClassNameResourceNameCollectionFactory.php

@@ -15,6 +15,9 @@
 
 use ApiPlatform\Metadata\Resource\ResourceNameCollection;
 
+/**
+ * @internal
+ */
 final class ClassNameResourceNameCollectionFactory implements ResourceNameCollectionFactoryInterface
 {
     /**

src/State/DefaultErrorProvider.php

@@ -14,18 +14,14 @@
 namespace ApiPlatform\State;
 
 use ApiPlatform\Metadata\Operation;
-use ApiPlatform\Symfony\Validator\Exception\ConstraintViolationListAwareExceptionInterface;
 
-class DefaultErrorProvider implements ProviderInterface
+/**
+ * @internal
+ */
+final class DefaultErrorProvider implements ProviderInterface
 {
     public function provide(Operation $operation, array $uriVariables = [], array $context = []): object
     {
-        $exception = $context['previous_data'];
-
-        if ($exception instanceof ConstraintViolationListAwareExceptionInterface && '_api_validation_errors_jsonapi' === $context['operation_name']) {
-            return $exception->getConstraintViolationList();
-        }
-
-        return $exception;
+        return $context['previous_data'];
     }
 }

src/State/JsonApi/DefaultErrorProvider.php

@@ -0,0 +1,34 @@
+<?php
+
+/*
+ * This file is part of the API Platform project.
+ *
+ * (c) Kévin Dunglas <dunglas@gmail.com>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+declare(strict_types=1);
+
+namespace ApiPlatform\State\JsonApi;
+
+use ApiPlatform\Metadata\Operation;
+use ApiPlatform\Symfony\Validator\Exception\ConstraintViolationListAwareExceptionInterface;
+
+/**
+ * @internal
+ */
+final class DefaultErrorProvider implements ProviderInterface
+{
+    public function provide(Operation $operation, array $uriVariables = [], array $context = []): object
+    {
+        $exception = $context['previous_data'];
+
+        if ($exception instanceof ConstraintViolationListAwareExceptionInterface) {
+            return $exception->getConstraintViolationList();
+        }
+
+        return $exception;
+    }
+}

src/Symfony/Bundle/Resources/config/state.xml

@@ -58,6 +58,11 @@
             <tag name="api_platform.state_provider" key="api_platform.state_provider.default_error" />
         </service>
 
+        <service id="api_platform.state_provider.json_api.default_error" class="ApiPlatform\State\JsonApi\DefaultErrorProvider" >
+            <tag name="api_platform.state_provider" key="ApiPlatform\State\JsonApi\DefaultErrorProvider" />
+            <tag name="api_platform.state_provider" key="api_platform.state_provider.jsonapi.default_error" />
+        </service>
+
         <service id="ApiPlatform\State\ObjectProvider" alias="api_platform.state_provider.object" />
     </services>
 </container>