Keep nullability when inlining ref

Author: ben-schreiber

modules/openapi-generator/src/main/java/org/openapitools/codegen/utils/ModelUtils.java

@@ -1410,7 +1410,12 @@ public static Schema unaliasSchema(OpenAPI openAPI,
                             schemaMappings);
                 }
             } else {
-                return unaliasSchema(openAPI, allSchemas.get(ModelUtils.getSimpleRef(schema.get$ref())), schemaMappings);
+                Schema unaliased = unaliasSchema(openAPI, allSchemas.get(ModelUtils.getSimpleRef(schema.get$ref())), schemaMappings);
+                // Preserve nullable property from the original schema reference
+                if (Boolean.TRUE.equals(schema.getNullable())) {
+                    unaliased.setNullable(true);
+                }
+                return unaliased;
             }
         }
         return schema;

modules/openapi-generator/src/test/java/org/openapitools/codegen/utils/ModelUtilsTest.java

@@ -242,6 +242,22 @@ public void testAliasedTypeIsNotUnaliasedIfUsedForImportMapping() {
         Assert.assertEquals(stringSchema, ModelUtils.unaliasSchema(openAPI, emailSchema, new HashMap<>()));
     }
 
+    /**
+     * This test verifies that nullable property is preserved when unaliasing a schema reference.
+     * See https://github.com/OpenAPITools/openapi-generator/issues/21612
+     */
+    @Test
+    public void testNullableUnaliasedSchema() {
+        Schema refSchema = new Schema().$ref("#/components/schemas/MyString").nullable(true);
+        StringSchema myStringSchema = new StringSchema();
+        OpenAPI openAPI = TestUtils.createOpenAPIWithOneSchema("MyString", myStringSchema);
+
+        Schema unaliased = ModelUtils.unaliasSchema(openAPI, refSchema, new HashMap<>());
+
+        Assert.assertTrue(unaliased.getNullable(), "Nullable property should be preserved after unaliasing");
+        Assert.assertNotSame(unaliased, refSchema, "Unaliased schema should not be the same instance as the original reference");
+    }
+
     /**
      * Issue https://github.com/OpenAPITools/openapi-generator/issues/1624.
      * ModelUtils.isFreeFormObject() should not throw an NPE when passed an empty