From 66805bdf05696bdf67c79393cb31d786b07a6a5e Mon Sep 17 00:00:00 2001
From: rebelonion <87634197+rebelonion@users.noreply.github.com>
Date: Tue, 21 May 2024 11:28:46 -0500
Subject: [PATCH] fix: list view crash

---
 .../ani/dantotsu/home/AnimePageAdapter.kt     |  4 +--
 .../java/ani/dantotsu/home/HomeFragment.kt    |  8 ++---
 .../ani/dantotsu/home/MangaPageAdapter.kt     |  4 +--
 .../dantotsu/media/MediaListViewActivity.kt   |  7 +++-
 .../ExtensionTestSettingsBottomDialog.kt      |  2 +-
 .../res/layout/activity_media_list_view.xml   | 36 +++++++++----------
 6 files changed, 31 insertions(+), 30 deletions(-)

diff --git a/app/src/main/java/ani/dantotsu/home/AnimePageAdapter.kt b/app/src/main/java/ani/dantotsu/home/AnimePageAdapter.kt
index 605d1cab58..df0e093989 100644
--- a/app/src/main/java/ani/dantotsu/home/AnimePageAdapter.kt
+++ b/app/src/main/java/ani/dantotsu/home/AnimePageAdapter.kt
@@ -268,11 +268,11 @@ class AnimePageAdapter : RecyclerView.Adapter<AnimePageAdapter.AnimePageViewHold
                 LinearLayoutManager.HORIZONTAL,
                 false
             )
+        MediaListViewActivity.passedMedia = media.toCollection(ArrayList())
         more.setOnClickListener {
             ContextCompat.startActivity(
                 it.context, Intent(it.context, MediaListViewActivity::class.java)
-                    .putExtra("title", string)
-                    .putExtra("media",  media as ArrayList<Media>),
+                    .putExtra("title", string),
                 null
             )
         }
diff --git a/app/src/main/java/ani/dantotsu/home/HomeFragment.kt b/app/src/main/java/ani/dantotsu/home/HomeFragment.kt
index bedc73ea2c..fc729b9d89 100644
--- a/app/src/main/java/ani/dantotsu/home/HomeFragment.kt
+++ b/app/src/main/java/ani/dantotsu/home/HomeFragment.kt
@@ -233,10 +233,10 @@ class HomeFragment : Fragment() {
                             false
                         )
                         more.setOnClickListener { i ->
+                            MediaListViewActivity.passedMedia = it
                             ContextCompat.startActivity(
                                 i.context, Intent(i.context, MediaListViewActivity::class.java)
-                                    .putExtra("title", string)
-                                    .putExtra("media", it),
+                                    .putExtra("title", string),
                                 null
                             )
                         }
@@ -393,11 +393,11 @@ class HomeFragment : Fragment() {
                         true
                     }
                     binding.homeHiddenItemsMore.setSafeOnClickListener { _ ->
+                        MediaListViewActivity.passedMedia = it
                         ContextCompat.startActivity(
                             requireActivity(),
                             Intent(requireActivity(), MediaListViewActivity::class.java)
-                                .putExtra("title", getString(R.string.hidden))
-                                .putExtra("media", it),
+                                .putExtra("title", getString(R.string.hidden)),
                             null
                         )
                     }
diff --git a/app/src/main/java/ani/dantotsu/home/MangaPageAdapter.kt b/app/src/main/java/ani/dantotsu/home/MangaPageAdapter.kt
index e26c734794..2e3e6a8cd0 100644
--- a/app/src/main/java/ani/dantotsu/home/MangaPageAdapter.kt
+++ b/app/src/main/java/ani/dantotsu/home/MangaPageAdapter.kt
@@ -271,10 +271,10 @@ class MangaPageAdapter : RecyclerView.Adapter<MangaPageAdapter.MangaPageViewHold
                 false
             )
         more.setOnClickListener {
+            MediaListViewActivity.passedMedia = media.toCollection(ArrayList())
             ContextCompat.startActivity(
                 it.context, Intent(it.context, MediaListViewActivity::class.java)
-                    .putExtra("title", string)
-                    .putExtra("media",  media as ArrayList<Media>),
+                    .putExtra("title", string),
                 null
             )
         }
diff --git a/app/src/main/java/ani/dantotsu/media/MediaListViewActivity.kt b/app/src/main/java/ani/dantotsu/media/MediaListViewActivity.kt
index 32011d0fff..60a9e89845 100644
--- a/app/src/main/java/ani/dantotsu/media/MediaListViewActivity.kt
+++ b/app/src/main/java/ani/dantotsu/media/MediaListViewActivity.kt
@@ -52,7 +52,8 @@ class MediaListViewActivity: AppCompatActivity() {
         binding.listAppBar.setBackgroundColor(primaryColor)
         binding.listTitle.setTextColor(primaryTextColor)
         val screenWidth = resources.displayMetrics.run { widthPixels / density }
-        val mediaList = intent.getSerialized("media") as? ArrayList<Media> ?: ArrayList()
+        val mediaList = passedMedia ?: intent.getSerialized("media") as? ArrayList<Media> ?: ArrayList()
+        if (passedMedia != null) passedMedia = null
         val view = PrefManager.getCustomVal("mediaView", 0)
         var mediaView: View = when (view) {
             1 -> binding.mediaList
@@ -85,4 +86,8 @@ class MediaListViewActivity: AppCompatActivity() {
             if (view == 1) 1 else (screenWidth / 120f).toInt()
         )
     }
+
+    companion object {
+        var passedMedia: ArrayList<Media>? = null
+    }
 }
diff --git a/app/src/main/java/ani/dantotsu/parsers/ExtensionTestSettingsBottomDialog.kt b/app/src/main/java/ani/dantotsu/parsers/ExtensionTestSettingsBottomDialog.kt
index 1b7777eabf..5bd6bb8590 100644
--- a/app/src/main/java/ani/dantotsu/parsers/ExtensionTestSettingsBottomDialog.kt
+++ b/app/src/main/java/ani/dantotsu/parsers/ExtensionTestSettingsBottomDialog.kt
@@ -132,7 +132,7 @@ class ExtensionTestSettingsBottomDialog : BottomSheetDialogFragment() {
         }
 
         var extensionType = "anime"
-        var testType = "ping"
+        var testType = "basic"
         var searchQuery = "Chainsaw Man"
         var extensionsToTest: MutableList<String> = mutableListOf()
     }
diff --git a/app/src/main/res/layout/activity_media_list_view.xml b/app/src/main/res/layout/activity_media_list_view.xml
index 987a3b34a9..04a8af1191 100644
--- a/app/src/main/res/layout/activity_media_list_view.xml
+++ b/app/src/main/res/layout/activity_media_list_view.xml
@@ -1,14 +1,11 @@
 <?xml version="1.0" encoding="utf-8"?>
-<androidx.coordinatorlayout.widget.CoordinatorLayout xmlns:android="http://schemas.android.com/apk/res/android"
+<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
     xmlns:tools="http://schemas.android.com/tools"
     android:layout_width="match_parent"
     android:layout_height="match_parent"
+    android:orientation="vertical"
     xmlns:app="http://schemas.android.com/apk/res-auto">
 
-    <androidx.core.widget.NestedScrollView
-        android:layout_width="match_parent"
-        android:layout_height="wrap_content">
-
         <com.google.android.material.appbar.AppBarLayout
             android:id="@+id/listAppBar"
             android:layout_width="match_parent"
@@ -60,20 +57,19 @@
                     tools:ignore="ContentDescription,ImageContrastCheck" />
             </LinearLayout>
 
-
-            <ani.dantotsu.FadingEdgeRecyclerView
-                android:id="@+id/mediaRecyclerView"
-                android:layout_width="match_parent"
-                android:layout_height="match_parent"
-                android:paddingStart="20dp"
-                android:paddingEnd="20dp"
-                android:requiresFadingEdge="horizontal"
-                tools:itemCount="4"
-                tools:layoutManager="androidx.recyclerview.widget.LinearLayoutManager"
-                tools:listitem="@layout/item_media_compact"
-                tools:orientation="horizontal" />
-
         </com.google.android.material.appbar.AppBarLayout>
-    </androidx.core.widget.NestedScrollView>
 
-</androidx.coordinatorlayout.widget.CoordinatorLayout>
+    <ani.dantotsu.FadingEdgeRecyclerView
+        android:id="@+id/mediaRecyclerView"
+        android:layout_width="match_parent"
+        android:layout_height="match_parent"
+        android:paddingStart="20dp"
+        android:paddingEnd="20dp"
+        android:requiresFadingEdge="horizontal"
+        android:nestedScrollingEnabled="false"
+        tools:itemCount="4"
+        tools:layoutManager="androidx.recyclerview.widget.LinearLayoutManager"
+        tools:listitem="@layout/item_media_compact"
+        tools:orientation="horizontal" />
+
+</LinearLayout>