[JavaScript] Handle reserved operationId/model name in JS client

modules/swagger-codegen/src/main/java/io/swagger/codegen/languages/JavascriptClientCodegen.java

@@ -293,14 +293,18 @@ public String toParamName(String name) {
     public String toModelName(String name) {
         name = sanitizeName(name);  // FIXME parameter should not be assigned. Also declare it as "final"
 
+        // camelize the model name
+        // phone_number => PhoneNumber
+        name = camelize(name);
+
         // model name cannot use reserved keyword, e.g. return
         if (isReservedWord(name)) {
-            throw new RuntimeException(name + " (reserved word) cannot be used as a model name");
+            String modelName = "Object" + name;
+            LOGGER.warn(name + " (reserved word) cannot be used as model name. Renamed to " + modelName);
+            return modelName;
         }
 
-        // camelize the model name
-        // phone_number => PhoneNumber
-        return camelize(name);
+        return name;
     }
 
     @Override
@@ -419,12 +423,16 @@ public String toOperationId(String operationId) {
             throw new RuntimeException("Empty method/operation name (operationId) not allowed");
         }
 
+        operationId = camelize(sanitizeName(operationId), true);
+
         // method name cannot use reserved keyword, e.g. return
         if (isReservedWord(operationId)) {
-            operationId = escapeReservedWord(operationId);
+            String newOperationId = camelize("call_" + operationId, true);
+            LOGGER.warn(operationId + " (reserved word) cannot be used as method name. Renamed to " + newOperationId);
+            return newOperationId;
         }
 
-        return camelize(sanitizeName(operationId), true);
+        return operationId;
     }
 
     @Override

samples/client/petstore/javascript/src/api/PetApi.js

@@ -330,12 +330,12 @@
      * @param {function} callback the callback function, accepting three arguments: error, data, response
      *   data is of type: 'String'
      */
-    self.getPetByIdWithByteArray = function(petId, callback) {
+    self.petPetIdtestingByteArraytrueGet = function(petId, callback) {
       var postBody = null;
 
       // verify the required parameter 'petId' is set
       if (petId == null) {
-        throw "Missing the required parameter 'petId' when calling getPetByIdWithByteArray";
+        throw "Missing the required parameter 'petId' when calling petPetIdtestingByteArraytrueGet";
       }