From 0f16057f43c795dbf5eee6211599ef26b832ec57 Mon Sep 17 00:00:00 2001 From: ChiragAgg5k Date: Tue, 25 Mar 2025 19:51:01 +0530 Subject: [PATCH 1/3] chore: add another space --- templates/python/package/services/service.py.twig | 1 + 1 file changed, 1 insertion(+) diff --git a/templates/python/package/services/service.py.twig b/templates/python/package/services/service.py.twig index 463f68719..6ef530c4b 100644 --- a/templates/python/package/services/service.py.twig +++ b/templates/python/package/services/service.py.twig @@ -31,6 +31,7 @@ class {{ service.name | caseUcfirst }}(Service): def {{ method.name | caseSnake }}(self{% if method.parameters.all|length > 0 %}, {% endif %}{% for parameter in method.parameters.all %}{{ parameter.name | escapeKeyword | caseSnake }}: {{ parameter | getPropertyType(method) | raw }}{% if not parameter.required %} = None{% endif %}{% if not loop.last %}, {% endif %}{% endfor %}{% if 'multipart/form-data' in method.consumes %}, on_progress = None{% endif %}) -> {% if method.type == 'webAuth' %}str{% else %}Dict[str, Any]{% endif %}: """ {% autoescape false %}{{ method.description | replace({"\n": "\n "}) }}{% endautoescape %} + {% if method.parameters.all|length > 0 or 'multipart/form-data' in method.consumes %} Parameters From 2d0d7659342f2eaa6df3154b179e7e6986c7a23a Mon Sep 17 00:00:00 2001 From: ChiragAgg5k Date: Tue, 25 Mar 2025 21:29:42 +0530 Subject: [PATCH 2/3] fix: return type for location --- templates/python/package/services/service.py.twig | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/templates/python/package/services/service.py.twig b/templates/python/package/services/service.py.twig index 6ef530c4b..14f7a3481 100644 --- a/templates/python/package/services/service.py.twig +++ b/templates/python/package/services/service.py.twig @@ -28,7 +28,7 @@ class {{ service.name | caseUcfirst }}(Service): super({{ service.name | caseUcfirst }}, self).__init__(client) {% for method in service.methods %} - def {{ method.name | caseSnake }}(self{% if method.parameters.all|length > 0 %}, {% endif %}{% for parameter in method.parameters.all %}{{ parameter.name | escapeKeyword | caseSnake }}: {{ parameter | getPropertyType(method) | raw }}{% if not parameter.required %} = None{% endif %}{% if not loop.last %}, {% endif %}{% endfor %}{% if 'multipart/form-data' in method.consumes %}, on_progress = None{% endif %}) -> {% if method.type == 'webAuth' %}str{% else %}Dict[str, Any]{% endif %}: + def {{ method.name | caseSnake }}(self{% if method.parameters.all|length > 0 %}, {% endif %}{% for parameter in method.parameters.all %}{{ parameter.name | escapeKeyword | caseSnake }}: {{ parameter | getPropertyType(method) | raw }}{% if not parameter.required %} = None{% endif %}{% if not loop.last %}, {% endif %}{% endfor %}{% if 'multipart/form-data' in method.consumes %}, on_progress = None{% endif %}) -> {% if method.type == 'webAuth' or method.type == 'location' %}str{% else %}Dict[str, Any]{% endif %}: """ {% autoescape false %}{{ method.description | replace({"\n": "\n "}) }}{% endautoescape %} @@ -48,6 +48,8 @@ class {{ service.name | caseUcfirst }}(Service): ------- {% if method.type == 'webAuth' %}str Authentication response as a string + {% elseif method.type == 'location' %}str + Response as a string {% else %}Dict[str, Any] API response as a dictionary {% endif %} From c1a931e3ff49a365b4e3079cbf65f8f7ff409c12 Mon Sep 17 00:00:00 2001 From: ChiragAgg5k Date: Tue, 25 Mar 2025 22:02:10 +0530 Subject: [PATCH 3/3] fix: to use bytes --- templates/python/package/services/service.py.twig | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/templates/python/package/services/service.py.twig b/templates/python/package/services/service.py.twig index 14f7a3481..2eca9a794 100644 --- a/templates/python/package/services/service.py.twig +++ b/templates/python/package/services/service.py.twig @@ -28,7 +28,7 @@ class {{ service.name | caseUcfirst }}(Service): super({{ service.name | caseUcfirst }}, self).__init__(client) {% for method in service.methods %} - def {{ method.name | caseSnake }}(self{% if method.parameters.all|length > 0 %}, {% endif %}{% for parameter in method.parameters.all %}{{ parameter.name | escapeKeyword | caseSnake }}: {{ parameter | getPropertyType(method) | raw }}{% if not parameter.required %} = None{% endif %}{% if not loop.last %}, {% endif %}{% endfor %}{% if 'multipart/form-data' in method.consumes %}, on_progress = None{% endif %}) -> {% if method.type == 'webAuth' or method.type == 'location' %}str{% else %}Dict[str, Any]{% endif %}: + def {{ method.name | caseSnake }}(self{% if method.parameters.all|length > 0 %}, {% endif %}{% for parameter in method.parameters.all %}{{ parameter.name | escapeKeyword | caseSnake }}: {{ parameter | getPropertyType(method) | raw }}{% if not parameter.required %} = None{% endif %}{% if not loop.last %}, {% endif %}{% endfor %}{% if 'multipart/form-data' in method.consumes %}, on_progress = None{% endif %}) -> {% if method.type == 'webAuth' %}str{% elseif method.type == 'location' %}bytes{% else %}Dict[str, Any]{% endif %}: """ {% autoescape false %}{{ method.description | replace({"\n": "\n "}) }}{% endautoescape %} @@ -48,8 +48,8 @@ class {{ service.name | caseUcfirst }}(Service): ------- {% if method.type == 'webAuth' %}str Authentication response as a string - {% elseif method.type == 'location' %}str - Response as a string + {% elseif method.type == 'location' %}bytes + Response as bytes {% else %}Dict[str, Any] API response as a dictionary {% endif %}