Merge pull request #121 from lohanidamodar/feat-php-sdk-exceptions

feat-php-sdk-exceptions

Author: eldadfux

example.php

@@ -45,7 +45,11 @@ function getSSLPage($url) {
     }
 
     // PHP
-    $sdk  = new SDK(new PHP(), new Swagger2($spec));
+    $php = new PHP();
+    $php
+        ->setComposerVendor('appwrite')
+        ->setComposerPackage('appwrite');
+    $sdk  = new SDK($php, new Swagger2($spec));
 
     $sdk
         ->setName('NAME')

src/SDK/Language/PHP.php

@@ -169,6 +169,12 @@ public function getFiles()
                 'template'      => '/php/src/Client.php.twig',
                 'minify'        => false,
             ],
+            [
+                'scope'         => 'default',
+                'destination'   => 'src/{{ spec.title | caseUcfirst}}/{{ spec.title | caseUcfirst}}Exception.php',
+                'template'      => '/php/src/Exception.php.twig',
+                'minify'        => false,
+            ],
             [
                 'scope'         => 'default',
                 'destination'   => '/src/{{ spec.title | caseUcfirst}}/Service.php',

templates/php/src/Client.php.twig

@@ -2,8 +2,6 @@
 
 namespace {{ spec.title | caseUcfirst }};
 
-use Exception;
-
 class Client
 {
     const METHOD_GET = 'GET';
@@ -114,7 +112,7 @@ class Client
      * @param array $params
      * @param array $headers
      * @return array|string
-     * @throws Exception
+     * @throws {{spec.title | caseUcfirst}}Exception
      */
     public function call($method, $path = '', $headers = array(), array $params = array())
     {
@@ -182,12 +180,17 @@ class Client
             break;
         }
 
-        if ((curl_errno($ch)/* || 200 != $responseStatus*/)) {
-            throw new Exception(curl_error($ch) . ' with status code ' . $responseStatus, $responseStatus);
+        if (curl_errno($ch)) {
+            throw new {{spec.title | caseUcfirst}}Exception(curl_error($ch), $responseStatus, $responseBody);
         }
-
+        
         curl_close($ch);
 
+        if($responseStatus >= 400) {
+            throw new {{spec.title | caseUcfirst}}Exception($responseBody['message'], $responseStatus, $responseBody);
+        }
+
+
         return $responseBody;
     }
 

templates/php/src/Exception.php.twig

@@ -0,0 +1,32 @@
+<?php
+
+namespace {{ spec.title | caseUcfirst }};
+
+use Exception;
+
+class {{spec.title | caseUcfirst}}Exception extends Exception {
+
+  /**
+   * @var mixed
+   */
+  private $response;
+
+  /**
+   * @param String $message
+   * @param int $code
+   * @param mixed $response
+   */
+  public function __construct($message = null, $code = 0, $response = null)
+  {
+      parent::__construct($message, $code);
+      $this->response = $response;
+  }
+  
+  /**
+   * @return mixed
+   */
+  final public function getResponse()
+  {
+      return $this->response;
+  }
+}