Fix CodegenModel.allVars to include vars of composed and parent models #377
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
The
allVars
field inCodegenModel
is doesn't contain the composed model properties and only contains the inherited properties. This is because theallProperties
field passed duringaddVars
in ComposedSchema instance check offromModel
doesn't contain the properties of the composed model.This commit adds the code to add the composed model properties to
allProperties
variable, that is used to add the variables inallVars
variable. This is similar to how it is handled in the 2.0 DefaultCodegen.java file.Testing
model.mustache
handlebar to useallVars
inswagger_types
.Pet
class before the changes (pasting only the swagger_types part)Pet
class after the changesmvn surefire:test
passes.Environment
3.0.0
branch, 3.0.9-SNAPSHOT