Fix boolean value handling in HTTP request body (#1515)

* fix boolean http body handling

* fix typo in parameter name

samples/client/petstore/javascript-promise-es6/.openapi-generator/VERSION

@@ -1 +1 @@
-3.3.2-SNAPSHOT
+3.3.4-SNAPSHOT

samples/client/petstore/javascript-promise-es6/README.md

@@ -98,6 +98,7 @@ Class | Method | HTTP request | Description
 *OpenApiPetstore.FakeApi* | [**testClientModel**](docs/FakeApi.md#testClientModel) | **PATCH** /fake | To test \"client\" model
 *OpenApiPetstore.FakeApi* | [**testEndpointParameters**](docs/FakeApi.md#testEndpointParameters) | **POST** /fake | Fake endpoint for testing various parameters 假端點 偽のエンドポイント 가짜 엔드 포인트 
 *OpenApiPetstore.FakeApi* | [**testEnumParameters**](docs/FakeApi.md#testEnumParameters) | **GET** /fake | To test enum parameters
+*OpenApiPetstore.FakeApi* | [**testGroupParameters**](docs/FakeApi.md#testGroupParameters) | **DELETE** /fake | Fake endpoint to test group parameters (optional)
 *OpenApiPetstore.FakeApi* | [**testInlineAdditionalProperties**](docs/FakeApi.md#testInlineAdditionalProperties) | **POST** /fake/inline-additionalProperties | test inline additionalProperties
 *OpenApiPetstore.FakeApi* | [**testJsonFormData**](docs/FakeApi.md#testJsonFormData) | **GET** /fake/jsonFormData | test json serialization of form data
 *OpenApiPetstore.FakeClassnameTags123Api* | [**testClassname**](docs/FakeClassnameTags123Api.md#testClassname) | **PATCH** /fake_classname_test | To test class name in snake case

samples/client/petstore/javascript-promise-es6/docs/Category.md

@@ -4,6 +4,6 @@
 Name | Type | Description | Notes
 ------------ | ------------- | ------------- | -------------
 **id** | **Number** |  | [optional] 
-**name** | **String** |  | [optional] 
+**name** | **String** |  | [default to 'default-name']
 
 

samples/client/petstore/javascript-promise-es6/docs/FakeApi.md

@@ -13,6 +13,7 @@ Method | HTTP request | Description
 [**testClientModel**](FakeApi.md#testClientModel) | **PATCH** /fake | To test \"client\" model
 [**testEndpointParameters**](FakeApi.md#testEndpointParameters) | **POST** /fake | Fake endpoint for testing various parameters 假端點 偽のエンドポイント 가짜 엔드 포인트 
 [**testEnumParameters**](FakeApi.md#testEnumParameters) | **GET** /fake | To test enum parameters
+[**testGroupParameters**](FakeApi.md#testGroupParameters) | **DELETE** /fake | Fake endpoint to test group parameters (optional)
 [**testInlineAdditionalProperties**](FakeApi.md#testInlineAdditionalProperties) | **POST** /fake/inline-additionalProperties | test inline additionalProperties
 [**testJsonFormData**](FakeApi.md#testJsonFormData) | **GET** /fake/jsonFormData | test json serialization of form data
 
@@ -444,6 +445,59 @@ No authorization required
  - **Content-Type**: application/x-www-form-urlencoded
  - **Accept**: Not defined
 
+<a name="testGroupParameters"></a>
+# **testGroupParameters**
+> testGroupParameters(requiredStringGroup, requiredBooleanGroup, requiredInt64Group, opts)
+
+Fake endpoint to test group parameters (optional)
+
+Fake endpoint to test group parameters (optional)
+
+### Example
+```javascript
+import OpenApiPetstore from 'open_api_petstore';
+
+let apiInstance = new OpenApiPetstore.FakeApi();
+let requiredStringGroup = 56; // Number | Required String in group parameters
+let requiredBooleanGroup = true; // Boolean | Required Boolean in group parameters
+let requiredInt64Group = 789; // Number | Required Integer in group parameters
+let opts = {
+  'stringGroup': 56, // Number | String in group parameters
+  'booleanGroup': true, // Boolean | Boolean in group parameters
+  'int64Group': 789 // Number | Integer in group parameters
+};
+apiInstance.testGroupParameters(requiredStringGroup, requiredBooleanGroup, requiredInt64Group, opts).then(() => {
+  console.log('API called successfully.');
+}, (error) => {
+  console.error(error);
+});
+
+```
+
+### Parameters
+
+Name | Type | Description  | Notes
+------------- | ------------- | ------------- | -------------
+ **requiredStringGroup** | **Number**| Required String in group parameters | 
+ **requiredBooleanGroup** | **Boolean**| Required Boolean in group parameters | 
+ **requiredInt64Group** | **Number**| Required Integer in group parameters | 
+ **stringGroup** | **Number**| String in group parameters | [optional] 
+ **booleanGroup** | **Boolean**| Boolean in group parameters | [optional] 
+ **int64Group** | **Number**| Integer in group parameters | [optional] 
+
+### Return type
+
+null (empty response body)
+
+### Authorization
+
+No authorization required
+
+### HTTP request headers
+
+ - **Content-Type**: Not defined
+ - **Accept**: Not defined
+
 <a name="testInlineAdditionalProperties"></a>
 # **testInlineAdditionalProperties**
 > testInlineAdditionalProperties(requestBody)

samples/client/petstore/javascript-promise-es6/src/ApiClient.js

@@ -392,7 +392,7 @@ class ApiClient {
                     }
                 }
             }
-        } else if (bodyParam) {
+        } else if (bodyParam !== null && bodyParam !== undefined) {
             request.send(bodyParam);
         }
 
@@ -578,4 +578,4 @@ ApiClient.CollectionFormatEnum = {
 * @type {module:ApiClient}
 */
 ApiClient.instance = new ApiClient();
-export default ApiClient;
+export default ApiClient;

samples/client/petstore/javascript-promise-es6/src/api/FakeApi.js

@@ -552,6 +552,85 @@ export default class FakeApi {
     }
 
 
+    /**
+     * Fake endpoint to test group parameters (optional)
+     * Fake endpoint to test group parameters (optional)
+     * @param {Number} requiredStringGroup Required String in group parameters
+     * @param {Boolean} requiredBooleanGroup Required Boolean in group parameters
+     * @param {Number} requiredInt64Group Required Integer in group parameters
+     * @param {Object} opts Optional parameters
+     * @param {Number} opts.stringGroup String in group parameters
+     * @param {Boolean} opts.booleanGroup Boolean in group parameters
+     * @param {Number} opts.int64Group Integer in group parameters
+     * @return {Promise} a {@link https://www.promisejs.org/|Promise}, with an object containing HTTP response
+     */
+    testGroupParametersWithHttpInfo(requiredStringGroup, requiredBooleanGroup, requiredInt64Group, opts) {
+      opts = opts || {};
+      let postBody = null;
+
+      // verify the required parameter 'requiredStringGroup' is set
+      if (requiredStringGroup === undefined || requiredStringGroup === null) {
+        throw new Error("Missing the required parameter 'requiredStringGroup' when calling testGroupParameters");
+      }
+
+      // verify the required parameter 'requiredBooleanGroup' is set
+      if (requiredBooleanGroup === undefined || requiredBooleanGroup === null) {
+        throw new Error("Missing the required parameter 'requiredBooleanGroup' when calling testGroupParameters");
+      }
+
+      // verify the required parameter 'requiredInt64Group' is set
+      if (requiredInt64Group === undefined || requiredInt64Group === null) {
+        throw new Error("Missing the required parameter 'requiredInt64Group' when calling testGroupParameters");
+      }
+
+
+      let pathParams = {
+      };
+      let queryParams = {
+        'required_string_group': requiredStringGroup,
+        'required_int64_group': requiredInt64Group,
+        'string_group': opts['stringGroup'],
+        'int64_group': opts['int64Group']
+      };
+      let headerParams = {
+        'required_boolean_group': requiredBooleanGroup,
+        'boolean_group': opts['booleanGroup']
+      };
+      let formParams = {
+      };
+
+      let authNames = [];
+      let contentTypes = [];
+      let accepts = [];
+      let returnType = null;
+
+      return this.apiClient.callApi(
+        '/fake', 'DELETE',
+        pathParams, queryParams, headerParams, formParams, postBody,
+        authNames, contentTypes, accepts, returnType
+      );
+    }
+
+    /**
+     * Fake endpoint to test group parameters (optional)
+     * Fake endpoint to test group parameters (optional)
+     * @param {Number} requiredStringGroup Required String in group parameters
+     * @param {Boolean} requiredBooleanGroup Required Boolean in group parameters
+     * @param {Number} requiredInt64Group Required Integer in group parameters
+     * @param {Object} opts Optional parameters
+     * @param {Number} opts.stringGroup String in group parameters
+     * @param {Boolean} opts.booleanGroup Boolean in group parameters
+     * @param {Number} opts.int64Group Integer in group parameters
+     * @return {Promise} a {@link https://www.promisejs.org/|Promise}
+     */
+    testGroupParameters(requiredStringGroup, requiredBooleanGroup, requiredInt64Group, opts) {
+      return this.testGroupParametersWithHttpInfo(requiredStringGroup, requiredBooleanGroup, requiredInt64Group, opts)
+        .then(function(response_and_data) {
+          return response_and_data.data;
+        });
+    }
+
+
     /**
      * test inline additionalProperties
      * @param {Object.<String, {String: String}>} requestBody request body

samples/client/petstore/javascript-promise-es6/src/model/Category.js

@@ -22,10 +22,11 @@ class Category {
     /**
      * Constructs a new <code>Category</code>.
      * @alias module:model/Category
+     * @param name {String} 
      */
-    constructor() { 
+    constructor(name) { 
         
-        Category.initialize(this);
+        Category.initialize(this, name);
     }
 
     /**
@@ -33,7 +34,8 @@ class Category {
      * This method is used by the constructors of any subclasses, in order to implement multiple inheritance (mix-ins).
      * Only for internal use.
      */
-    static initialize(obj) { 
+    static initialize(obj, name) { 
+        obj['name'] = name;
     }
 
     /**
@@ -67,8 +69,9 @@ Category.prototype['id'] = undefined;
 
 /**
  * @member {String} name
+ * @default 'default-name'
  */
-Category.prototype['name'] = undefined;
+Category.prototype['name'] = 'default-name';