Bug Report Checklist
Description
Vertx 5 support was added in #23563. However, the supportVertxFuture option is incompatible with the useVertx5 option.
The generated code will fail to compile because vertx changed its interfaces in the bump from 4 to 5. Promise no longer extends Handler, which means we can no longer directly pass a promise into a param that expects a handler.
openapi-generator version
7.22.0
OpenAPI declaration file content or url
This sample is basically just the vertx5 sample but with the addition of the supportVertxFuture option. It will generate code that fails to compile.
Generation Details
After generating the sample code, attempting to compile the generate code with mvn clean install will reproduce the problem.
Steps to reproduce
- Add the
supportVertxFuture option to the vertx5 sample config (like [this](vertx5 sample))
- Run
generate-samples.sh
- Go to the generated sample code.
- Run the command
mvn clean install
- Generated code will fail to compile because
Promise no longer extends Handler
Related issues/PRs
None found
Suggest a fix
We need to update vertx/api.mustache to no longer directly pass the promise into the handler param.
Bug Report Checklist
Description
Vertx 5 support was added in #23563. However, the
supportVertxFutureoption is incompatible with theuseVertx5option.The generated code will fail to compile because vertx changed its interfaces in the bump from 4 to 5.
Promiseno longer extendsHandler, which means we can no longer directly pass apromiseinto a param that expects ahandler.openapi-generator version
7.22.0
OpenAPI declaration file content or url
This sample is basically just the vertx5 sample but with the addition of the
supportVertxFutureoption. It will generate code that fails to compile.Generation Details
After generating the sample code, attempting to compile the generate code with
mvn clean installwill reproduce the problem.Steps to reproduce
supportVertxFutureoption to the vertx5 sample config (like [this](vertx5 sample))generate-samples.shmvn clean installPromiseno longer extendsHandlerRelated issues/PRs
None found
Suggest a fix
We need to update vertx/api.mustache to no longer directly pass the promise into the handler param.