Skip to content

fix: [OpenAPI] Api client method with oneOf primitive param stays simplified in OpenAPI generator 7.22.0#1179

Open
CharlesDuboisSAP wants to merge 5 commits into
mainfrom
fix-one-of-7.22
Open

fix: [OpenAPI] Api client method with oneOf primitive param stays simplified in OpenAPI generator 7.22.0#1179
CharlesDuboisSAP wants to merge 5 commits into
mainfrom
fix-one-of-7.22

Conversation

@CharlesDuboisSAP
Copy link
Copy Markdown
Contributor

@CharlesDuboisSAP CharlesDuboisSAP commented May 22, 2026

Context

The OpenAPI generator 7.22.0 broke generated api classes.

Feature scope:

  • Added a reproducible unit test: DataModelGeneratorIntegrationTest.integrationTests() for enum API_CLASS_VENDOR_EXTENSION_YAML.
  • fix in GenerationConfigurationConverter.preprocessSpecification() (has been commented out)

Definition of Done

  • Functionality scope stated & covered
  • Tests cover the scope above
  • Error handling created / updated & covered by the tests above
  • Documentation updated
  • Release notes updated

@CharlesDuboisSAP CharlesDuboisSAP self-assigned this May 22, 2026
@CharlesDuboisSAP CharlesDuboisSAP added the please review Request to review a pull request label May 22, 2026
Copy link
Copy Markdown
Contributor Author

@CharlesDuboisSAP CharlesDuboisSAP left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A better solution could be found by finding what changed from the openAPI generator 7.21.0 to 7.22.0. Especially in the org.openapitools.codegen classes containing usages of getParameter(). You can debug the existing modified unit test of this PR to explore the suspected OpenAPI classes.

@CharlesDuboisSAP CharlesDuboisSAP added the do not merge Pull request must not be merged label May 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do not merge Pull request must not be merged please review Request to review a pull request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant