fix (python): skip passing None to non nullable params

Author: ChiragAgg5k

src/SDK/Language/Python.php

@@ -399,6 +399,13 @@ public function getFilters(): array
             new TwigFilter('getPropertyType', function ($value, $method = []) {
                 return $this->getTypeName($value, $method);
             }),
+            new TwigFilter('formatParamValue', function (string $paramName, string $paramType, bool $isMultipartFormData) {
+                // For multipart/form-data, convert booleans to lowercase strings
+                if ($isMultipartFormData && $paramType !== 'string' && $paramType !== 'array') {
+                    return "str({$paramName}).lower() if type({$paramName}) is bool else {$paramName}";
+                }
+                return $paramName;
+            }),
         ];
     }
 }

templates/php/docs/service.md.twig

@@ -30,11 +30,6 @@
 | {{ parameter.name }} | {{ parameter.type }} | {{ parameter.description | raw }} | {{ parameter.default }} |
 {% endfor %}
 {% endif %}
-{% if method.parameters.formData | length > 0 %}
-{% for parameter in method.parameters.formData %}
-| {{ parameter.name }} | {{ parameter.type }} | {{ parameter.description | raw }} | {{ parameter.default }} |
-{% endfor %}
-{% endif %}
 
 {% endif %}
 {% endfor %}

templates/python/base/params.twig

@@ -12,20 +12,22 @@
 {% endfor %}
 
 {% for parameter in method.parameters.query %}
-        api_params['{{ parameter.name }}'] = {{ parameter.name | escapeKeyword | caseSnake }}
-{% endfor %}
-{% for parameter in method.parameters.body %}
-{% if method.consumes[0] == "multipart/form-data" and ( parameter.type != "string" and parameter.type != "array" ) %}
-        api_params['{{ parameter.name }}'] = str({{ parameter.name | escapeKeyword | caseSnake }}).lower() if type({{ parameter.name | escapeKeyword | caseSnake }}) is bool else {{ parameter.name | escapeKeyword | caseSnake }}
+{% if not parameter.nullable and not parameter.required %}
+        if {{ parameter.name | escapeKeyword | caseSnake }} is not None:
+            api_params['{{ parameter.name }}'] = {{ parameter.name | escapeKeyword | caseSnake }}
 {% else %}
         api_params['{{ parameter.name }}'] = {{ parameter.name | escapeKeyword | caseSnake }}
 {% endif %}
 {% endfor %}
-{% for parameter in method.parameters.formData %}
-{% if method.consumes[0] == "multipart/form-data" and ( parameter.type != "string" and parameter.type != "array" ) %}
-        api_params['{{ parameter.name }}'] = str({{ parameter.name | escapeKeyword | caseSnake }}).lower() if type({{ parameter.name | escapeKeyword | caseSnake }}) is bool else {{ parameter.name | escapeKeyword | caseSnake }}
+{% for parameter in method.parameters.body %}
+{% set paramName = parameter.name | escapeKeyword | caseSnake %}
+{% set isMultipart = method.consumes[0] == "multipart/form-data" %}
+{% set formattedValue = paramName | formatParamValue(parameter.type, isMultipart) %}
+{% if not parameter.nullable and not parameter.required %}
+        if {{ paramName }} is not None:
+            api_params['{{ parameter.name }}'] = {{ formattedValue }}
 {% else %}
-        api_params['{{ parameter.name }}'] = {{ parameter.name | escapeKeyword | caseSnake }}
+        api_params['{{ parameter.name }}'] = {{ formattedValue }}
 {% endif %}
 {% endfor %}
 {% endif %}