-
Notifications
You must be signed in to change notification settings - Fork 6k
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
Ruby client [2.X] Added inheritance/polymorphism (allOf) to ruby client #11969
base: master
Are you sure you want to change the base?
Ruby client [2.X] Added inheritance/polymorphism (allOf) to ruby client #11969
Conversation
@Override | ||
protected void addImport(CodegenModel m, String type) { | ||
if (m.parent != null && m.parent.equals(type)) { | ||
super.addImport(m, type); | ||
} | ||
} | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Only adds the parent class for importing as the ruby client does not need to know the type of the values it works with, but does need to import the parent class
@Override | ||
public String toModelImport(String name) { | ||
return this.toModelFilename(name); | ||
} | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixes the import statement to use the file name rather than the model name as that is what Ruby uses to import the additional class
{{#allVars}} | ||
:'{{{name}}}' => :'{{{baseName}}}'{{#hasMore}},{{/hasMore}} | ||
{{/vars}} | ||
{{/allVars}} | ||
} | ||
end | ||
|
||
# Attribute type mapping. | ||
def self.swagger_types | ||
{ | ||
{{#vars}} | ||
{{#allVars}} | ||
:'{{{name}}}' => :'{{{datatype}}}'{{#hasMore}},{{/hasMore}} | ||
{{/vars}} | ||
{{/allVars}} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Due to the way the deserializer works in the api_client, the parent values must be present in swagger_types
and attribute_map
{{#parent}} | ||
|
||
# call parent's initialize | ||
super(attributes) | ||
{{/parent}} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The parent constructor is called with the values passed in to the constructor of the child
PR checklist
./bin/
to update Petstore sample so that CIs can verify the change. (For instance, only need to run./bin/{LANG}-petstore.sh
and./bin/security/{LANG}-petstore.sh
if updating the {LANG} (e.g. php, ruby, python, etc) code generator or {LANG} client's mustache templates). Windows batch files can be found in.\bin\windows\
.3.0.0
branch for changes related to OpenAPI spec 3.0. Default:master
.Description of the PR
This takes some of the changes made to OpenAPI v3.0 and expands on them to add inheritance/ polymorphism to Swagger 2.0