Added the FQCN to the @Size-annotation. #975
Conversation
There was a problem hiding this comment.
I think post-processing the generated models is unnecessary and it's merely a configuration issue.
After reporting this bug, I've dropped this project and used OpenAPI Generator directly to generate the respective models.
This is the working configuration:
annotationLibrary: "none"
asyncNative: true
dateLibrary: "java8"
disableMultipart: true
documentationProvider: "none"
generateBuilders: true
hideGenerationTimestamp: true
library: "microprofile"
microprofileMutiny: true
microprofileRestClientVersion: "3.0"
openApiNullable: false
serializationLibrary: "jackson"
useBeanValidation: true
useJakartaEe: true
useRuntimeException: true
The missing piece is probably microprofileRestClientVersion: "3.0" and potentially useJakartaEe: true.
https://github.com/OpenAPITools/openapi-generator/blob/v7.11.0/modules/openapi-generator/src/main/java/org/openapitools/codegen/languages/JavaClientCodegen.java#L289
|
@joschi @denvitaharen, then we can set/expose these configurations since it seems that the generator already sets the correct variables based on the set of properties. |
[0.9.x] SubjectThanks to @ricardozanini for pointing me in the right direction.
The solution is to post process the models and find the
@Sizeannotation and replace them with the FQCN instead.This fixed bug: #925
I took the update from #926 that @joschi made to reproduce the problem and used it to verify that the fix works. So that pull request can be closed when this is merge.
I tried to write an test to verify that the Size annotation is there, but since the Size annotation is not on the variable
listOfStringsbut rather on the type of the list, I can't really understand how I can check that.