don't invoke protections toggle listener without user interaction

duckduckgo · Nov 16, 2023 · f92d55e · f92d55e
1 parent c8fee28
commit f92d55e
Showing 1 changed file with 10 additions and 1 deletion.
diff --git a/app/src/main/java/com/duckduckgo/app/brokensite/view/SiteProtectionsToggle.kt b/app/src/main/java/com/duckduckgo/app/brokensite/view/SiteProtectionsToggle.kt
@@ -43,7 +43,9 @@ class SiteProtectionsToggle @JvmOverloads constructor(
 
     fun setOnProtectionsToggledListener(listener: (Boolean) -> Unit) {
         binding.protectionsSwitch.setOnCheckedChangeListener { _, isChecked ->
-            listener.invoke(isChecked)
+            if (isChecked != areProtectionsEnabled()) {
+                listener.invoke(isChecked)
+            }
         }
     }
 
@@ -80,4 +82,11 @@ class SiteProtectionsToggle @JvmOverloads constructor(
             }
         }
     }
+
+    private fun areProtectionsEnabled(): Boolean = when (state) {
+        SiteProtectionsState.ENABLED -> true
+        SiteProtectionsState.DISABLED,
+        SiteProtectionsState.DISABLED_BY_REMOTE_CONFIG,
+        -> false
+    }
 }