chore: simplify check and deprecated old methods

Author: ChiragAgg5k

example.php

@@ -38,8 +38,8 @@ function getSSLPage($url) {
     }
 
     // Leave the platform you want uncommented
-    $platform = 'client';
-    // $platform = 'console';
+    // $platform = 'client';
+    $platform = 'console';
     // $platform = 'server';
 
     $version = '1.7.x';

templates/web/src/services/template.ts.twig

@@ -28,7 +28,7 @@ export class {{ service.name | caseUcfirst }} {
     {%~ for method in service.methods %}
     /**
     {%~ if method.description %}
-     * {{ method.description | replace({'\n': '\n     * '}) }}
+     * {{ method.description | replace({'\n': '\n     * '}) | raw }}
     {%~ endif %}
      *
     {%~ for parameter in method.parameters.all %}
@@ -46,6 +46,9 @@ export class {{ service.name | caseUcfirst }} {
      */
     {%~ if method.parameters.all|length > 0 %}
     {{ method.name | caseCamel }}{{ method.responseModel | getGenerics(spec) | raw }}(params: { {% for parameter in method.parameters.all %}{{ parameter.name | caseCamel | escapeKeyword }}{% if not parameter.required or parameter.nullable %}?{% endif %}: {{ parameter | getPropertyType(method) | raw }}{% if not loop.last %}, {% endif %}{% endfor %} {% if 'multipart/form-data' in method.consumes %}, onProgress?: (progress: UploadProgress) => {}{% endif %} }): {{ method | getReturn(spec) | raw }};
+    /**
+     * @deprecated This API has been deprecated.
+     */
     {{ method.name | caseCamel }}{{ method.responseModel | getGenerics(spec) | raw }}({% for parameter in method.parameters.all %}{{ parameter.name | caseCamel | escapeKeyword }}{% if not parameter.required or parameter.nullable %}?{% endif %}: {{ parameter | getPropertyType(method) | raw }}{% if not loop.last %}, {% endif %}{% endfor %}{% if 'multipart/form-data' in method.consumes %}, onProgress?: (progress: UploadProgress) => {}{% endif %}): {{ method | getReturn(spec) | raw }};
     {{ method.name | caseCamel }}{{ method.responseModel | getGenerics(spec) | raw }}(
         {% if method.parameters.all|length > 0 %}paramsOrFirst{% if not method.parameters.all[0].required or method.parameters.all[0].nullable %}?{% endif %}: { {% for parameter in method.parameters.all %}{{ parameter.name | caseCamel | escapeKeyword }}{% if not parameter.required or parameter.nullable %}?{% endif %}: {{ parameter | getPropertyType(method) | raw }}{% if not loop.last %}, {% endif %}{% endfor %}{% if 'multipart/form-data' in method.consumes %}, onProgress?: (progress: UploadProgress) => {} {% endif %} } | {{ method.parameters.all[0] | getPropertyType(method) | raw }}{% if method.parameters.all|length > 1 %},
@@ -58,8 +61,7 @@ export class {{ service.name | caseUcfirst }} {
         let onProgress: ((progress: UploadProgress) => {});
         {%~ endif %}
 
-        if (typeof paramsOrFirst === 'object' && paramsOrFirst !== null && !Array.isArray(paramsOrFirst)
-            {% if method.parameters.all|length > 0 %}&& ({% for parameter in method.parameters.all %}'{{ parameter.name | caseCamel | escapeKeyword }}' in paramsOrFirst{% if not loop.last %} || {% endif %}{% endfor %} || Object.keys(paramsOrFirst).length === 0){% endif %}) {
+        if (paramsOrFirst && typeof paramsOrFirst === 'object' && !Array.isArray(paramsOrFirst)) {
             params = paramsOrFirst as { {% for parameter in method.parameters.all %}{{ parameter.name | caseCamel | escapeKeyword }}{% if not parameter.required or parameter.nullable %}?{% endif %}: {{ parameter | getPropertyType(method) | raw }}{% if not loop.last %}, {% endif %}{% endfor %} };
             {%~ if 'multipart/form-data' in method.consumes %}
             onProgress = paramsOrFirst.onProgress as ((progress: UploadProgress) => {});