[BUG][Spring] generated api interface doesn't have @RequestParam annotation

[t2y]

Bug Report Checklist

• Have you provided a full/minimal spec to reproduce the issue?
• Have you validated the input using an OpenAPI validator (example)?
• What's the version of OpenAPI Generator used?
• Have you search for related issues/PRs?
• What's the actual output vs expected output?
• [Optional] Bounty to sponsor the fix (example)

Description

I updated openapi-generator from 4.0.1 to 4.0.2. Then, I found generated api interface from same spec is different between 4.0.1 and 4.0.2.

For some reason, @RequestParam is not generated for api parameters. Here is an example.

• 4.0.1

@ApiParam(value = "", allowableValues = "json, csv, tsv", defaultValue = "json")
@Valid
@RequestParam(value = "format", required = false, defaultValue = "json")
Format format

• 4.0.2

@ApiParam(value = "", allowableValues = "json, csv, tsv", defaultValue = "json")
@Valid
Format format

openapi-generator version

4.0.2

OpenAPI declaration file content or url

openapi: 3.0.2
info:
  title: info
  description: info
  version: 0.1.0

paths:
  /example/api:
    get:
      summary: summary
      description: description
      parameters:
      - $ref: '#/components/parameters/requiredQueryParam'
      - $ref: '#/components/parameters/formatParam'
      responses:
        200:
          description: response
          content:
            application/json:
              schema:
                type: string

components:
  parameters:
    requiredQueryParam:
      description: set query
      in: query
      name: query
      required: true
      schema:
        type: string
    formatParam:
      description: set format
      in: query
      name: format
      required: false
      schema:
        $ref: '#/components/schemas/format'

  schemas:
    format:
      default: json
      description: response format
      enum:
      - json
      - csv
      type: string

Command line used for generation

$ mvn clean package
$ java -jar modules/openapi-generator-cli/target/openapi-generator-cli.jar generate -i ./simple-example-spec/openapi.spec -g spring -o output

Steps to reproduce

Generate api code for spring with 4.0.1 and 4.0.2. Then compare the api parameters of api interface.

Related issues/PRs

I couldn't find any similar issues.

Suggest a fix

[t2y]

I found some parameters have @RequestParam and some parameters don't.

[macjohnny]

@t2y can you give an example spec and point to the parameters that generate @RequestParam and those that dont?

[t2y]

@macjohnny Now I made a test environment to run openapi-generator cli for git bisect. I will investigate which revision affects this issue in this weekend.

Currently, my production spec is complicated, so I will try to create simple example spec if possible.

[t2y]

I confirmed 4.0.3 still has this issue.

[t2y]

I did git bisect. This revision is the first bad commit.

• 146c1fb
• [BUG] Query Parameter Serialization (Spring) #2655

[t2y]

@macjohnny I updated openapi.yaml in above description to be able to reproduce this issue.

In my environment, generated api interface has diff between 0d701b7 and 146c1fb as below. The format parameter doesn't have @RequestParam annotation. Could you confirm?

$ diff -u s3-good/src/main/java/org/openapitools/api/ExampleApi.java s3-bad/src/main/java/org/openapitools/api/ExampleApi.java
--- s3-good/src/main/java/org/openapitools/api/ExampleApi.java	2019-07-13 14:46:47.000000000 +0900
+++ s3-bad/src/main/java/org/openapitools/api/ExampleApi.java	2019-07-13 14:49:11.000000000 +0900
@@ -26,7 +26,7 @@
 import java.util.List;
 import java.util.Map;
 import java.util.Optional;
-@javax.annotation.Generated(value = "org.openapitools.codegen.languages.SpringCodegen", date = "2019-07-13T14:46:46.993+09:00[Asia/Tokyo]")
+@javax.annotation.Generated(value = "org.openapitools.codegen.languages.SpringCodegen", date = "2019-07-13T14:49:11.556+09:00[Asia/Tokyo]")

 @Validated
 @Api(value = "example", description = "the example API")
@@ -42,7 +42,7 @@
     @RequestMapping(value = "/example/api",
         produces = { "application/json" },
         method = RequestMethod.GET)
-    default ResponseEntity<String> exampleApiGet(@NotNull @ApiParam(value = "set query", required = true) @Valid @RequestParam(value = "query", required = true) String query,@ApiParam(value = "set format", allowableValues = "json, csv", defaultValue = "json") @Valid @RequestParam(value = "format", required = false, defaultValue="json") Format format) {
+    default ResponseEntity<String> exampleApiGet(@NotNull @ApiParam(value = "set query", required = true) @Valid @RequestParam(value = "query", required = true) String query,@ApiParam(value = "set format", allowableValues = "json, csv", defaultValue = "json") @Valid Format format) {
         return new ResponseEntity<>(HttpStatus.NOT_IMPLEMENTED);

     }

[macjohnny]

@t2y thanks for the analysis
a workaround for your problem could be to inline the parameter, i.e.

components:
    formatParam:
      description: set format
      in: query
      name: format
      required: false
      default: json
      description: response format
      enum:
        - json
        - csv
      type: string

I guess either components.schemas should not be used for query params or

openapi-generator/modules/openapi-generator/src/main/java/org/openapitools/codegen/DefaultCodegen.java

Line 2213 in 146c1fb

property.isModel = ModelUtils.isModel(p);

should be extended to also check that the type is object
@wing328 @jimschubert @jmini what is your opinion?

[t2y]

remind, @wing328 @jimschubert @jmini what do you think about above solution?

[jimschubert]

@t2y thanks for pinging. I'm very far behind on emails and didn't see the last mention in the thread.

It looks like property.isModel assignment could add an && !ModelUtils.isEnum(p)? This function doesn't exist, but may be necessary for this case

[manderson23]

I also see missing @RequestParam with date-time format.

- name: start
  in: query
  schema:
     type: string
     format: date-time
  required: true
  description: The start time.

If I delete format to have the parameter a String then @RequestParam is present.

[t2y]

@macjohnny @jimschubert This bug is critical for me, and block upgrading version. Do you have a plan to fix?

[macjohnny]

@t2y would you like to submit a PR with a fix?

[t2y]

@macjohnny I don't know the background of this issue. But, I can try to fix and create a PR, so I will do it next week if no one will fix.

[macjohnny]

This would be very appreciated