chore: remove response attribute from appwriteexception

appwrite · Jan 28, 2025 · 6e45683 · 6e45683
1 parent 612b2e0
commit 6e45683
Show file tree

Hide file tree

Showing 21 changed files with 28 additions and 71 deletions.
diff --git a/templates/android/library/src/main/java/io/package/Client.kt.twig b/templates/android/library/src/main/java/io/package/Client.kt.twig
@@ -486,8 +486,7 @@ class Client @JvmOverloads constructor(
                         {{ spec.title | caseUcfirst }}Exception(
                             map["message"] as? String ?: "",
                             (map["code"] as Number).toInt(),
-                            map["type"] as? String ?: "",
-                            body
+                            map["type"] as? String ?: ""
                         )
                     } else {
                         {{ spec.title | caseUcfirst }}Exception(body, response.code)

diff --git a/templates/android/library/src/main/java/io/package/exceptions/Exception.kt.twig b/templates/android/library/src/main/java/io/package/exceptions/Exception.kt.twig
@@ -5,6 +5,5 @@ import java.lang.Exception
 class {{spec.title | caseUcfirst}}Exception(
     override val message: String? = null,
     val code: Int? = null,
-    val type: String? = null,
-    val response: String? = null
+    val type: String? = null
 ) : Exception(message)
diff --git a/templates/cli/lib/client.js.twig b/templates/cli/lib/client.js.twig
@@ -148,7 +148,7 @@ class Client {
       try {
         json = JSON.parse(text);
       } catch (error) {
-        throw new {{spec.title | caseUcfirst}}Exception(text, response.status, "", text);
+        throw new {{spec.title | caseUcfirst}}Exception(text, response.status, "");
       }
 
       if (path !== '/account' && json.code === 401 && json.type === 'user_more_factors_required') {
@@ -158,7 +158,7 @@ class Client {
         globalConfig.setCurrentSession('');
         globalConfig.removeSession(current);
       }
-      throw new {{spec.title | caseUcfirst}}Exception(json.message, json.code, json.type, json);
+      throw new {{spec.title | caseUcfirst}}Exception(json.message, json.code, json.type);
     }
 
     if (responseType === "arraybuffer") {

diff --git a/templates/cli/lib/exception.js.twig b/templates/cli/lib/exception.js.twig
@@ -1,8 +1,7 @@
 class {{ spec.title| caseUcfirst }}Exception extends Error {
-  constructor(message, code, response) {
+  constructor(message, code) {
     super(message);
     this.code = code;
-    this.response = response;
   }
 }
 

diff --git a/templates/dart/lib/src/client_mixin.dart.twig b/templates/dart/lib/src/client_mixin.dart.twig
@@ -81,7 +81,6 @@ class ClientMixin {
           response['message'],
           response['code'],
           response['type'],
-          response,
         );
       } else {
         throw {{spec.title | caseUcfirst}}Exception(res.body);

diff --git a/templates/dart/lib/src/exception.dart.twig b/templates/dart/lib/src/exception.dart.twig
@@ -9,10 +9,9 @@ class {{spec.title | caseUcfirst}}Exception implements Exception {
   /// for more information.
   final String? type;
   final int? code;
-  final dynamic response;
 
   /// Initializes an {{spec.title | caseUcfirst}} Exception.
-  {{spec.title | caseUcfirst}}Exception([this.message = "", this.code, this.type, this.response]);
+  {{spec.title | caseUcfirst}}Exception([this.message = "", this.code, this.type]);
 
   /// Returns the error type, message, and code.
   @override

diff --git a/templates/deno/src/client.ts.twig b/templates/deno/src/client.ts.twig
@@ -111,9 +111,9 @@ export class Client {
             try {
                 json = JSON.parse(text);
             } catch (error) {
-                throw new {{spec.title | caseUcfirst}}Exception(text, response.status, "", text);
+                throw new {{spec.title | caseUcfirst}}Exception(text, response.status, "");
             }
-            throw new {{spec.title | caseUcfirst}}Exception(json.message, json.code, json.type, json);
+            throw new {{spec.title | caseUcfirst}}Exception(json.message, json.code, json.type);
         }
 
         if (responseType === "arraybuffer") {

diff --git a/templates/deno/src/exception.ts.twig b/templates/deno/src/exception.ts.twig
@@ -1,17 +1,15 @@
 export class {{ spec.title | caseUcfirst}}Exception {
     message: string;
     code: number;
-    response: any;
     type: string;
 
-    constructor(message: string, code: number = 0, type: string = "", response: any = "") {
+    constructor(message: string, code: number = 0, type: string = "") {
         this.message = message;
         this.code = code;
         this.type = type;
-        this.response = response;
     }
 
     public toString(): string {
-        return `${this.message} - ${this.code} - ${this.type} - ${JSON.stringify(this.response)}`;
+        return `${this.message} - ${this.code} - ${this.type}`;
     }
 }
diff --git a/templates/dotnet/Package/Exception.cs.twig b/templates/dotnet/Package/Exception.cs.twig
@@ -6,17 +6,14 @@ namespace {{spec.title | caseUcfirst}}
     {
         public int? Code { get; set; }
         public string? Type { get; set; } = null;
-        public string? Response { get; set; } = null;
 
         public {{spec.title | caseUcfirst}}Exception(
             string? message = null,
             int? code = null,
-            string? type = null,
-            string? response = null) : base(message)
+            string? type = null) : base(message)
         {
             this.Code = code;
             this.Type = type;
-            this.Response = response;
         }
         public {{spec.title | caseUcfirst}}Exception(string message, Exception inner)
         : base(message, inner)

diff --git a/templates/flutter/lib/src/client_mixin.dart.twig b/templates/flutter/lib/src/client_mixin.dart.twig
@@ -81,7 +81,6 @@ class ClientMixin {
           response['message'],
           response['code'],
           response['type'],
-          response,
         );
       } else {
         throw {{spec.title | caseUcfirst}}Exception(res.body);

diff --git a/templates/kotlin/src/main/kotlin/io/appwrite/Client.kt.twig b/templates/kotlin/src/main/kotlin/io/appwrite/Client.kt.twig
@@ -482,8 +482,7 @@ class Client @JvmOverloads constructor(
                         {{ spec.title | caseUcfirst }}Exception(
                             map["message"] as? String ?: "",
                             (map["code"] as Number).toInt(),
-                            map["type"] as? String ?: "",
-                            body
+                            map["type"] as? String ?: ""
                         )
                     } else {
                         {{ spec.title | caseUcfirst }}Exception(body, response.code)
@@ -533,8 +532,7 @@ class Client @JvmOverloads constructor(
                         {{ spec.title | caseUcfirst }}Exception(
                             map["message"] as? String ?: "",
                             (map["code"] as Number).toInt(),
-                            map["type"] as? String ?: "",
-                            body
+                            map["type"] as? String ?: ""
                         )
                     } else {
                         {{ spec.title | caseUcfirst }}Exception(body, response.code)

diff --git a/templates/kotlin/src/main/kotlin/io/appwrite/exceptions/Exception.kt.twig b/templates/kotlin/src/main/kotlin/io/appwrite/exceptions/Exception.kt.twig
@@ -5,6 +5,5 @@ import java.lang.Exception
 class {{spec.title | caseUcfirst}}Exception(
     override val message: String? = null,
     val code: Int? = null,
-    val type: String? = null,
-    val response: String? = null
+    val type: String? = null
 ) : Exception(message)
diff --git a/templates/node/src/client.ts.twig b/templates/node/src/client.ts.twig
@@ -20,15 +20,13 @@ type Headers = {
 
 class {{spec.title | caseUcfirst}}Exception extends Error {
     code: number;
-    response: string;
     type: string;
     constructor(message: string, code: number = 0, type: string = '', response: string = '') {
         super(message);
         this.name = '{{spec.title | caseUcfirst}}Exception';
         this.message = message;
         this.code = code;
         this.type = type;
-        this.response = response;
     }
 }
 
@@ -282,7 +280,7 @@ class Client {
         }
 
         if (400 <= response.status) {
-            throw new {{spec.title | caseUcfirst}}Exception(data?.message, response.status, data?.type, data);
+            throw new {{spec.title | caseUcfirst}}Exception(data?.message, response.status, data?.type);
         }
 
         return data;

diff --git a/templates/php/src/Client.php.twig b/templates/php/src/Client.php.twig
@@ -197,14 +197,14 @@ class Client
         }
 
         if (curl_errno($ch)) {
-            throw new {{spec.title | caseUcfirst}}Exception(curl_error($ch), $responseStatus, $responseBody['type'] ?? '', $responseBody);
+            throw new {{spec.title | caseUcfirst}}Exception(curl_error($ch), $responseStatus, $responseBody['type'] ?? '');
         }
         
         curl_close($ch);
 
         if($responseStatus >= 400) {
             if(is_array($responseBody)) {
-                throw new {{spec.title | caseUcfirst}}Exception($responseBody['message'], $responseStatus, $responseBody['type'] ?? '', json_encode($responseBody));
+                throw new {{spec.title | caseUcfirst}}Exception($responseBody['message'], $responseStatus, $responseBody['type'] ?? '');
             } else {
                 throw new {{spec.title | caseUcfirst}}Exception($responseBody, $responseStatus);
             }

diff --git a/templates/php/src/Exception.php.twig b/templates/php/src/Exception.php.twig
@@ -6,11 +6,6 @@ use Exception;
 
 class {{spec.title | caseUcfirst}}Exception extends Exception {
 
-  /**
-   * @var mixed
-   */
-  private ?string $response;
-
   /**
    * @var string
    */
@@ -20,16 +15,13 @@ class {{spec.title | caseUcfirst}}Exception extends Exception {
    * @param ?string $message
    * @param int $code
    * @param string $type
-   * @param ?string $response
    */
   public function __construct(
       ?string $message = null,
       int $code = 0,
-      string $type = '',
-      ?string $response = null
+      string $type = ''
   ) {
       parent::__construct($message, $code);
-      $this->response = $response;
       $this->type = $type;
   }
 
@@ -40,12 +32,4 @@ class {{spec.title | caseUcfirst}}Exception extends Exception {
   {
       return $this->type;
   }
-  
-  /**
-   * @return ?string
-   */
-  final public function getResponse(): ?string
-  {
-      return $this->response;
-  }
 }
diff --git a/templates/python/package/client.py.twig b/templates/python/package/client.py.twig
@@ -109,7 +109,7 @@ class Client:
             if response != None:
                 content_type = response.headers['Content-Type']
                 if content_type.startswith('application/json'):
-                    raise {{spec.title | caseUcfirst}}Exception(response.json()['message'], response.status_code, response.json().get('type'), response.json())
+                    raise {{spec.title | caseUcfirst}}Exception(response.json()['message'], response.status_code, response.json().get('type'))
                 else:
                     raise {{spec.title | caseUcfirst}}Exception(response.text, response.status_code)
             else:

diff --git a/templates/python/package/exception.py.twig b/templates/python/package/exception.py.twig
@@ -1,7 +1,6 @@
 class {{spec.title | caseUcfirst}}Exception(Exception):
-    def __init__(self, message, code = 0, type = None, response = None):
+    def __init__(self, message, code = 0, type = None):
         self.message = message
         self.code = code
         self.type = type
-        self.response = response
         super().__init__(self.message)
diff --git a/templates/react-native/src/client.ts.twig b/templates/react-native/src/client.ts.twig
@@ -88,15 +88,13 @@ export type UploadProgress = {
 
 class {{spec.title | caseUcfirst}}Exception extends Error {
     code: number;
-    response: string;
     type: string;
-    constructor(message: string, code: number = 0, type: string = '', response: string = '') {
+    constructor(message: string, code: number = 0, type: string = '') {
         super(message);
         this.name = '{{spec.title | caseUcfirst}}Exception';
         this.message = message;
         this.code = code;
         this.type = type;
-        this.response = response;
     }
 }
 
@@ -430,7 +428,7 @@ class Client {
             }
 
             if (400 <= response.status) {
-                throw new {{spec.title | caseUcfirst}}Exception(data?.message, response.status, data?.type, data);
+                throw new {{spec.title | caseUcfirst}}Exception(data?.message, response.status, data?.type);
             }
 
             const cookieFallback = response.headers.get('X-Fallback-Cookies');

diff --git a/templates/ruby/lib/container/client.rb.twig b/templates/ruby/lib/container/client.rb.twig
@@ -248,11 +248,11 @@ module {{ spec.title | caseUcfirst }}
                 begin
                     result = JSON.parse(response.body)
                 rescue JSON::ParserError => e
-                    raise {{spec.title | caseUcfirst}}::Exception.new(response.body, response.code, nil, response)
+                    raise {{spec.title | caseUcfirst}}::Exception.new(response.body, response.code)
                 end
 
                 if response.code.to_i >= 400
-                    raise {{spec.title | caseUcfirst}}::Exception.new(result['message'], result['status'], result['type'], result)
+                    raise {{spec.title | caseUcfirst}}::Exception.new(result['message'], result['status'], result['type'])
                 end
 
                 unless response_type.respond_to?("from")
@@ -263,7 +263,7 @@ module {{ spec.title | caseUcfirst }}
             end
 
             if response.code.to_i >= 400
-                raise {{spec.title | caseUcfirst}}::Exception.new(response.body, response.code, response)
+                raise {{spec.title | caseUcfirst}}::Exception.new(response.body, response.code)
             end
 
             if response.respond_to?("body_permitted?")

diff --git a/templates/ruby/lib/container/exception.rb.twig b/templates/ruby/lib/container/exception.rb.twig
@@ -2,13 +2,11 @@ module {{spec.title | caseUcfirst}}
     class Exception < StandardError
         attr_reader :code
         attr_reader :response
-        attr_reader :type
 
-        def initialize(message, code = 0, type = nil, response = nil)
+        def initialize(message, code = 0, type = nil)
             super(message)
             @code = code
             @type = type
-            @response = response
         end
     end
 end
diff --git a/templates/web/src/client.ts.twig b/templates/web/src/client.ts.twig
@@ -262,11 +262,6 @@ class {{spec.title | caseUcfirst}}Exception extends Error {
      */
     code: number;
 
-    /**
-     * The response string associated with the exception.
-     */
-    response: string;
-
     /**
      * Error type.
      * See [Error Types]({{sdk.url}}/docs/response-codes#errorTypes) for more information.
@@ -281,13 +276,12 @@ class {{spec.title | caseUcfirst}}Exception extends Error {
      * @param {string} type - The error type. Default is an empty string.
      * @param {string} response - The response string. Default is an empty string.
      */
-    constructor(message: string, code: number = 0, type: string = '', response: string = '') {
+    constructor(message: string, code: number = 0, type: string = '') {
         super(message);
         this.name = '{{spec.title | caseUcfirst}}Exception';
         this.message = message;
         this.code = code;
         this.type = type;
-        this.response = response;
     }
 }
 
@@ -688,7 +682,7 @@ class Client {
         }
 
         if (400 <= response.status) {
-            throw new {{spec.title | caseUcfirst}}Exception(data?.message, response.status, data?.type, data);
+            throw new {{spec.title | caseUcfirst}}Exception(data?.message, response.status, data?.type);
         }
 
         const cookieFallback = response.headers.get('X-Fallback-Cookies');