Fix subResourcePath when using tags in java-jersey (!215) · Merge requests · OpenAPI Tools / openapi-generator

Merged Administrator requested to merge github/fork/tht13/java-jersey-tags-path-issue into master Jun 04, 2018

Created by: tht13

PR checklist

Read the contribution guidelines.
Ran the shell script under ./bin/ to update Petstore sample so that CIs can verify the change. (For instance, only need to run ./bin/{LANG}-petstore.sh and ./bin/security/{LANG}-petstore.sh if updating the {LANG} (e.g. php, ruby, python, etc) code generator or {LANG} client's mustache templates). Windows batch files can be found in .\bin\windows\.
Filed the PR against the correct branch: Default: master.
Copied the technical committee to review the pull request if your PR is targeting a particular programming language.

Description of the PR

I am porting this PR over from swagger-codegen

The java jersey useTags option added in #6278 breaks functionality for subResourcePaths (@PATH is only added to the class route, not each method), this causes conflicts on startup as swagger-core assumes every method in the class is using the same path, and flags "conflicting methods" when all other annotations are identical (ie http type and @produces and @consumes).

View https://github.com/OpenAPITools/openapi-generator/pull/215/commits/bc26d7305ae9d589c2f39b46af5d22f44baa97dc#diff-774426b5ae03223547c129bb2cc6bed4R38 for an example of the missing @PATH annotations

@wing328