mirror of
https://github.com/libre-tube/LibreTube.git
synced 2024-12-14 06:10:31 +05:30
allow fetching more comment replies
This commit is contained in:
parent
51b8af4606
commit
e836625223
@ -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
|
||||
}
|
||||
}
|
||||
|
@ -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>
|
||||
|
@ -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>
|
||||
|
Loading…
Reference in New Issue
Block a user