add config option for Python backwards compatibility

Author: ctreatma

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

@@ -403,6 +403,10 @@ public static enum ENUM_PROPERTY_NAMING_TYPE {camelCase, PascalCase, snake_case,
     public static final String LEGACY_DISCRIMINATOR_BEHAVIOR = "legacyDiscriminatorBehavior";
     public static final String LEGACY_DISCRIMINATOR_BEHAVIOR_DESC = "Set to false for generators with better support for discriminators. (Python, Java, Go, PowerShell, C# have this enabled by default).";
 
+    public static final String LEGACY_DEFAULT_DISALLOW_ADDITIONAL_PROPERTIES_IF_NOT_PRESENT_BEHAVIOR = "legacyDefaultDisallowAdditionalPropertiesIfNotPresentBehavior";
+    public static final String LEGACY_DEFAULT_DISALLOW_ADDITIONAL_PROPERTIES_IF_NOT_PRESENT_BEHAVIOR_DESC = "Set to true for Python generators so that generated code matches what was previously produced when `disallowAdditionalProperties` was unspecified";
+
+
     public static final String USE_SINGLE_REQUEST_PARAMETER = "useSingleRequestParameter";
     public static final String USE_SINGLE_REQUEST_PARAMETER_DESC = "Setting this property to true will generate functions with a single argument containing all API endpoint parameters instead of one argument per parameter.";
 

modules/openapi-generator/src/main/java/org/openapitools/codegen/languages/AbstractPythonCodegen.java

@@ -51,6 +51,8 @@ public abstract class AbstractPythonCodegen extends DefaultCodegen implements Co
     protected String packageName = "openapi_client";
     @Setter protected String packageVersion = "1.0.0";
     @Setter protected String projectName; // for setup.py, e.g. petstore-api
+    @Setter
+    protected boolean legacyDisallowAdditionalPropertiesDefaultBehavior = false;
     protected boolean hasModelsToImport = Boolean.FALSE;
     protected String mapNumberTo = "Union[StrictFloat, StrictInt]";
     protected Map<Character, String> regexModifiers;
@@ -145,6 +147,10 @@ public AbstractPythonCodegen() {
     public void processOpts() {
         super.processOpts();
 
+        if (additionalProperties.containsKey(CodegenConstants.LEGACY_DEFAULT_DISALLOW_ADDITIONAL_PROPERTIES_IF_NOT_PRESENT_BEHAVIOR)) {
+            setLegacyDisallowAdditionalPropertiesDefaultBehavior(Boolean.parseBoolean(additionalProperties.get(CodegenConstants.LEGACY_DEFAULT_DISALLOW_ADDITIONAL_PROPERTIES_IF_NOT_PRESENT_BEHAVIOR).toString()));
+        }
+
         if (StringUtils.isEmpty(System.getenv("PYTHON_POST_PROCESS_FILE"))) {
             LOGGER.info("Environment variable PYTHON_POST_PROCESS_FILE not defined so the Python code may not be properly formatted. To define it, try 'export PYTHON_POST_PROCESS_FILE=\"/usr/local/bin/yapf -i\"' (Linux/Mac)");
             LOGGER.info("NOTE: To enable file post-processing, 'enablePostProcessFile' must be set to `true` (--enable-post-process-file for CLI).");
@@ -975,6 +981,10 @@ private ModelsMap postProcessModelsMap(ModelsMap objs) {
             // set the extensions if the key is absent
             model.getVendorExtensions().putIfAbsent("x-py-readonly", readOnlyFields);
 
+            if (legacyDisallowAdditionalPropertiesDefaultBehavior) {
+                model.vendorExtensions.putIfAbsent("x-py-legacy-disallow-additional-properties-default-behavior", true);
+            }
+
             // remove the items of postponedModelImports in modelImports to avoid circular imports error
             if (!modelImports.isEmpty() && !postponedModelImports.isEmpty()) {
                 modelImports.removeAll(postponedModelImports);

modules/openapi-generator/src/main/resources/python-fastapi/model_generic.mustache

@@ -260,15 +260,15 @@ class {{classname}}({{#parent}}{{{.}}}{{/parent}}{{^parent}}BaseModel{{/parent}}
         if not isinstance(obj, dict):
             return cls.model_validate(obj)
 
-        {{^vendorExtensions.x-py-legacy-additional-properties-validation}}
+        {{^vendorExtensions.x-py-legacy-disallow-additional-properties-default-behavior}}
         {{^isAdditionalPropertiesTrue}}
         # raise errors for additional fields in the input
         for _key in obj.keys():
             if _key not in cls.__properties:
                 raise ValueError("Error due to additional fields (not defined in {{classname}}) in the input: " + _key)
 
         {{/isAdditionalPropertiesTrue}}
-        {{/vendorExtensions.x-py-legacy-additional-properties-validation}}
+        {{/vendorExtensions.x-py-legacy-disallow-additional-properties-default-behavior}}
         _obj = cls.model_validate({
             {{#allVars}}
             {{#isContainer}}

modules/openapi-generator/src/main/resources/python-pydantic-v1/model_generic.mustache

@@ -252,15 +252,15 @@ class {{classname}}({{#parent}}{{{.}}}{{/parent}}{{^parent}}BaseModel{{/parent}}
         if not isinstance(obj, dict):
             return {{{classname}}}.parse_obj(obj)
 
-        {{^vendorExtensions.x-py-legacy-additional-properties-validation}}
+        {{^vendorExtensions.x-py-legacy-disallow-additional-properties-default-behavior}}
         {{^isAdditionalPropertiesTrue}}
         # raise errors for additional fields in the input
         for _key in obj.keys():
             if _key not in cls.__properties:
                 raise ValueError("Error due to additional fields (not defined in {{classname}}) in the input: " + obj)
 
         {{/isAdditionalPropertiesTrue}}
-        {{/vendorExtensions.x-py-legacy-additional-properties-validation}}
+        {{/vendorExtensions.x-py-legacy-disallow-additional-properties-default-behavior}}
         _obj = {{{classname}}}.parse_obj({
             {{#allVars}}
             {{#isContainer}}

modules/openapi-generator/src/main/resources/python/model_generic.mustache

@@ -279,15 +279,15 @@ class {{classname}}({{#parent}}{{{.}}}{{/parent}}{{^parent}}BaseModel{{/parent}}
         if not isinstance(obj, dict):
             return cls.model_validate(obj)
 
-        {{^vendorExtensions.x-py-legacy-additional-properties-validation}}
+        {{^vendorExtensions.x-py-legacy-disallow-additional-properties-default-behavior}}
         {{^isAdditionalPropertiesTrue}}
         # raise errors for additional fields in the input
         for _key in obj.keys():
             if _key not in cls.__properties:
                 raise ValueError("Error due to additional fields (not defined in {{classname}}) in the input: " + _key)
 
         {{/isAdditionalPropertiesTrue}}
-        {{/vendorExtensions.x-py-legacy-additional-properties-validation}}
+        {{/vendorExtensions.x-py-legacy-disallow-additional-properties-default-behavior}}
         _obj = cls.model_validate({
             {{#allVars}}
             {{#isContainer}}