Skip to content

fix the value for @Generated java annotation#4366

Merged
wing328 merged 1 commit intoswagger-api:masterfrom
trigan-d:patch-1
Dec 12, 2016
Merged

fix the value for @Generated java annotation#4366
wing328 merged 1 commit intoswagger-api:masterfrom
trigan-d:patch-1

Conversation

@trigan-d
Copy link
Copy Markdown
Contributor

com.my.Generator.class.toString() returns "class com.my.Generator", and this value is then used in the javax.annotation.Generated annotation like that: @generated(value = "class com.my.Generator").
Should use Generator.class.getName() instead, so we end up with @generated(value = "com.my.Generator")

com.my.Generator.class.toString() returns "class com.my.Generator", and this value is then used in the javax.annotation.Generated annotation like that: @generated(value = "class com.my.Generator").
Should use Generator.class.getName() instead, so we end up with @generated(value = "com.my.Generator")
@wing328 wing328 added this to the v2.2.2 milestone Dec 12, 2016
@wing328
Copy link
Copy Markdown
Contributor

wing328 commented Dec 12, 2016

@trigan-d The change looks good to me. Thanks for the PR to fix the issue.

@wing328 wing328 merged commit bf50ea1 into swagger-api:master Dec 12, 2016
@wing328
Copy link
Copy Markdown
Contributor

wing328 commented Dec 12, 2016

(If your company is using Swagger Codegen, please consider adding your company name to this list)

davidgri pushed a commit to davidgri/swagger-codegen that referenced this pull request May 11, 2017
com.my.Generator.class.toString() returns "class com.my.Generator", and this value is then used in the javax.annotation.Generated annotation like that: @generated(value = "class com.my.Generator").
Should use Generator.class.getName() instead, so we end up with @generated(value = "com.my.Generator")
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants