Merge pull request libre-tube#7345 from Bnyro/master

Bnyro · web-flow · commit 75aa69d0966a · 2025-04-25T15:34:43.000+02:00
fix: confirm unsubscribe preference only works if disabled instead of enabled
diff --git a/app/src/main/java/com/github/libretube/ui/extensions/SetupSubscriptionButton.kt b/app/src/main/java/com/github/libretube/ui/extensions/SetupSubscriptionButton.kt
@@ -62,7 +62,7 @@ fun TextView.setupSubscriptionButton(
                     updateUIStateAndNotifyObservers()
                 }
 
-                if (!PreferenceHelper.getBoolean(PreferenceKeys.CONFIRM_UNSUBSCRIBE, false)) {
+                if (PreferenceHelper.getBoolean(PreferenceKeys.CONFIRM_UNSUBSCRIBE, false)) {
                     showUnsubscribeDialog(context, channelName) {
                         CoroutineScope(Dispatchers.Main).launch { unsubscribeAction() }
                     }

Original file line number	Diff line number	Diff line change
`@@ -62,7 +62,7 @@ fun TextView.setupSubscriptionButton(`
`62`	`62`	`updateUIStateAndNotifyObservers()`
`63`	`63`	`}`
`64`	`64`
`65`		`- if (!PreferenceHelper.getBoolean(PreferenceKeys.CONFIRM_UNSUBSCRIBE, false)) {`
	`65`	`+ if (PreferenceHelper.getBoolean(PreferenceKeys.CONFIRM_UNSUBSCRIBE, false)) {`
`66`	`66`	`showUnsubscribeDialog(context, channelName) {`
`67`	`67`	`CoroutineScope(Dispatchers.Main).launch { unsubscribeAction() }`
`68`	`68`	`}`