clean up ExtensionDetailsController

az4521 · Mar 10, 2024 · f0c8dba · f0c8dba
1 parent a9c66c0
commit f0c8dba
Showing 1 changed file with 10 additions and 9 deletions.
diff --git a/app/src/main/java/eu/kanade/tachiyomi/ui/extension/ExtensionDetailsController.kt b/app/src/main/java/eu/kanade/tachiyomi/ui/extension/ExtensionDetailsController.kt
@@ -33,6 +33,7 @@ class ExtensionDetailsController(bundle: Bundle? = null) :
      */
     private var adapter: ExtensionDetailsPrefsButtonAdapter? = null
 
+    private var sources: List<ConfigurableSource>? = null
     override fun inflateView(inflater: LayoutInflater, container: ViewGroup): View {
         binding = ExtensionDetailControllerBinding.inflate(inflater)
         return binding.root
@@ -82,21 +83,21 @@ class ExtensionDetailsController(bundle: Bundle? = null) :
         binding.extensionDetailsRecycler.adapter = adapter
         binding.extensionDetailsRecycler.setHasFixedSize(true)
 
+        sources = presenter.extension?.sources?.filterIsInstance<ConfigurableSource>()
+
         adapter!!.updateDataSet(
-            presenter.extension?.sources?.filterIsInstance<ConfigurableSource>()
-                ?.map { ExtensionDetailsPrefsButtonItem(it.toString()) }
+            sources?.map { ExtensionDetailsPrefsButtonItem(it.toString()) }
         )
+    }
 
-        /*presenter.extension?.sources?.filterIsInstance<ConfigurableSource>()?.forEach { source ->
-            binding.extensionPrefs.visible()
-            binding.extensionPrefs.clicks()
-                .onEach { openPreferences(source.id) }
-                .launchIn(scope)
-        }*/
+    override fun onDestroyView(view: View) {
+        adapter = null
+        sources = null
+        super.onDestroyView(view)
     }
 
     override fun onItemClick(view: View?, position: Int): Boolean {
-        val id = presenter.extension?.sources?.filterIsInstance<ConfigurableSource>()?.get(position)?.id
+        val id = sources?.get(position)?.id
         return if (id != null) {
             openPreferences(id)
             true