fix(locality): improve suppressSDKNullAssignment to handle null-to-value transitions

jremy42 · jremy42 · commit 324668473814 · 2025-09-10T06:55:58.000+02:00
diff --git a/internal/locality/regional/schemas.go b/internal/locality/regional/schemas.go
@@ -37,5 +37,5 @@ func Schema() *schema.Schema {
 }
 
 func suppressSDKNullAssignment(k, old, new string, d *schema.ResourceData) bool {
-	return new == "" && old != ""
+	return (new == "" && old != "") || (old == "" && new != "")
 }
diff --git a/internal/locality/zonal/schemas.go b/internal/locality/zonal/schemas.go
@@ -37,5 +37,5 @@ func Schema() *schema.Schema {
 }
 
 func suppressSDKNullAssignment(k, old, new string, d *schema.ResourceData) bool {
-	return new == "" && old != ""
+	return (new == "" && old != "") || (old == "" && new != "")
 }

Original file line number	Diff line number	Diff line change
`@@ -37,5 +37,5 @@ func Schema() *schema.Schema {`
`37`	`37`	`}`
`38`	`38`
`39`	`39`	`func suppressSDKNullAssignment(k, old, new string, d *schema.ResourceData) bool {`
`40`		`- return new == "" && old != ""`
	`40`	`+ return (new == "" && old != "") \|\| (old == "" && new != "")`
`41`	`41`	`}`