Polishing

cachescrubber · Apr 7, 2022 · 3bb412a · 3bb412a
1 parent 3fd8712
commit 3bb412a
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/...les/openapi-generator/src/main/java/org/openapitools/codegen/languages/SpringCodegen.java b/...les/openapi-generator/src/main/java/org/openapitools/codegen/languages/SpringCodegen.java
@@ -904,7 +904,7 @@ public Map<String, ModelsMap> postProcessAllModels(Map<String, ModelsMap> objs)
         for (ModelsMap modelsAttrs : objs.values()) {
             for (ModelMap mo : modelsAttrs.getModels()) {
                 CodegenModel codegenModel = mo.getModel();
-                Set<String> additionalImports = new HashSet<>();
+                Set<String> inheritedImports = new HashSet<>();
                 Map<String, CodegenProperty> propertyHash = new HashMap<>(codegenModel.vars.size());
                 for (final CodegenProperty property : codegenModel.vars) {
                     propertyHash.put(property.name, property);
@@ -923,7 +923,7 @@ public Map<String, ModelsMap> postProcessAllModels(Map<String, ModelsMap> objs)
                             for (String imp: imports) {
                                 // Avoid dupes
                                 if (!codegenModel.getImports().contains(imp)) {
-                                    additionalImports.add(imp);
+                                    inheritedImports.add(imp);
                                     codegenModel.getImports().add(imp);
                                 }
                             }
@@ -932,7 +932,7 @@ public Map<String, ModelsMap> postProcessAllModels(Map<String, ModelsMap> objs)
                     parentCodegenModel = parentCodegenModel.getParentModel();
                 }
                 // There must be a better way ...
-                for (String imp: additionalImports) {
+                for (String imp: inheritedImports) {
                     String qimp = importMapping().get(imp);
                     if (qimp != null) {
                         Map<String,String> toAdd = new HashMap<>();

diff --git a/...enapi-generator/src/test/java/org/openapitools/codegen/java/spring/SpringCodegenTest.java b/...enapi-generator/src/test/java/org/openapitools/codegen/java/spring/SpringCodegenTest.java
@@ -1026,6 +1026,7 @@ public void oneOf_allOf() throws IOException {
         // previous bugs
         assertFileNotContains(Paths.get(outputPath + "/src/main/java/org/openapitools/model/BarRef.java"), "atTypesuper.hashCode");
         assertFileNotContains(Paths.get(outputPath + "/src/main/java/org/openapitools/model/BarRef.java"), "private String atBaseType");
+        // imports for inherited properties
         assertFileContains(Paths.get(outputPath + "/src/main/java/org/openapitools/model/PizzaSpeziale.java"), "import java.math.BigDecimal");
     }