From 18ac05ed80d6ffa31323bb2f6652a440a8493602 Mon Sep 17 00:00:00 2001
From: darken <darken@darken.eu>
Date: Fri, 12 Jul 2024 21:46:48 +0200
Subject: [PATCH] Fix crashes when entering settings

Likely from #209
---
 .../ui/settings/support/SupportFragment.kt     | 12 ------------
 .../reaction/ui/ReactionSettingsFragment.kt    | 12 ++++++------
 app/src/main/res/xml/preferences_support.xml   | 18 +-----------------
 3 files changed, 7 insertions(+), 35 deletions(-)

diff --git a/app/src/main/java/eu/darken/capod/main/ui/settings/support/SupportFragment.kt b/app/src/main/java/eu/darken/capod/main/ui/settings/support/SupportFragment.kt
index 6fd22a2c..72fc9a74 100644
--- a/app/src/main/java/eu/darken/capod/main/ui/settings/support/SupportFragment.kt
+++ b/app/src/main/java/eu/darken/capod/main/ui/settings/support/SupportFragment.kt
@@ -4,7 +4,6 @@ import android.os.Bundle
 import android.view.View
 import androidx.annotation.Keep
 import androidx.fragment.app.viewModels
-import androidx.preference.Preference
 import com.google.android.material.snackbar.Snackbar
 import dagger.hilt.android.AndroidEntryPoint
 import eu.darken.capod.R
@@ -27,17 +26,6 @@ class SupportFragment : PreferenceFragment3() {
 
     @Inject lateinit var clipboardHelper: ClipboardHelper
 
-    private val installIdPref by lazy { findPreference<Preference>("support.installid")!! }
-
-    override fun onPreferencesCreated() {
-        installIdPref.setOnPreferenceClickListener {
-            vm.copyInstallID()
-            true
-        }
-
-        super.onPreferencesCreated()
-    }
-
     override fun onViewCreated(view: View, savedInstanceState: Bundle?) {
         vm.clipboardEvent.observe2(this) { installId ->
             Snackbar.make(requireView(), installId, Snackbar.LENGTH_INDEFINITE)
diff --git a/app/src/main/java/eu/darken/capod/reaction/ui/ReactionSettingsFragment.kt b/app/src/main/java/eu/darken/capod/reaction/ui/ReactionSettingsFragment.kt
index bf888131..2184388b 100644
--- a/app/src/main/java/eu/darken/capod/reaction/ui/ReactionSettingsFragment.kt
+++ b/app/src/main/java/eu/darken/capod/reaction/ui/ReactionSettingsFragment.kt
@@ -5,9 +5,9 @@ import android.view.View
 import androidx.annotation.Keep
 import androidx.fragment.app.viewModels
 import androidx.lifecycle.asLiveData
-import androidx.preference.CheckBoxPreference
 import androidx.preference.ListPreference
 import androidx.preference.Preference
+import androidx.preference.TwoStatePreference
 import dagger.hilt.android.AndroidEntryPoint
 import eu.darken.capod.R
 import eu.darken.capod.common.bluetooth.BluetoothDevice2
@@ -51,17 +51,17 @@ class ReactionSettingsFragment : PreferenceFragment3() {
 
     override fun onPreferenceTreeClick(preference: Preference): Boolean {
         if (preference.key == reactionSettings.autoPlay.key && !isPro) {
-            preference as CheckBoxPreference
+            preference as TwoStatePreference
             upgradeRepo.launchBillingFlow(requireActivity())
             preference.isChecked = false
             return true
         } else if (preference.key == reactionSettings.autoPause.key && !isPro) {
-            preference as CheckBoxPreference
+            preference as TwoStatePreference
             upgradeRepo.launchBillingFlow(requireActivity())
             preference.isChecked = false
             return true
         } else if (preference.key == reactionSettings.autoConnect.key) {
-            preference as CheckBoxPreference
+            preference as TwoStatePreference
 
             if (!isPro) {
                 upgradeRepo.launchBillingFlow(requireActivity())
@@ -79,12 +79,12 @@ class ReactionSettingsFragment : PreferenceFragment3() {
                 return true
             }
         } else if (preference.key == reactionSettings.showPopUpOnCaseOpen.key && !isPro) {
-            preference as CheckBoxPreference
+            preference as TwoStatePreference
             upgradeRepo.launchBillingFlow(requireActivity())
             preference.isChecked = false
             return true
         } else if (preference.key == reactionSettings.showPopUpOnConnection.key && !isPro) {
-            preference as CheckBoxPreference
+            preference as TwoStatePreference
             upgradeRepo.launchBillingFlow(requireActivity())
             preference.isChecked = false
             return true
diff --git a/app/src/main/res/xml/preferences_support.xml b/app/src/main/res/xml/preferences_support.xml
index bb51809a..b11ea521 100644
--- a/app/src/main/res/xml/preferences_support.xml
+++ b/app/src/main/res/xml/preferences_support.xml
@@ -1,6 +1,5 @@
 <?xml version="1.0" encoding="utf-8"?>
-<PreferenceScreen xmlns:android="http://schemas.android.com/apk/res/android"
-    xmlns:app="http://schemas.android.com/apk/res-auto">
+<PreferenceScreen xmlns:android="http://schemas.android.com/apk/res/android">
     <eu.darken.capod.common.preferences.IntentPreference
         android:icon="@drawable/ic_discord_onsurface"
         android:summary="@string/discord_description"
@@ -18,19 +17,4 @@
             android:data="https://github.com/d4rken-org/capod/issues" />
     </eu.darken.capod.common.preferences.IntentPreference>
 
-    <!-- Hide Install-ID and bug reporting settings, but keep for possible use in the future.
-    <PreferenceCategory app:title="@string/settings_category_other_label">
-        <Preference
-            android:icon="@drawable/ic_id_onsurface"
-            android:key="support.installid"
-            android:summary="@string/settings_support_installid_desc"
-            android:title="@string/settings_support_installid_label" />
-
-        <CheckBoxPreference
-            android:icon="@drawable/ic_spider_thread_onsurface"
-            android:key="debug.bugreport.automatic.enabled"
-            android:summary="@string/settings_debug_autoreports_description"
-            android:title="@string/settings_debug_autoreports_label" />
-    </PreferenceCategory>
-    -->
 </PreferenceScreen>