From 512fdd8e90a3e9221f85bb5d2d291ab5356f294f Mon Sep 17 00:00:00 2001
From: Bnyro <yannick.fehler@web.de>
Date: Wed, 8 Jun 2022 18:29:00 +0200
Subject: [PATCH] functionality finished

---
 .../libretube/adapters/CommentsAdapter.kt     | 31 ++++++++++---------
 .../libretube/adapters/RepliesAdapter.kt      |  6 ++++
 app/src/main/res/layout/comments_row.xml      |  7 ++---
 3 files changed, 25 insertions(+), 19 deletions(-)

diff --git a/app/src/main/java/com/github/libretube/adapters/CommentsAdapter.kt b/app/src/main/java/com/github/libretube/adapters/CommentsAdapter.kt
index e35984299..52e1b9070 100644
--- a/app/src/main/java/com/github/libretube/adapters/CommentsAdapter.kt
+++ b/app/src/main/java/com/github/libretube/adapters/CommentsAdapter.kt
@@ -8,6 +8,7 @@ import android.widget.ImageView
 import android.widget.TextView
 import androidx.constraintlayout.motion.widget.MotionLayout
 import androidx.core.os.bundleOf
+import androidx.recyclerview.widget.LinearLayoutManager
 import androidx.recyclerview.widget.RecyclerView
 import com.github.libretube.MainActivity
 import com.github.libretube.R
@@ -16,11 +17,11 @@ import com.github.libretube.obj.Comment
 import com.github.libretube.obj.CommentsPage
 import com.github.libretube.util.RetrofitInstance
 import com.squareup.picasso.Picasso
+import java.io.IOException
 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,
@@ -31,6 +32,7 @@ class CommentsAdapter(
     private var isLoading = false
     private var nextPage = ""
     private lateinit var repliesRecView: RecyclerView
+    private lateinit var repliesPage: CommentsPage
 
     fun updateItems(newItems: List<Comment>) {
         var commentsSize = comments.size
@@ -77,15 +79,17 @@ class CommentsAdapter(
             }
         }
         repliesRecView = holder.v.findViewById(R.id.replies_recView)
-        // repliesRecView.layoutManager = LinearLayoutManager(holder.v.context)
+        repliesRecView.layoutManager = LinearLayoutManager(holder.v.context)
+        val repliesAdapter = RepliesAdapter(CommentsPage().comments)
+        repliesRecView.adapter = repliesAdapter
         holder.v.setOnClickListener {
-            if (repliesRecView.visibility == View.GONE) {
-                repliesRecView.visibility = View.VISIBLE
+            Log.e("clicekd", "clicked")
+            if (repliesAdapter.itemCount == 0) {
                 nextPage = comments[position].repliesPage!!
-                repliesRecView.adapter = CommentsAdapter(videoId, comments)
-                // fetchReplies()
+                fetchReplies(repliesAdapter)
+                // repliesAdapter.updateItems(repliesPage.comments)
             } else {
-                repliesRecView.visibility = View.GONE
+                repliesAdapter.clear()
             }
         }
     }
@@ -94,23 +98,20 @@ class CommentsAdapter(
         return comments.size
     }
 
-    private fun fetchReplies() {
-        CoroutineScope(Dispatchers.IO).launch {
+    private fun fetchReplies(repliesAdapter: RepliesAdapter) {
+        CoroutineScope(Dispatchers.Main).launch {
             if (!isLoading && nextPage != null) {
-                var response = CommentsPage()
                 isLoading = true
                 try {
-                    response = RetrofitInstance.api.getCommentsNextPage(videoId!!, nextPage!!)
+                    repliesPage = RetrofitInstance.api.getCommentsNextPage(videoId!!, nextPage!!)
                 } catch (e: IOException) {
                     println(e)
                     Log.e(TAG, "IOException, you might not have internet connection")
                 } catch (e: HttpException) {
                     Log.e(TAG, "HttpException, unexpected response," + e.response())
                 }
-                nextPage = if (response.nextpage != null) response.nextpage!!
-                else ""
-                // commentsAdapter?.updateItems(response.comments)
-                repliesRecView.adapter = RepliesAdapter(response.comments)
+                // nextPage = if (repliesPage.nextpage!! != null) repliesPage.nextpage!! else ""
+                repliesAdapter.updateItems(repliesPage.comments)
                 isLoading = false
             }
         }
diff --git a/app/src/main/java/com/github/libretube/adapters/RepliesAdapter.kt b/app/src/main/java/com/github/libretube/adapters/RepliesAdapter.kt
index 261c890b3..de06907ae 100644
--- a/app/src/main/java/com/github/libretube/adapters/RepliesAdapter.kt
+++ b/app/src/main/java/com/github/libretube/adapters/RepliesAdapter.kt
@@ -22,6 +22,12 @@ class RepliesAdapter(
     private var isLoading = false
     private var nextPage = ""
 
+    fun clear() {
+        val size: Int = replies.size
+        replies.clear()
+        notifyItemRangeRemoved(0, size)
+    }
+
     fun updateItems(newItems: List<Comment>) {
         var repliesSize = replies.size
         replies.addAll(newItems)
diff --git a/app/src/main/res/layout/comments_row.xml b/app/src/main/res/layout/comments_row.xml
index 97b0471f9..295108ce1 100644
--- a/app/src/main/res/layout/comments_row.xml
+++ b/app/src/main/res/layout/comments_row.xml
@@ -105,10 +105,9 @@
         android:id="@+id/replies_recView"
         android:layout_width="match_parent"
         android:layout_height="wrap_content"
-        android:layout_marginTop="20dp"
-        android:layout_marginLeft="20dp"
-        android:nestedScrollingEnabled="false"
-        android:visibility="gone" />
+        android:layout_marginLeft="16dp"
+        android:layout_marginBottom="10dp"
+        android:nestedScrollingEnabled="false" />
 
     </LinearLayout>
 </androidx.constraintlayout.widget.ConstraintLayout>