Force strings for multi-part upload at client level

Author: abnegate

composer.lock

@@ -49,6 +49,7 @@ class Client:
         data = {}
         json = {}
         files = {}
+        stringify = False
 
         headers = {**self._global_headers, **headers}
 
@@ -62,7 +63,7 @@ class Client:
 
         if headers['content-type'].startswith('multipart/form-data'):
             del headers['content-type']
-            
+            stringify = True
             for key in data.copy():
                 if isinstance(data[key], InputFile):
                     files[key] = (data[key].name, data[key].file)
@@ -72,7 +73,7 @@ class Client:
             response = requests.request(  # call method dynamically https://stackoverflow.com/a/4246075/2299554
                 method=method,
                 url=self._endpoint + path,
-                params=self.flatten(params),
+                params=self.flatten(params, stringify=stringify),
                 data=self.flatten(data),
                 json=json,
                 files=files,
@@ -152,7 +153,7 @@ class Client:
 
         return result
 
-    def flatten(self, data, prefix=''):
+    def flatten(self, data, prefix='', stringify=False):
         output = {}
         i = 0
 
@@ -163,9 +164,12 @@ class Client:
             i += 1
 
             if isinstance(value, list) or isinstance(value, dict):
-                output = {**output, **self.flatten(value, finalKey)}
+                output = {**output, **self.flatten(value, finalKey, stringify)}
             else:
-                output[finalKey] = value
+                if stringify:
+                    output[finalKey] = str(value)
+                else:
+                    output[finalKey] = value
 
         return output
 

templates/python/package/client.py.twig

@@ -27,17 +27,17 @@ class {{ service.name | caseUcfirst }}(Service):
 
 {% for parameter in method.parameters.query %}
         if {{ parameter.name | escapeKeyword | caseSnake }} is not None: 
-            params['{{ parameter.name }}'] = {% if method.consumes[0] == "multipart/form-data" and ( parameter.type != "string" and parameter.type != "array" ) %}'true' if {{ parameter.name | escapeKeyword | caseSnake }} else 'false'{% else %}{{ parameter.name | escapeKeyword | caseSnake }}{% endif %}
+            params['{{ parameter.name }}'] = {{ parameter.name | escapeKeyword | caseSnake }}
 
 {% endfor %}
 {% for parameter in method.parameters.body %}
         if {{ parameter.name | escapeKeyword | caseSnake }} is not None: 
-            params['{{ parameter.name }}'] = {% if method.consumes[0] == "multipart/form-data" and ( parameter.type != "string" and parameter.type != "array" ) %}'true' if {{ parameter.name | escapeKeyword | caseSnake }} else 'false'{% else %}{{ parameter.name | escapeKeyword | caseSnake }}{% endif %}
+            params['{{ parameter.name }}'] = {{ parameter.name | escapeKeyword | caseSnake }}
 
 {% endfor %}
 {% for parameter in method.parameters.formData %}
         if {{ parameter.name | escapeKeyword | caseSnake }} is not None: 
-            params['{{ parameter.name }}'] = {% if method.consumes[0] == "multipart/form-data" and ( parameter.type != "string" and parameter.type != "array" ) %}'true' if {{ parameter.name | escapeKeyword | caseSnake }} else 'false'{% else %}{{ parameter.name | escapeKeyword | caseSnake }}{% endif %}
+            params['{{ parameter.name }}'] = {{ parameter.name | escapeKeyword | caseSnake }}
 
 {% endfor %}
 {% if 'multipart/form-data' in method.consumes %}