From e836625223a336e46304f6fac7699d3b1fb9896b Mon Sep 17 00:00:00 2001
From: Bnyro <bnyro@tutanota.com>
Date: Thu, 13 Oct 2022 14:10:39 +0200
Subject: [PATCH] allow fetching more comment replies

---
 .../libretube/ui/adapters/CommentsAdapter.kt  | 66 +++++++++++--------
 app/src/main/res/layout/comments_row.xml      | 14 ++++
 app/src/main/res/values/strings.xml           |  1 +
 3 files changed, 53 insertions(+), 28 deletions(-)

diff --git a/app/src/main/java/com/github/libretube/ui/adapters/CommentsAdapter.kt b/app/src/main/java/com/github/libretube/ui/adapters/CommentsAdapter.kt
index 78a41468c..b0932c34d 100644
--- a/app/src/main/java/com/github/libretube/ui/adapters/CommentsAdapter.kt
+++ b/app/src/main/java/com/github/libretube/ui/adapters/CommentsAdapter.kt
@@ -22,8 +22,6 @@ import com.github.libretube.util.NavigationHelper
 import kotlinx.coroutines.CoroutineScope
 import kotlinx.coroutines.Dispatchers
 import kotlinx.coroutines.launch
-import retrofit2.HttpException
-import java.io.IOException
 
 class CommentsAdapter(
     private val videoId: String,
@@ -32,8 +30,7 @@ class CommentsAdapter(
 ) : RecyclerView.Adapter<CommentsViewHolder>() {
 
     private var isLoading = false
-    private var nextpage = ""
-    private var repliesPage = CommentsPage()
+    private lateinit var repliesPage: CommentsPage
 
     fun clear() {
         val size: Int = comments.size
@@ -68,33 +65,46 @@ class CommentsAdapter(
             ImageHelper.loadImage(comment.thumbnail, commentorImage)
             likesTextView.text = comment.likeCount?.toLong().formatShort()
 
-            if (comment.verified == true) {
-                verifiedImageView.visibility = View.VISIBLE
-            }
-            if (comment.pinned == true) {
-                pinnedImageView.visibility = View.VISIBLE
-            }
-            if (comment.hearted == true) {
-                heartedImageView.visibility = View.VISIBLE
-            }
-
-            if (comment.repliesPage != null) {
-                commentsAvailable.visibility = View.VISIBLE
-            }
+            if (comment.verified == true) verifiedImageView.visibility = View.VISIBLE
+            if (comment.pinned == true) pinnedImageView.visibility = View.VISIBLE
+            if (comment.hearted == true) heartedImageView.visibility = View.VISIBLE
+            if (comment.repliesPage != null) commentsAvailable.visibility = View.VISIBLE
 
             commentorImage.setOnClickListener {
                 NavigationHelper.navigateChannel(root.context, comment.commentorUrl)
             }
 
             repliesRecView.layoutManager = LinearLayoutManager(root.context)
-            val repliesAdapter = CommentsAdapter(videoId, CommentsPage().comments, true)
+            lateinit var repliesAdapter: CommentsAdapter
+            repliesAdapter = CommentsAdapter(
+                videoId,
+                mutableListOf(),
+                true
+            )
             repliesRecView.adapter = repliesAdapter
             if (!isRepliesAdapter && comment.repliesPage != null) {
                 root.setOnClickListener {
                     when {
                         repliesAdapter.itemCount.equals(0) -> {
-                            nextpage = comment.repliesPage
-                            fetchReplies(nextpage, repliesAdapter)
+                            fetchReplies(comment.repliesPage) {
+                                repliesAdapter.updateItems(it.comments)
+                                if (repliesPage.nextpage == null) {
+                                    showMore.visibility = View.GONE
+                                    return@fetchReplies
+                                }
+                                showMore.visibility = View.VISIBLE
+                                showMore.setOnClickListener {
+                                    if (repliesPage.nextpage == null) {
+                                        it.visibility = View.GONE
+                                        return@setOnClickListener
+                                    }
+                                    fetchReplies(
+                                        repliesPage.nextpage!!
+                                    ) {
+                                        repliesAdapter.updateItems(repliesPage.comments)
+                                    }
+                                }
+                            }
                         }
                         else -> repliesAdapter.clear()
                     }
@@ -106,6 +116,8 @@ class CommentsAdapter(
                 Toast.makeText(root.context, R.string.copied, Toast.LENGTH_SHORT).show()
                 true
             }
+
+            // if (isRepliesAdapter && comments)
         }
     }
 
@@ -113,19 +125,17 @@ class CommentsAdapter(
         return comments.size
     }
 
-    private fun fetchReplies(nextPage: String, repliesAdapter: CommentsAdapter) {
+    private fun fetchReplies(nextPage: String, onFinished: (CommentsPage) -> Unit) {
         CoroutineScope(Dispatchers.Main).launch {
             if (isLoading) return@launch
             isLoading = true
-            try {
-                repliesPage = RetrofitInstance.api.getCommentsNextPage(videoId, nextPage)
-            } catch (e: IOException) {
-                println(e)
+            repliesPage = try {
+                RetrofitInstance.api.getCommentsNextPage(videoId, nextPage)
+            } catch (e: Exception) {
                 Log.e(TAG(), "IOException, you might not have internet connection")
-            } catch (e: HttpException) {
-                Log.e(TAG(), "HttpException, unexpected response," + e.response())
+                return@launch
             }
-            repliesAdapter.updateItems(repliesPage.comments)
+            onFinished.invoke(repliesPage)
             isLoading = false
         }
     }
diff --git a/app/src/main/res/layout/comments_row.xml b/app/src/main/res/layout/comments_row.xml
index 68a0498f4..700936571 100644
--- a/app/src/main/res/layout/comments_row.xml
+++ b/app/src/main/res/layout/comments_row.xml
@@ -127,5 +127,19 @@
             android:background="@null"
             android:nestedScrollingEnabled="false" />
 
+        <com.google.android.material.button.MaterialButton
+            android:visibility="gone"
+            android:id="@+id/show_more"
+            style="@style/Widget.Material3.Button.ElevatedButton"
+            android:layout_width="wrap_content"
+            android:layout_height="wrap_content"
+            android:layout_gravity="center"
+            android:layout_marginVertical="8dp"
+            android:stateListAnimator="@null"
+            android:text="@string/show_more"
+            android:textColor="?android:attr/textColorPrimary"
+            android:textSize="12sp"
+            app:cornerRadius="20dp" />
+
     </LinearLayout>
 </androidx.constraintlayout.widget.ConstraintLayout>
diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml
index 2cb2e2f4f..01f25c24f 100644
--- a/app/src/main/res/values/strings.xml
+++ b/app/src/main/res/values/strings.xml
@@ -345,6 +345,7 @@
     <string name="playlistNameReversed">Playlist name (reversed)</string>
     <string name="recentlyUpdated">Recently updated</string>
     <string name="recentlyUpdatedReversed">Recently updated (reversed)</string>
+    <string name="show_more">Show more</string>
 
     <!-- Notification channel strings -->
     <string name="download_channel_name">Download Service</string>