Improve update client errors handling

webgriffe · Sep 2, 2024 · 0ade2e0 · 0ade2e0
1 parent c558924
commit 0ade2e0
Showing 1 changed file with 14 additions and 6 deletions.
diff --git a/src/Client/ActiveCampaignResourceClient.php b/src/Client/ActiveCampaignResourceClient.php
@@ -191,14 +191,22 @@ public function update(int $activeCampaignResourceId, ResourceInterface $resourc
             $serializedResource,
         ));
         if (($statusCode = $response->getStatusCode()) !== 200) {
-            if ($statusCode === 404) {
-                /** @var array{message: string} $errorResponse */
-                $errorResponse = json_decode($response->getBody()->getContents(), true, 512, \JSON_THROW_ON_ERROR);
+            switch ($statusCode) {
+                case 404:
+                    /** @var array{message: string} $errorResponse */
+                    $errorResponse = json_decode($response->getBody()->getContents(), true, 512, \JSON_THROW_ON_ERROR);
 
-                throw new NotFoundHttpException($errorResponse['message']);
-            }
+                    throw new NotFoundHttpException($errorResponse['message']);
+                case 422:
+                    /** @var array{errors: array{title: string, detail: string, code: string, source: array{pointer: string}}} $errorResponse */
+                    $errorResponse = json_decode($response->getBody()->getContents(), true, 512, \JSON_THROW_ON_ERROR);
+                    /** @var string[] $titles */
+                    $titles = array_column($errorResponse['errors'], 'title');
 
-            throw new HttpException($statusCode, $response->getReasonPhrase(), null, $response->getHeaders());
+                    throw new UnprocessableEntityHttpException(implode('; ', $titles));
+                default:
+                    throw new HttpException($statusCode, $response->getReasonPhrase(), null, $response->getHeaders());
+            }
         }
 
         /** @var UpdateResourceResponseInterface $updateResourceResponse */