Fix problem with schema_change_handling validation for SAP connecto…

…rs (#376)

CHANGELOG.md

@@ -5,7 +5,12 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
-## [Unreleased](https://github.com/fivetran/terraform-provider-fivetran/compare/v1.4.0...HEAD)
+## [Unreleased](https://github.com/fivetran/terraform-provider-fivetran/compare/v1.4.1...HEAD)
+
+## [1.4.1](https://github.com/fivetran/terraform-provider-fivetran/compare/v1.4.0...v1.4.1)
+
+## Fixed
+- Problem with `schema_change_handling` validation for SAP connectors
 
 ## [1.4.0](https://github.com/fivetran/terraform-provider-fivetran/compare/v1.3.2...v1.4.0)
 

fivetran/framework/core/model/connector_schema_config.go

@@ -95,8 +95,10 @@ func (d *ConnectorSchemaResourceModel) ReadFromResponse(response connectors.Conn
 	}
 
 	// SAP connectors will not accept schemaChangeHandling in request and will return ALLOW_COLUMNS as default
-	if d.SchemaChangeHandling.IsNull() || d.SchemaChangeHandling.IsUnknown() || d.SchemaChangeHandling.ValueString() != "" {
+	if !d.SchemaChangeHandling.IsNull() && !d.SchemaChangeHandling.IsUnknown() && d.SchemaChangeHandling.ValueString() != "" {
 		d.SchemaChangeHandling = types.StringValue(response.Data.SchemaChangeHandling)
+	} else {
+		d.SchemaChangeHandling = types.StringNull()
 	}
 }