-
-
Notifications
You must be signed in to change notification settings - Fork 6.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[java] implement #18032 add builder pattern to java client and spring generator #18650
Conversation
thanks for the PR. let's give it a try |
…t and spring generator (OpenAPITools#18650) * add builder pattern to java client and spring generator * regenerate samples * update doc --------- Co-authored-by: jpfinne <jeanpaul@finne.be>
@wing328 The builder option is missing from the 7.6.0 release notes. |
just updated it to highlight this change as well. (the release notes are only the highlight, not the full list of changes) |
@wing328 is this available in |
@kevin1chun I'm the main contributor of this feature. Feedbacks are welcome |
Thanks @jpfinne |
Hi How would you pass the option in the command below ? Both do not produce builder code.
|
You need to add a space after additional-properties:
Even if the code is in AbstractJavaCodeGen, not all libraries are tested with the bulder.
If you want to use the builder in the default java library, you need to take a copy of https://github.com/OpenAPITools/openapi-generator/blob/master/modules/openapi-generator/src/main/resources/Java/libraries/okhttp-gson/pojo.mustache This can be done with the template customization. See https://openapi-generator.tech/docs/templating/ |
based on #18648
Fix #18032. Add a new configOptions: generateBuilders to java client and spring generators
The name of the option is the same as in JavaJAXSRSSpecServerCodeGen.
Spring + lombok: generateBuilders + generateConstructorWithAllArgs configOptions can be used instead of lombok.
The builder is not created when lombok.Data is configured (issue with fluent setters not generated)
Improve generation of no args constructor.
Also minor fix for the generateConstructorWithAllArgs and Lombok.AllArgsConstructor (there was some typos)
PR checklist
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.1.0 minor release - breaking changes with fallbacks),8.0.x
(breaking changes without fallbacks)cc @bbdouglas (2017/07) @sreeshas (2017/08) @jfiala (2017/08) @lukoyanov (2017/09) @cbornet (2017/09) @jeff9finger (2018/01) @karismann (2019/03) @Zomzog (2019/04) @lwlee2608 (2019/10) @martin-mfg (2023/08)
Java Spring | @cachescrubber (2022/02) @welshm (2022/02) @MelleD (2022/02) @atextor (2022/02) @manedev79 (2022/02) @javisst (2022/02) @borsch (2022/02) @banlevente (2022/02) @Zomzog (2022/09) @martin-mfg (2023/08)