Merge pull request #1166 from appwrite/fix-snakes

Fix snake language multi gen with same cased name

Author: abnegate

templates/deno/mod.ts.twig

@@ -6,7 +6,7 @@ import { ID } from "./src/id.ts";
 import { InputFile } from "./src/inputFile.ts";
 import { {{spec.title | caseUcfirst}}Exception } from "./src/exception.ts";
 {% for service in spec.services %}
-import { {{service.name | caseUcfirst}} } from "./src/services/{{service.name | caseDash}}.ts";
+import { {{service.name | caseUcfirst}} } from "./src/services/{{service.name | caseKebab}}.ts";
 {% endfor %}
 {% for enum in spec.enums %}
 import { {{enum.name | caseUcfirst}} } from "./src/enums/{{enum.name | caseKebab}}.ts";

templates/python/package/services/service.py.twig

@@ -27,6 +27,17 @@ class {{ service.name | caseUcfirst }}(Service):
     def __init__(self, client) -> None:
         super({{ service.name | caseUcfirst }}, self).__init__(client)
 {% for method in service.methods %}
+{% set methodNameSnake = method.name | caseSnake %}
+{# Check if this method should be skipped (is deprecated and has a non-deprecated duplicate) #}
+{% set shouldSkip = false %}
+{% if method.deprecated %}
+{% for otherMethod in service.methods %}
+{% if not otherMethod.deprecated and (otherMethod.name | caseSnake) == methodNameSnake %}
+{% set shouldSkip = true %}
+{% endif %}
+{% endfor %}
+{% endif %}
+{% if not shouldSkip %}
 
     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 %}:
         """
@@ -76,4 +87,5 @@ class {{ service.name | caseUcfirst }}(Service):
 {% else %}
 {{ include('python/base/requests/api.twig') }}
 {% endif %}
+{% endif %}
 {% endfor %}

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

@@ -8,6 +8,17 @@ module {{spec.title | caseUcfirst}}
         end
 
 {% for method in service.methods %}
+{% set methodNameSnake = method.name | caseSnake %}
+{# Check if this method should be skipped (is deprecated and has a non-deprecated duplicate) #}
+{% set shouldSkip = false %}
+{% if method.deprecated %}
+{% for otherMethod in service.methods %}
+{% if not otherMethod.deprecated and (otherMethod.name | caseSnake) == methodNameSnake %}
+{% set shouldSkip = true %}
+{% endif %}
+{% endfor %}
+{% endif %}
+{% if not shouldSkip %}
 {% if method.deprecated %}
         #
 {%~ if method.since and method.replaceWith %}
@@ -33,7 +44,7 @@ module {{spec.title | caseUcfirst}}
 {% endif %}
         end
 
-        
+{% endif %}
 {% endfor %}
     end 
 end