From 4e85975d231ee41e9ddf2f1c87e96b19dc234a30 Mon Sep 17 00:00:00 2001
From: Kyle Johnson <kyle@solidstategroup.com>
Date: Fri, 6 Sep 2024 09:33:02 +0100
Subject: [PATCH] fix: multivariate toggle (#4594)

---
 frontend/web/components/FeatureRow.js | 36 +++++++++++++--------------
 1 file changed, 17 insertions(+), 19 deletions(-)

diff --git a/frontend/web/components/FeatureRow.js b/frontend/web/components/FeatureRow.js
index 4e306161ee68..a878bfdf873b 100644
--- a/frontend/web/components/FeatureRow.js
+++ b/frontend/web/components/FeatureRow.js
@@ -141,6 +141,21 @@ class TheComponent extends Component {
     const changeRequestsEnabled = Utils.changeRequestsEnabled(
       environment && environment.minimum_change_request_approvals,
     )
+    const onChange = ()=> {
+        if(disableControls) {
+          return;
+        }
+        if (
+          projectFlag?.multivariate_options?.length ||
+          Utils.changeRequestsEnabled(
+            environment.minimum_change_request_approvals,
+          )
+        ) {
+          this.editFeature(projectFlag, environmentFlags[id])
+          return
+        }
+        this.confirmToggle()
+    }
     const isCompact = getViewMode() === 'compact'
     if (this.props.condensed) {
       return (
@@ -171,14 +186,7 @@ class TheComponent extends Component {
                     : '-off'
                 }`}
                 checked={environmentFlags[id] && environmentFlags[id].enabled}
-                onChange={() => {
-                  if (disableControls) return
-                  if (changeRequestsEnabled) {
-                    this.editFeature(projectFlag, environmentFlags[id])
-                    return
-                  }
-                  this.confirmToggle()
-                }}
+                onChange={onChange}
               />
             </Row>
           </div>
@@ -357,17 +365,7 @@ class TheComponent extends Component {
                 : '-off'
             }`}
             checked={environmentFlags[id] && environmentFlags[id].enabled}
-            onChange={() => {
-              if (
-                Utils.changeRequestsEnabled(
-                  environment.minimum_change_request_approvals,
-                )
-              ) {
-                this.editFeature(projectFlag, environmentFlags[id])
-                return
-              }
-              this.confirmToggle()
-            }}
+            onChange={onChange}
           />
         </div>