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][openapi-generator-gradle-plugin] Property value not used #5935

Closed
6 tasks
kierans opened this issue Apr 15, 2020 · 0 comments · Fixed by #5936
Closed
6 tasks

[BUG][openapi-generator-gradle-plugin] Property value not used #5935

kierans opened this issue Apr 15, 2020 · 0 comments · Fixed by #5936

Comments

@kierans
Copy link
Contributor

kierans commented Apr 15, 2020

Bug Report Checklist

  • Have you provided a full/minimal spec to reproduce the issue?
  • Have you validated the input using an OpenAPI validator (example)?
  • What's the version of OpenAPI Generator used?
  • Have you search for related issues/PRs?
  • What's the actual output vs expected output?
  • [Optional] Bounty to sponsor the fix (example)
Description

When using Gradle Properties in strings the value of the property isn't used eg:

Successfully generated code to property(class java.lang.String, property(class java.lang.String, fixed(class java.lang.String, /Users/kieran/Projects/project/build/generated/openapi)))
openapi-generator version
plugins {
    id 'org.openapi.generator' version '4.3.0'
}
Command line used for generation
$ ./gradlew openApiGenerate
Steps to reproduce
  1. Create a fresh Gradle project following the README
  2. Pick an API spec. If none handy, the Petstore will do
  3. Run command to generate code
  4. Read console output
Related issues/PRs

None that I can find

Suggest a fix

Where properties are used in strings, or in console output, the value should be get() first

@kierans kierans changed the title [BUG][openapi-generator-gradle-plugin] Description [BUG][openapi-generator-gradle-plugin] Property value not used Apr 16, 2020
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.

1 participant