Closed ReneZeidler closed 3 months ago
Thanks for the PR but your commit (as shown in the Commits tab) is not linked to your Github account, which means this PR won't count as your contribution in https://github.com/OpenAPITools/openapi-generator/graphs/contributors.
Let me know if you need help fixing it.
your commit (as shown in the Commits tab) is not linked to your Github account
Thanks for the note, I fixed the commit
tested locally to confirm the fix.
--- a/.openapi-generator/FILES
+++ b/.openapi-generator/FILES
@@ -7,7 +7,6 @@ build.gradle
build.sbt
docs/DefaultApi.md
docs/Limits.md
-docs/LimitsAllOfRefWithDescriptionAndReadonly.md
docs/NumberRange.md
git_push.sh
gradle.properties
@@ -38,5 +37,4 @@ src/main/java/org/openapitools/client/auth/HttpBasicAuth.java
src/main/java/org/openapitools/client/auth/HttpBearerAuth.java
src/main/java/org/openapitools/client/model/AbstractOpenApiSchema.java
src/main/java/org/openapitools/client/model/Limits.java
-src/main/java/org/openapitools/client/model/LimitsAllOfRefWithDescriptionAndReadonly.java
src/main/java/org/openapitools/client/model/NumberRange.java
@wing328 Hello, when will it be possible to get an update with this fix?
Fixes #15077
The previous fix for this in #16096 is incomplete because it still generates unnecessary inline models when readOnly or nullable is used in conjunction with other properties like description. This commit fixes the logic error and adds testcases.
Schema (full schema is included
issue_15077.yaml
):Previous output (excerpt from output using
typescript-angular
generator, but the issue is independent of any language-specific generator):The property
allOfRefWithDescriptionAndReadonly
generates an unnecessary inline schema that is a copy ofNumberRange
. The description is also missing from the property (and is instead applied to the generated inline model). ThereadOnly
property is completely ignored.Output with PR:
No unnecessary inline schema is created in all cases. The
allOfRefWithDescriptionAndReadonly
correctly has its description included as a comment and has thereadonly
modifier set.PR checklist
(For Windows users, please run the script in Git BASH) Commit all changed files. This is important, as CI jobs will verify all generator outputs of your HEAD commit as it would merge with master. These must match the expectations made by your contribution. You may regenerate an individual generator by passing the relevant config(s) as an argument to the script, for example
./bin/generate-samples.sh bin/configs/java*
. IMPORTANT: Do NOT purge/delete any folders/files (e.g. tests) when regenerating the samples as manually written tests may be removed.master
(upcoming 7.6.0 minor release - breaking changes with fallbacks),8.0.x
(breaking changes without fallbacks)