Updated handleResponse function to give more detailed error message (#41

)

* Updated handleResponse function to give more detailed error message

* Updated CHANGELOG

* Created new exception and updated inheritance structure

* updated from pr feedback

* update to readme and changelog

CHANGELOG.md

@@ -1,5 +1,9 @@
 ## CHANGELOG
 
+### 2.2.5
+- Update - Add error details into KlaviyoAPI handleResponse function
+- Update - Add KlaviyoApiException 
+
 ### 2.2.4
 - Fix - Instantiate KlaviyoRateLimitException properly
 - Update - Add retryAfter as an array key for the KlaviyoRateLimitException message

README.md

@@ -151,7 +151,18 @@ $client->profiles->getAllProfileMetricsTimeline( 'ProfileId' );
 $client->profiles->getProfileMetricTimeline( 'ProfileId', 'MetricId' );
 ```
 
-## Rate Limiting
-  If a rate limit happens it will throw a Klaviyo/Exception/KlaviyoRateLimitException.
+## Exceptions
+
+### Klaviyo\Exception\KlaviyoApiException
+  Thrown when there is an issue making an API request. After you catch this exception, you can use getMessage() and it will return a string containing more details about the issue that occurred.
+
+### Klaviyo\Exception\KlaviyoRateLimitException
+  If a rate limit happens it will throw a Klaviyo\Exception\KlaviyoRateLimitException.
   After you catch this exception you can use getMessage() and it will return a JSON encoded array:
   `{"detail":"Request was throttled. Expected available in 26.0 seconds.","retryAfter":26}`
+
+### Klaviyo\Exception\KlaviyoAuthenticationException
+  Thrown when there is an authentication error when making an API request, usually caused by an invalid API key.  
+
+### Klaviyo\Exception\KlaviyoResourceNotFoundException
+  Thrown when the system attempts to update a property that doesn't exist. For example, attempting to update a list that doesn't exist on the account.

src/Exception/KlaviyoApiException.php

@@ -0,0 +1,5 @@
+<?php
+
+namespace Klaviyo\Exception;
+
+class KlaviyoApiException extends KlaviyoException {}

src/Exception/KlaviyoAuthenticationException.php

@@ -2,4 +2,4 @@
 
 namespace Klaviyo\Exception;
 
-class KlaviyoAuthenticationException extends KlaviyoException {}
+class KlaviyoAuthenticationException extends KlaviyoApiException {}

src/Exception/KlaviyoRateLimitException.php

@@ -2,4 +2,4 @@
 
 namespace Klaviyo\Exception;
 
-class KlaviyoRateLimitException extends KlaviyoException {}
+class KlaviyoRateLimitException extends KlaviyoApiException {}

src/Exception/KlaviyoResourceNotFoundException.php

@@ -2,4 +2,4 @@
 
 namespace Klaviyo\Exception;
 
-class KlaviyoResourceNotFoundException extends KlaviyoException {}
+class KlaviyoResourceNotFoundException extends KlaviyoApiException {}

src/KlaviyoAPI.php

@@ -6,6 +6,7 @@
 use Klaviyo\Exception\KlaviyoException;
 use Klaviyo\Exception\KlaviyoRateLimitException;
 use Klaviyo\Exception\KlaviyoResourceNotFoundException;
+use Klaviyo\Exception\KlaviyoApiException;
 use Klaviyo\Model\ProfileModel;
 
 abstract class KlaviyoAPI
@@ -31,7 +32,6 @@ abstract class KlaviyoAPI
      */
     const ERROR_INVALID_API_KEY = 'Invalid API Key.';
     const ERROR_RESOURCE_DOES_NOT_EXIST = 'The requested resource does not exist.';
-    const ERROR_NON_200_STATUS = 'Request Failed with HTTP Status Code: %s';
 
     /**
      * Request options
@@ -175,15 +175,15 @@ private function request( $method, $path, $options, $isPublic = false, $isV1 = f
     private function handleResponse( $response, $statusCode, $isPublic )
     {
         if ( $statusCode == 403 ) {
-            throw new KlaviyoAuthenticationException(self::ERROR_INVALID_API_KEY);
+            throw new KlaviyoAuthenticationException(self::ERROR_INVALID_API_KEY, $statusCode);
         } else if ( $statusCode == 404 ) {
-            throw new KlaviyoResourceNotFoundException(self::ERROR_RESOURCE_DOES_NOT_EXIST);
+            throw new KlaviyoResourceNotFoundException( self::ERROR_RESOURCE_DOES_NOT_EXIST, $statusCode);
         } else if ( $statusCode == 429 ) {
             throw new KlaviyoRateLimitException(
-                $this->returnRateLimit( $this->decodeJsonResponse( $response ) )
+                $this->returnRateLimit( $this->decodeJsonResponse( $response ), $statusCode )
             );
         } else if ( $statusCode != 200 ) {
-            throw new KlaviyoException( sprintf( self::ERROR_NON_200_STATUS, $statusCode ) );
+            throw new KlaviyoApiException($this->decodeJsonResponse( $response )['detail'], $statusCode);
         }
 
         if ( $isPublic ) {