Update internal/services/appservice/helpers/linux_web_app_schema.go

Hi @katbyte, I'm not sure that I got your point, since line 977 `len(autoHeal.Triggers) == 0`, line 979 `autoHeal.Triggers[0]` would trigger an index out of range panic right? Do you mean `if len(autoHeal.Triggers) == 1 {` for line 977? I personal prefer return early style to reduce nested blocks' depth and avoid [arrow code](https://blog.codinghorror.com/flattening-arrow-code/), but this is HashiCorp's project so I would be happy to follow your guidance. Co-authored-by: kt <[email protected]>
hashicorp · Apr 17, 2023 · 2b3cccb · 2b3cccb
1 parent ef69562
commit 2b3cccb
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/internal/services/appservice/helpers/linux_web_app_schema.go b/internal/services/appservice/helpers/linux_web_app_schema.go
@@ -975,11 +975,9 @@ func expandAutoHealSettingsLinux(autoHealSettings []AutoHealSettingLinux) *web.A
 	}
 
 	if len(autoHeal.Triggers) == 0 {
-		return result
-	}
 
-	triggers := autoHeal.Triggers[0]
-	if len(triggers.Requests) == 1 {
+		triggers := autoHeal.Triggers[0]
+		if len(triggers.Requests) == 1 {
 		result.Triggers.Requests = &web.RequestsBasedTrigger{
 			Count:        pointer.To(int32(triggers.Requests[0].Count)),
 			TimeInterval: pointer.To(triggers.Requests[0].Interval),