diff --git a/src/generators/clean-arch.generator.ts b/src/generators/clean-arch.generator.ts index aa71354..78b8dc9 100644 --- a/src/generators/clean-arch.generator.ts +++ b/src/generators/clean-arch.generator.ts @@ -271,7 +271,11 @@ export function generateCleanArchitecture( let returnType = 'void'; let returnBaseType = 'void'; let isListContainer = false; - const responseSchema = op.responses?.['200']?.content?.['application/json']?.schema; + const successCode = ['200', '201', '202', '203'].find((code) => op.responses?.[code]); + const responseSchema = + successCode !== undefined + ? op.responses?.[successCode]?.content?.['application/json']?.schema + : undefined; if (responseSchema) { if (responseSchema.$ref) { returnType = responseSchema.$ref.split('/').pop() || 'unknown'; @@ -283,6 +287,9 @@ export function generateCleanArchitecture( } } + const hasQueryParams = (op.parameters || []).some((p) => p.in === 'query'); + const hasBodyParam = !!op.requestBody; + tagsMap[tag].push({ nickname: op.operationId || `${method}${pathKey.replace(/\//g, '_')}`, summary: op.summary || '', @@ -294,15 +301,17 @@ export function generateCleanArchitecture( ...p, '-last': i === allParams.length - 1 })), - hasQueryParams: (op.parameters || []).some((p) => p.in === 'query'), + hasQueryParams, queryParams: (op.parameters || []) .filter((p) => p.in === 'query') .map((p, i: number, arr: unknown[]) => ({ paramName: p.name, '-last': i === arr.length - 1 })), - hasBodyParam: !!op.requestBody, + hasBodyParam, bodyParam: 'body', + hasOptions: hasQueryParams || hasBodyParam, + hasBothParamsAndBody: hasQueryParams && hasBodyParam, returnType: returnType !== 'void' ? returnType : false, returnBaseType: returnBaseType !== 'void' ? returnBaseType : false, returnTypeVarName: returnType !== 'void' ? toCamelCase(returnType) : false, diff --git a/src/types/openapi.types.ts b/src/types/openapi.types.ts index 1791429..0aa3a8f 100644 --- a/src/types/openapi.types.ts +++ b/src/types/openapi.types.ts @@ -115,6 +115,8 @@ export interface TagOperation { queryParams: unknown[]; hasBodyParam: boolean; bodyParam: string; + hasOptions: boolean; + hasBothParamsAndBody: boolean; returnType: string | boolean; returnBaseType: string | boolean; returnTypeVarName: string | boolean; diff --git a/templates/api.repository.impl.mustache b/templates/api.repository.impl.mustache index e40cb8a..a466595 100644 --- a/templates/api.repository.impl.mustache +++ b/templates/api.repository.impl.mustache @@ -32,26 +32,20 @@ export class {{classname}}RepositoryImpl extends MRepository implements {{classn {{#operation}} {{nickname}}({{#allParams}}{{paramName}}{{^required}}?{{/required}}: {{{dataType}}}{{^-last}}, {{/-last}}{{/allParams}}): Observable<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}void{{/returnType}}> { {{#isListContainer}} - return this.{{httpMethod}}<{{{returnBaseType}}}Dto>('{{path}}'{{#hasQueryParams}}, { - params: { {{#queryParams}}{{paramName}}{{^-last}}, {{/-last}}{{/queryParams}} } - }{{/hasQueryParams}}{{#hasBodyParam}}, {{bodyParam}}{{/hasBodyParam}}) + return this.{{httpMethod}}<{{{returnBaseType}}}Dto>('{{path}}'{{#hasOptions}}, { {{#hasQueryParams}}params: { {{#queryParams}}{{paramName}}{{^-last}}, {{/-last}}{{/queryParams}} }{{/hasQueryParams}}{{#hasBothParamsAndBody}}, {{/hasBothParamsAndBody}}{{#hasBodyParam}}body{{/hasBodyParam}} }{{/hasOptions}}) .pipe( map((response) => response.{{#vendorExtensions}}{{x-response-property}}{{/vendorExtensions}}{{^vendorExtensions}}items{{/vendorExtensions}}.map({{{returnBaseTypeVarName}}}Mapper)) ); {{/isListContainer}} {{^isListContainer}} {{#returnType}} - return this.{{httpMethod}}<{{{returnType}}}Dto>('{{path}}'{{#hasQueryParams}}, { - params: { {{#queryParams}}{{paramName}}{{^-last}}, {{/-last}}{{/queryParams}} } - }{{/hasQueryParams}}{{#hasBodyParam}}, {{bodyParam}}{{/hasBodyParam}}) + return this.{{httpMethod}}<{{{returnType}}}Dto>('{{path}}'{{#hasOptions}}, { {{#hasQueryParams}}params: { {{#queryParams}}{{paramName}}{{^-last}}, {{/-last}}{{/queryParams}} }{{/hasQueryParams}}{{#hasBothParamsAndBody}}, {{/hasBothParamsAndBody}}{{#hasBodyParam}}body{{/hasBodyParam}} }{{/hasOptions}}) .pipe( map({{{returnTypeVarName}}}Mapper) ); {{/returnType}} {{^returnType}} - return this.{{httpMethod}}('{{path}}'{{#hasQueryParams}}, { - params: { {{#queryParams}}{{paramName}}{{^-last}}, {{/-last}}{{/queryParams}} } - }{{/hasQueryParams}}{{#hasBodyParam}}, {{bodyParam}}{{/hasBodyParam}}); + return this.{{httpMethod}}('{{path}}'{{#hasOptions}}, { {{#hasQueryParams}}params: { {{#queryParams}}{{paramName}}{{^-last}}, {{/-last}}{{/queryParams}} }{{/hasQueryParams}}{{#hasBothParamsAndBody}}, {{/hasBothParamsAndBody}}{{#hasBodyParam}}body{{/hasBodyParam}} }{{/hasOptions}}); {{/returnType}} {{/isListContainer}} }