filter nil (#6916) (#4956)

modular-magician · Edward Sun · web-flow · commit 8023c05b55b6 · 2022-12-06T13:51:34.000-08:00
Co-authored-by: Edward Sun &lt;sunedward@google.com&gt;
Signed-off-by: Modular Magician &lt;magic-modules@google.com&gt;

Signed-off-by: Modular Magician &lt;magic-modules@google.com&gt;
Co-authored-by: Edward Sun &lt;sunedward@google.com&gt;
diff --git a/.changelog/6916.txt b/.changelog/6916.txt
@@ -0,0 +1,3 @@
+```release-note:bug
+cloud composer: fixed a crash when updating `ip_allocation_policy` of `google_composer_environment`
+```
diff --git a/google-beta/resource_composer_environment.go b/google-beta/resource_composer_environment.go
@@ -1766,7 +1766,7 @@ func expandComposerEnvironmentConfigNodeConfig(v interface{}, d *schema.Resource
 
 func expandComposerEnvironmentIPAllocationPolicy(v interface{}, d *schema.ResourceData, config *Config) (*composer.IPAllocationPolicy, error) {
 	l := v.([]interface{})
-	if len(l) == 0 {
+	if len(l) == 0 || l[0] == nil {
 		return nil, nil
 	}
 	raw := l[0]

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,3 @@`
	`1`	+```release-note:bug
	`2`	+cloud composer: fixed a crash when updating `ip_allocation_policy` of `google_composer_environment`
	`3`	+```
Original file line number	Diff line number	Diff line change
`@@ -1766,7 +1766,7 @@ func expandComposerEnvironmentConfigNodeConfig(v interface{}, d *schema.Resource`
`1766`	`1766`
`1767`	`1767`	`func expandComposerEnvironmentIPAllocationPolicy(v interface{}, d schema.ResourceData, config Config) (*composer.IPAllocationPolicy, error) {`
`1768`	`1768`	`l := v.([]interface{})`
`1769`		`- if len(l) == 0 {`
	`1769`	`+ if len(l) == 0 \|\| l[0] == nil {`
`1770`	`1770`	`return nil, nil`
`1771`	`1771`	`}`
`1772`	`1772`	`raw := l[0]`