Fixing boolean logic which requires that if an account is specified, then domain must be as well

maxdrib · maxdrib · commit f81d0871ed4f · 2022-01-20T16:40:36.000-06:00
diff --git a/api/v1alpha3/cloudstackcluster_webhook.go b/api/v1alpha3/cloudstackcluster_webhook.go
@@ -62,7 +62,7 @@ func (r *CloudStackCluster) ValidateCreate() error {
 		errorList = append(errorList, field.Forbidden(field.NewPath("spec", "identityRef", "kind"), "must be a Secret"))
 	}
 
-	if (r.Spec.Account == "") != (r.Spec.Domain == "") {
+	if (r.Spec.Account != "") && (r.Spec.Domain == "") {
 		errorList = append(errorList, field.Required(field.NewPath("spec", "account"), "account and domain must be specified together"))
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -62,7 +62,7 @@ func (r *CloudStackCluster) ValidateCreate() error {`
`62`	`62`	`errorList = append(errorList, field.Forbidden(field.NewPath("spec", "identityRef", "kind"), "must be a Secret"))`
`63`	`63`	`}`
`64`	`64`
`65`		`- if (r.Spec.Account == "") != (r.Spec.Domain == "") {`
	`65`	`+ if (r.Spec.Account != "") && (r.Spec.Domain == "") {`
`66`	`66`	`errorList = append(errorList, field.Required(field.NewPath("spec", "account"), "account and domain must be specified together"))`
`67`	`67`	`}`
`68`	`68`