@ApiProperty 's operation is not correct on OpenApi, Swagger queries are not sending

Is there an existing issue for this?

• I have searched the existing issues

Minimum reproduction code

https://github.com/p29hieu-sub-projects/nestjs-swagger

Steps to reproduce

Proplem: When i use ApiProperty decorator from nestjs/swagger, i can’t use them on OpenApi.

I created a dto file like:

export class GetProductsDto implements IGetProductsDto {
 @ApiProperty({
   example: 37.7752315,
 })
 @IsNotEmpty()
 latitude: number;

 @ApiProperty({
   example: -122.418075,
 })
 @IsNotEmpty()
 longitude: number;
}

then, i used it:

 @ApiQuery({
      type: GetProductsDto,
    })
  @Get("products")
  async getProducts(@Query() query: GetProductsDto): Promise<any> {
    return {
      data: [],
    };
  }

In my browser:

When i press Try it out and Excecute buttons:

Input fields have been filled but the error still occurs.

When i use required: false in ApiProperty

export class GetProductsDto implements IGetProductsDto {
 @ApiProperty({
   example: 37.7752315,
   required: false
 })
 @IsNotEmpty()
 latitude: number;

 @ApiProperty({
   example: -122.418075,
   required: false
 })
 @IsNotEmpty()
 longitude: number;
}

Then I press Try it out and Excecute buttons again, In Curl, the params is not true:

Please give me some help. Thanks!

  @nestjs/cli: "^8.0.0",
  OS: ubuntu 20.04
 Browser: Microsoft Edge Version 94.0.992.38 

Package version

^5.0.9

NestJS version

^8.0.0

Node.js version

14.17.2

In which operating systems have you tested?

• macOS
• Windows
• Linux

Other

No response

[kamilmysliwiec]

Please, report this issue in the corresponding repository https://github.com/swagger-api/swagger-ui

[mathis-m]

Workarround, put this in your main.ts it will make the parameters distinct:

const document = SwaggerModule.createDocument(app, options);
const distinctParams : (arr: any) => (ParameterObject | ReferenceObject)[] = 
  (arr) => [...new Map(arr.map(item => [item.in + item.name + item.$ref, item])).values()] as (ParameterObject | ReferenceObject)[]
for (const path in document.paths || {}) {
  const pathObj = document.paths[path]
  const methodParams = [
    {key: "get", params: pathObj.get?.parameters},
    {key: "put", params: pathObj.put?.parameters},
    {key: "post", params: pathObj.post?.parameters},
    {key: "delete", params: pathObj.delete?.parameters},
    {key: "options", params: pathObj.options?.parameters},
    {key: "head", params: pathObj.head?.parameters},
    {key: "patch", params: pathObj.patch?.parameters},
    {key: "trace", params: pathObj.trace?.parameters},
  ]
  if(pathObj.parameters) {
    document.paths[path].parameters = distinctParams(document.paths[path].parameters)
  }
  methodParams.forEach((x) => {
    if(x.params) {
      document.paths[path][x.key].parameters = distinctParams(x.params)
    }
  })
}

SwaggerModule.setup('/docs', app, document);

_Originally posted by @mathis-m in https://github.com/swagger-api/swagger-ui/issues/7539#issuecomment-938268189_