Skip to content
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

[BUG] [JAVA][MVN] 6.3.0 Generation of arrays with elements containing anyOf is broken #14708

Closed
5 of 6 tasks
mvanglabbeek opened this issue Feb 15, 2023 · 2 comments · Fixed by #14715
Closed
5 of 6 tasks

Comments

@mvanglabbeek
Copy link

Bug Report Checklist

  • Have you provided a full/minimal spec to reproduce the issue?
  • Have you validated the input using an OpenAPI validator (example)?
  • Have you tested with the latest master to confirm the issue still exists?
  • Have you searched for related issues/PRs?
  • What's the actual output vs expected output?
  • [Optional] Sponsorship to speed up the bug fix or feature request (example)
Description

When generating code using openapi-generator-maven-plugin 6.3.0 with an array containing an object having anyOf (see provided example), the generated code is broken:

joiner.add(getArrayItems().get(i).toUrlQueryString(String.format("%sarrayItems%s%s", prefix, suffix,
          "".equals(suffix) ? "" : String.format("%s%d%s", containerPrefix, i, containerSuffix))));

Cannot resolve method 'toUrlQueryString' in 'ArrayItem'

Without the anyOf in the OpenAPI yaml file, the code is correct.

openapi-generator version

Version 6.3.0, it worked fine in 6.2.1

OpenAPI declaration file content or url

https://gist.github.com/mvanglabbeek/30951b64448e74f8a8182e2f0b57e995

Generation Details

Via, openapi-generator-maven-plugin see https://gist.github.com/mvanglabbeek/30951b64448e74f8a8182e2f0b57e995

Steps to reproduce

Generate the code using the openapi-generator, using mvn 3.8.1

Related issues/PRs

#14178

Suggest a fix
@wing328
Copy link
Member

wing328 commented Feb 16, 2023

fixed via #14715

please pull the latest master to give it a try

@mvanglabbeek
Copy link
Author

Yeah, it's fixed. Thanks for the quick action!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants