Skip to content

Commit e82c12c

Browse files
committed
updated tests, added parentSchema
1 parent 5a956df commit e82c12c

File tree

4 files changed

+8
-7
lines changed

4 files changed

+8
-7
lines changed

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
import java.util.*;
66

77
public class CodegenModel {
8-
public String parent;
8+
public String parent, parentSchema;
99
public String name, classname, description, classVarName, modelJson, dataType;
1010
public String unescapedDescription;
1111
public String defaultValue;

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

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -789,10 +789,11 @@ public CodegenModel fromModel(String name, Model model, Map<String, Model> allDe
789789
final RefModel parent = (RefModel) composed.getParent();
790790
if (parent != null) {
791791
final String parentRef = parent.getSimpleRef();
792-
m.parent = parentRef;
793-
addImport(m, toModelName(parent.getSimpleRef()));
792+
m.parentSchema = parentRef;
793+
m.parent = toModelName(parent.getSimpleRef());
794+
addImport(m, m.parent);
794795
if (!supportsInheritance && allDefinitions != null) {
795-
final Model parentModel = allDefinitions.get(parentRef);
796+
final Model parentModel = allDefinitions.get(m.parentSchema);
796797
if (parentModel instanceof ModelImpl) {
797798
final ModelImpl _parent = (ModelImpl) parentModel;
798799
if (_parent.getProperties() != null) {

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

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -455,8 +455,8 @@ public String toOperationId(String operationId) {
455455
public CodegenModel fromModel(String name, Model model, Map<String, Model> allDefinitions) {
456456
CodegenModel codegenModel = super.fromModel(name, model, allDefinitions);
457457

458-
if (allDefinitions != null && codegenModel != null && codegenModel.parent != null && codegenModel.hasEnums) {
459-
final Model parentModel = allDefinitions.get(codegenModel.parent);
458+
if (allDefinitions != null && codegenModel != null && codegenModel.parentSchema != null && codegenModel.hasEnums) {
459+
final Model parentModel = allDefinitions.get(codegenModel.parentSchema);
460460
final CodegenModel parentCodegenModel = super.fromModel(codegenModel.parent, parentModel);
461461
codegenModel = this.reconcileInlineEnums(codegenModel, parentCodegenModel);
462462
}

modules/swagger-codegen/src/test/java/io/swagger/codegen/java/JavaModelEnumTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -80,7 +80,7 @@ public void overrideEnumTest() {
8080

8181
Assert.assertEquals(cm.name, "sample");
8282
Assert.assertEquals(cm.classname, "Sample");
83-
Assert.assertEquals(cm.parent, "parentModel");
83+
Assert.assertEquals(cm.parent, "ParentModel");
8484
Assert.assertTrue(cm.imports.contains("ParentModel"));
8585

8686
// Assert that only the unshared/uninherited enum remains

0 commit comments

Comments
 (0)