jpfinne
jpfinne
@swpalmer-cl It works with a slightly different syntax Tested with 7.3.0 ```yaml BaseObject: type: object required: - id properties: id: type: string description: Object ID name: type: string description: Optional...
I've looked at the code. The order is decided in DefaultCodeGen.updateModelForComposedSchema It uses io.swagger.v3.oas.models.media.ComposedSchema to find the list of Properties and the list of AllOf. The issue is that ComposedSchema...
Using REFACTOR_ALLOF_WITH_PROPERTIES_ONLY, the properties are last. Nice. Of course it breaks all the generated constructors (lombok allArgsConstructor, readonly fields constructor for java client, required fields constructor for spring generator...). IMHO...
> there's already a normalizer rule for that: REFACTOR_ALLOF_WITH_PROPERTIES_ONLY > > https://github.com/openapitools/openapi-generator/blob/master/docs/customization.md#openapi-normalizer I don't think it works if REF_AS_PARENT_IN_ALLOF is enabled. I meant a new normalizer that runs ** before...
It seems the code on master has not the issue anymore. I'll do more tests later today
I reproduce on master using this test in DefaultCodegenTest using the same contract: ``` @Test public void testAllVars_issue_18340() { final OpenAPI openAPI = TestUtils.parseFlattenSpec("src/test/resources/3_0/spring/issue_18340.yaml"); // the bug happens with or...
Additional info. Moving the properties to the list of allOf fixes the issue: ``` Cat: allOf: - $ref: '#/components/schemas/Pet' - type: object properties: name: type: string ``` allVars = petType,name...
@Squeeeez The lombok generation has many issues: see #17793 What is your use case of `@lombok` annotations in generated code? What do you miss from the non `@lombok` code? Just...
@wing328 Sorry I'm new to the whole process. I've managed to install a portable version of git bash on my Windows laptop. And I can regenerate the samples. [JacksonTest on...
@wing328 thank you for your review. The latest commit uses a lambda for useJsonTypeName. I'm a bit scared by the amount of samples updated. There was some differences in rust...