Skip to content

Commit 5d05cc0

Browse files
null-devwing328
authored andcommitted
Fix body serialization in javascript-flowtyped generator when body is falsy (#2499)
1 parent 28ae33c commit 5d05cc0

File tree

2 files changed

+8
-8
lines changed
  • modules/openapi-generator/src/main/resources/Javascript-Flowtyped
  • samples/client/petstore/javascript-flowtyped/src

2 files changed

+8
-8
lines changed

modules/openapi-generator/src/main/resources/Javascript-Flowtyped/api.mustache

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -214,7 +214,7 @@ export const {{classname}}FetchParamCreator = function (configuration?: Configur
214214
{{/hasFormParams}}
215215
{{#bodyParam}}
216216
const needsSerialization = (typeof {{paramName}} !== "string") || localVarRequestOptions.headers['Content-Type'] === 'application/json';
217-
localVarRequestOptions.body = needsSerialization ? JSON.stringify({{paramName}} || {}) : ((({{paramName}}:any):string) || "");
217+
localVarRequestOptions.body = needsSerialization ? JSON.stringify({{paramName}} != null ? {{paramName}} : {}) : ((({{paramName}}:any):string) || "");
218218
{{/bodyParam}}
219219

220220
return {

samples/client/petstore/javascript-flowtyped/src/api.js

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -323,7 +323,7 @@ export const PetApiFetchParamCreator = function (configuration?: Configuration)
323323
delete localVarUrlObj.search;
324324
localVarRequestOptions.headers = Object.assign({}, localVarHeaderParameter, options.headers);
325325
const needsSerialization = (typeof body !== "string") || localVarRequestOptions.headers['Content-Type'] === 'application/json';
326-
localVarRequestOptions.body = needsSerialization ? JSON.stringify(body || {}) : (((body:any):string) || "");
326+
localVarRequestOptions.body = needsSerialization ? JSON.stringify(body != null ? body : {}) : (((body:any):string) || "");
327327

328328
return {
329329
url: url.format(localVarUrlObj),
@@ -515,7 +515,7 @@ export const PetApiFetchParamCreator = function (configuration?: Configuration)
515515
delete localVarUrlObj.search;
516516
localVarRequestOptions.headers = Object.assign({}, localVarHeaderParameter, options.headers);
517517
const needsSerialization = (typeof body !== "string") || localVarRequestOptions.headers['Content-Type'] === 'application/json';
518-
localVarRequestOptions.body = needsSerialization ? JSON.stringify(body || {}) : (((body:any):string) || "");
518+
localVarRequestOptions.body = needsSerialization ? JSON.stringify(body != null ? body : {}) : (((body:any):string) || "");
519519

520520
return {
521521
url: url.format(localVarUrlObj),
@@ -878,7 +878,7 @@ export const StoreApiFetchParamCreator = function (configuration?: Configuration
878878
delete localVarUrlObj.search;
879879
localVarRequestOptions.headers = Object.assign({}, localVarHeaderParameter, options.headers);
880880
const needsSerialization = (typeof body !== "string") || localVarRequestOptions.headers['Content-Type'] === 'application/json';
881-
localVarRequestOptions.body = needsSerialization ? JSON.stringify(body || {}) : (((body:any):string) || "");
881+
localVarRequestOptions.body = needsSerialization ? JSON.stringify(body != null ? body : {}) : (((body:any):string) || "");
882882

883883
return {
884884
url: url.format(localVarUrlObj),
@@ -997,7 +997,7 @@ export const UserApiFetchParamCreator = function (configuration?: Configuration)
997997
delete localVarUrlObj.search;
998998
localVarRequestOptions.headers = Object.assign({}, localVarHeaderParameter, options.headers);
999999
const needsSerialization = (typeof body !== "string") || localVarRequestOptions.headers['Content-Type'] === 'application/json';
1000-
localVarRequestOptions.body = needsSerialization ? JSON.stringify(body || {}) : (((body:any):string) || "");
1000+
localVarRequestOptions.body = needsSerialization ? JSON.stringify(body != null ? body : {}) : (((body:any):string) || "");
10011001

10021002
return {
10031003
url: url.format(localVarUrlObj),
@@ -1027,7 +1027,7 @@ export const UserApiFetchParamCreator = function (configuration?: Configuration)
10271027
delete localVarUrlObj.search;
10281028
localVarRequestOptions.headers = Object.assign({}, localVarHeaderParameter, options.headers);
10291029
const needsSerialization = (typeof body !== "string") || localVarRequestOptions.headers['Content-Type'] === 'application/json';
1030-
localVarRequestOptions.body = needsSerialization ? JSON.stringify(body || {}) : (((body:any):string) || "");
1030+
localVarRequestOptions.body = needsSerialization ? JSON.stringify(body != null ? body : {}) : (((body:any):string) || "");
10311031

10321032
return {
10331033
url: url.format(localVarUrlObj),
@@ -1057,7 +1057,7 @@ export const UserApiFetchParamCreator = function (configuration?: Configuration)
10571057
delete localVarUrlObj.search;
10581058
localVarRequestOptions.headers = Object.assign({}, localVarHeaderParameter, options.headers);
10591059
const needsSerialization = (typeof body !== "string") || localVarRequestOptions.headers['Content-Type'] === 'application/json';
1060-
localVarRequestOptions.body = needsSerialization ? JSON.stringify(body || {}) : (((body:any):string) || "");
1060+
localVarRequestOptions.body = needsSerialization ? JSON.stringify(body != null ? body : {}) : (((body:any):string) || "");
10611061

10621062
return {
10631063
url: url.format(localVarUrlObj),
@@ -1206,7 +1206,7 @@ export const UserApiFetchParamCreator = function (configuration?: Configuration)
12061206
delete localVarUrlObj.search;
12071207
localVarRequestOptions.headers = Object.assign({}, localVarHeaderParameter, options.headers);
12081208
const needsSerialization = (typeof body !== "string") || localVarRequestOptions.headers['Content-Type'] === 'application/json';
1209-
localVarRequestOptions.body = needsSerialization ? JSON.stringify(body || {}) : (((body:any):string) || "");
1209+
localVarRequestOptions.body = needsSerialization ? JSON.stringify(body != null ? body : {}) : (((body:any):string) || "");
12101210

12111211
return {
12121212
url: url.format(localVarUrlObj),

0 commit comments

Comments
 (0)