Merge pull request #487 from appwrite/feat-refactor-ruby-templates

Author: lohanidamodar

templates/ruby/base/params.twig

@@ -0,0 +1,27 @@
+            params = {
+{% for parameter in method.parameters.query | merge(method.parameters.body)  %}
+                {{ parameter.name }}: {% if parameter.isGlobal %}@{% endif %}{{ parameter.name | caseSnake | escapeKeyword }},
+{% endfor %}
+            }
+            
+            headers = {
+{% for parameter in method.parameters.header %}
+                "{{ parameter.name }}": {{ parameter.name | caseCamel | escapeKeyword }},
+{% endfor %}
+{% for key, header in method.headers %}
+                "{{ key }}": '{{ header }}',
+{% endfor %}
+            }
+{% if method.parameters.all | length %}
+{% for parameter in method.parameters.all %}
+{% if parameter.required %}
+            if {% if parameter.isGlobal %}@{% endif %}{{ parameter.name | caseSnake | escapeKeyword }}.nil?
+                raise {{spec.title | caseUcfirst}}::Exception.new('Missing required parameter: "{{ parameter.name | caseCamel | escapeKeyword }}"')
+            end
+
+{% endif %}
+{% endfor %}
+{% for parameter in method.parameters.path %}
+                .gsub('{{ '{' }}{{ parameter.name }}{{ '}' }}', {% if parameter.isGlobal %}@{% endif %}{{ parameter.name | caseSnake | escapeKeyword }})
+{% endfor %}
+{% endif %}

templates/ruby/base/requests/api.twig

@@ -0,0 +1,9 @@
+            @client.call(
+                method: '{{ method.method | caseUpper }}',
+                path: path,
+                headers: headers,
+                params: params,
+{% if method.responseModel and method.responseModel != 'any' %}
+                response_type: Models::{{method.responseModel | caseUcfirst}}
+{% endif %}
+            )

templates/ruby/base/requests/file.twig

@@ -0,0 +1,19 @@
+            id_param_name = {% if method.parameters.all | filter(p => p.isUploadID) | length > 0 %}{% for parameter in method.parameters.all | filter(parameter => parameter.isUploadID) %}"{{ parameter.name }}"{% endfor %}{% else %}nil{% endif %}
+
+{% for parameter in method.parameters.all %}
+{% if parameter.type == 'file' %}
+            param_name = '{{ parameter.name }}'
+
+{% endif %}
+{% endfor %}
+            @client.chunked_upload(
+                path: path,
+                headers: headers,
+                params: params,
+                param_name: param_name,
+                id_param_name: id_param_name,
+                on_progress: on_progress,
+{% if method.responseModel and method.responseModel != 'any' %}
+                response_type: Models::{{method.responseModel | caseUcfirst}}
+{% endif %}
+            )

templates/ruby/lib/container/services/service.rb.twig

@@ -25,67 +25,18 @@ module {{spec.title | caseUcfirst}}
         #
         # @return [{{ method.responseModel | caseUcfirst }}]
         def {{ method.name | caseSnake }}({% for parameter in method.parameters.all | filter((param) => not param.isGlobal) %}{{ parameter.name | caseSnake | escapeKeyword }}:{% if not parameter.required %} nil{% endif %}{% if not loop.last %}, {% endif %}{% endfor %}{% if 'multipart/form-data' in method.consumes %}, on_progress: nil{% endif %})
-{% for parameter in method.parameters.all %}
-{% if parameter.required %}
-            if {% if parameter.isGlobal %}@{% endif %}{{ parameter.name | caseSnake | escapeKeyword }}.nil?
-                raise {{spec.title | caseUcfirst}}::Exception.new('Missing required parameter: "{{ parameter.name | caseCamel | escapeKeyword }}"')
-            end
 
-{% endif %}
-{% endfor %}
             path = '{{ method.path }}'
-{% for parameter in method.parameters.path %}
-                .gsub('{{ '{' }}{{ parameter.name }}{{ '}' }}', {% if parameter.isGlobal %}@{% endif %}{{ parameter.name | caseSnake | escapeKeyword }})
-{% endfor %}
-
-            params = {
-{% for parameter in method.parameters.query | merge(method.parameters.body)  %}
-                {{ parameter.name }}: {% if parameter.isGlobal %}@{% endif %}{{ parameter.name | caseSnake | escapeKeyword }},
-{% endfor %}
-            }
-
-            headers = {
-{% for parameter in method.parameters.header %}
-                "{{ parameter.name }}": {{ parameter.name | caseCamel | escapeKeyword }},
-{% endfor %}
-{% for key, header in method.headers %}
-                "{{ key }}": '{{ header }}',
-{% endfor %}
-            }
 
+{{ include('ruby/base/params.twig')}}
 {% if 'multipart/form-data' in method.consumes %}
-            id_param_name = {% if method.parameters.all | filter(p => p.isUploadID) | length > 0 %}{% for parameter in method.parameters.all | filter(parameter => parameter.isUploadID) %}"{{ parameter.name }}"{% endfor %}{% else %}nil{% endif %}
-
-{% for parameter in method.parameters.all %}
-{% if parameter.type == 'file' %}
-            param_name = '{{ parameter.name }}'
-
-{% endif %}
-{% endfor %}
-            @client.chunked_upload(
-                path: path,
-                headers: headers,
-                params: params,
-                param_name: param_name,
-                id_param_name: id_param_name,
-                on_progress: on_progress,
-{% if method.responseModel and method.responseModel != 'any' %}
-                response_type: Models::{{method.responseModel | caseUcfirst}}
-{% endif %}
-            )
+{{ include('ruby/base/requests/file.twig')}}
 {% else %}
-            @client.call(
-                method: '{{ method.method | caseUpper }}',
-                path: path,
-                headers: headers,
-                params: params,
-{% if method.responseModel and method.responseModel != 'any' %}
-                response_type: Models::{{method.responseModel | caseUcfirst}}
-{% endif %}
-            )
+{{ include('ruby/base/requests/api.twig')}}
 {% endif %}
         end
 
+        
 {% endfor %}
     end 
 end