Skip to content

Commit a0bd0ee

Browse files
committed
Issue #2449 SubClass annotations are missing from the base class
1 parent 73c567f commit a0bd0ee

File tree

9 files changed

+48
-5
lines changed

9 files changed

+48
-5
lines changed

modules/swagger-codegen/src/main/java/io/swagger/codegen/CodegenModel.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,7 @@ public class CodegenModel {
1717
// References to parent and interface CodegenModels. Only set when code generator supports inheritance.
1818
public CodegenModel parentModel;
1919
public List<CodegenModel> interfaceModels;
20+
public List<CodegenModel> children;
2021

2122
public String name, classname, title, description, classVarName, modelJson, dataType;
2223
public String classFilename; // store the class file name, mainly used for import

modules/swagger-codegen/src/main/java/io/swagger/codegen/DefaultCodegen.java

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -180,6 +180,17 @@ public Map<String, Object> postProcessAllModels(Map<String, Object> objs) {
180180
}
181181
}
182182
}
183+
// Let parent know about all its children
184+
for (String name : allModels.keySet()) {
185+
CodegenModel cm = allModels.get(name);
186+
if (cm.parent == null) continue;
187+
CodegenModel parent = allModels.get(cm.parent);
188+
if (parent == null) continue;
189+
if (parent.children == null) {
190+
parent.children = new ArrayList<CodegenModel>();
191+
}
192+
parent.children.add(cm);
193+
}
183194
}
184195
return objs;
185196
}

modules/swagger-codegen/src/main/java/io/swagger/codegen/languages/AbstractJavaCodegen.java

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -253,6 +253,8 @@ public void processOpts() {
253253
importMapping.put("ApiModelProperty", "io.swagger.annotations.ApiModelProperty");
254254
importMapping.put("ApiModel", "io.swagger.annotations.ApiModel");
255255
importMapping.put("JsonProperty", "com.fasterxml.jackson.annotation.JsonProperty");
256+
importMapping.put("JsonSubTypes", "com.fasterxml.jackson.annotation.JsonSubTypes");
257+
importMapping.put("JsonTypeInfo", "com.fasterxml.jackson.annotation.JsonTypeInfo");
256258
importMapping.put("JsonCreator", "com.fasterxml.jackson.annotation.JsonCreator");
257259
importMapping.put("JsonValue", "com.fasterxml.jackson.annotation.JsonValue");
258260
importMapping.put("SerializedName", "com.google.gson.annotations.SerializedName");
@@ -628,6 +630,10 @@ public CodegenModel fromModel(String name, Model model, Map<String, Model> allDe
628630
if(codegenModel.description != null) {
629631
codegenModel.imports.add("ApiModel");
630632
}
633+
if (codegenModel.discriminator != null && additionalProperties.containsKey("jackson")) {
634+
codegenModel.imports.add("JsonSubTypes");
635+
codegenModel.imports.add("JsonTypeInfo");
636+
}
631637
if (allDefinitions != null && codegenModel.parentSchema != null && codegenModel.hasEnums) {
632638
final Model parentModel = allDefinitions.get(codegenModel.parentSchema);
633639
final CodegenModel parentCodegenModel = super.fromModel(codegenModel.parent, parentModel);

modules/swagger-codegen/src/main/resources/Java/pojo.mustache

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
* {{#description}}{{.}}{{/description}}{{^description}}{{classname}}{{/description}}
33
*/{{#description}}
44
@ApiModel(description = "{{{description}}}"){{/description}}
5-
{{>generatedAnnotation}}
5+
{{>generatedAnnotation}}{{#discriminator}}{{>typeInfoAnnotation}}{{/discriminator}}
66
public class {{classname}} {{#parent}}extends {{{parent}}} {{/parent}}{{#parcelableModel}}implements Parcelable {{#serializableModel}}, Serializable {{/serializableModel}}{{/parcelableModel}}{{^parcelableModel}}{{#serializableModel}}implements Serializable {{/serializableModel}}{{/parcelableModel}}{
77
{{#vars}}
88
{{#isEnum}}
Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
{{#jackson}}
2+
@JsonTypeInfo(use = JsonTypeInfo.Id.NAME, include = JsonTypeInfo.As.PROPERTY, property = "{{discriminator}}" )
3+
@JsonSubTypes({
4+
{{#children}}@JsonSubTypes.Type(value = {{name}}.class, name = "{{name}}"),{{/children}}
5+
}){{/jackson}}

samples/client/petstore/java/feign/src/main/java/io/swagger/client/model/Animal.java

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,13 +28,18 @@
2828
import java.util.Objects;
2929
import com.fasterxml.jackson.annotation.JsonProperty;
3030
import com.fasterxml.jackson.annotation.JsonCreator;
31+
import com.fasterxml.jackson.annotation.JsonSubTypes;
32+
import com.fasterxml.jackson.annotation.JsonTypeInfo;
3133
import io.swagger.annotations.ApiModel;
3234
import io.swagger.annotations.ApiModelProperty;
3335

3436
/**
3537
* Animal
3638
*/
37-
39+
@JsonTypeInfo(use = JsonTypeInfo.Id.NAME, include = JsonTypeInfo.As.PROPERTY, property = "className" )
40+
@JsonSubTypes({
41+
@JsonSubTypes.Type(value = Dog.class, name = "Dog"),@JsonSubTypes.Type(value = Cat.class, name = "Cat"),
42+
})
3843
public class Animal {
3944
@JsonProperty("className")
4045
private String className = null;

samples/client/petstore/java/jersey1/src/main/java/io/swagger/client/model/Animal.java

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,13 +28,18 @@
2828
import java.util.Objects;
2929
import com.fasterxml.jackson.annotation.JsonProperty;
3030
import com.fasterxml.jackson.annotation.JsonCreator;
31+
import com.fasterxml.jackson.annotation.JsonSubTypes;
32+
import com.fasterxml.jackson.annotation.JsonTypeInfo;
3133
import io.swagger.annotations.ApiModel;
3234
import io.swagger.annotations.ApiModelProperty;
3335

3436
/**
3537
* Animal
3638
*/
37-
39+
@JsonTypeInfo(use = JsonTypeInfo.Id.NAME, include = JsonTypeInfo.As.PROPERTY, property = "className" )
40+
@JsonSubTypes({
41+
@JsonSubTypes.Type(value = Dog.class, name = "Dog"),@JsonSubTypes.Type(value = Cat.class, name = "Cat"),
42+
})
3843
public class Animal {
3944
@JsonProperty("className")
4045
private String className = null;

samples/client/petstore/java/jersey2-java8/src/main/java/io/swagger/client/model/Animal.java

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,13 +28,18 @@
2828
import java.util.Objects;
2929
import com.fasterxml.jackson.annotation.JsonProperty;
3030
import com.fasterxml.jackson.annotation.JsonCreator;
31+
import com.fasterxml.jackson.annotation.JsonSubTypes;
32+
import com.fasterxml.jackson.annotation.JsonTypeInfo;
3133
import io.swagger.annotations.ApiModel;
3234
import io.swagger.annotations.ApiModelProperty;
3335

3436
/**
3537
* Animal
3638
*/
37-
39+
@JsonTypeInfo(use = JsonTypeInfo.Id.NAME, include = JsonTypeInfo.As.PROPERTY, property = "className" )
40+
@JsonSubTypes({
41+
@JsonSubTypes.Type(value = Dog.class, name = "Dog"),@JsonSubTypes.Type(value = Cat.class, name = "Cat"),
42+
})
3843
public class Animal {
3944
@JsonProperty("className")
4045
private String className = null;

samples/client/petstore/java/jersey2/src/main/java/io/swagger/client/model/Animal.java

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,13 +28,18 @@
2828
import java.util.Objects;
2929
import com.fasterxml.jackson.annotation.JsonProperty;
3030
import com.fasterxml.jackson.annotation.JsonCreator;
31+
import com.fasterxml.jackson.annotation.JsonSubTypes;
32+
import com.fasterxml.jackson.annotation.JsonTypeInfo;
3133
import io.swagger.annotations.ApiModel;
3234
import io.swagger.annotations.ApiModelProperty;
3335

3436
/**
3537
* Animal
3638
*/
37-
39+
@JsonTypeInfo(use = JsonTypeInfo.Id.NAME, include = JsonTypeInfo.As.PROPERTY, property = "className" )
40+
@JsonSubTypes({
41+
@JsonSubTypes.Type(value = Dog.class, name = "Dog"),@JsonSubTypes.Type(value = Cat.class, name = "Cat"),
42+
})
3843
public class Animal {
3944
@JsonProperty("className")
4045
private String className = null;

0 commit comments

Comments
 (0)