fix hashCode and equals for AdditionalProperties and generateAliasAsM…

[jpfinne]

Fix compilation error for hashcode #20795

(replace #20831 that have merge issues)

Equals also uses additionalProperties if present.

The fixes are for java feign/resttemplate/jersey2/jersey3/native/okhttp-gson/webclient and JavaSpring

@NilsDammenhayn
The generated code still uses the "new" way of 7.12.0

private Map<String, Object> additionalProperties;

instead of

extends HashMap<String, Object>

This PR only fixes the compilation error for resttemplate, webclient and feign

Microprofile still uses the extends HashMap

PR checklist

• Read the contribution guidelines.
• Pull Request title clearly describes the work in the pull request and Pull Request description provides details about how to validate the work. Missing information here may result in delayed response from the community.
• Run the following to build the project and update samples:

  ./mvnw clean package || exit
  ./bin/generate-samples.sh ./bin/configs/*.yaml || exit
  ./bin/utils/export_docs_generators.sh || exit
  

  (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.
• File the PR against the correct branch: master (upcoming 7.x.0 minor release - breaking changes with fallbacks), 8.0.x (breaking changes without fallbacks)
• If your PR is targeting a particular programming language, @mention the [technical committee]@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)

[martin-mfg]

What happened to the changes in modules/openapi-generator/src/main/resources/Java/pojo.mustache? It seems they got lost - or are they not needed?
Other than this the changes look good to me.

[jpfinne]

What happened to the changes in modules/openapi-generator/src/main/resources/Java/pojo.mustache? It seems they got lost - or are they not needed? Other than this the changes look good to me.

'{{>libraries/xxx/additional_properties}} are only defined in the pojo.mustache for the individual libraries.
So Map<String, {{{.}}}> additionalProperties is not generated by the default java/pojo. mustache

Thanks for the review

[jpfinne]

@martin-mfg @wing328 Is there anything that blocks the merge of this PR?

[martin-mfg]

@martin-mfg @wing328 Is there anything that blocks the merge of this PR?

Mainly the merge conflicts. Please resolve them when you have time. Other than this the PR looks good to me. So then we just need to wait for @wing328 to merge it.

[JoaoBrlt]

Hi @jpfinne! 👋 Thanks for the pull request! I recently also fixed how additional properties are handled by the restclient library in #20947 and the pull request has been merged. If you have time, could you also fix the pojo.mustache file for this library too? Otherwise, just tell me and I'll create a separate PR for that. Cheers!