From 285bf9d92944c49c36d1828bc388b12a97525b1a Mon Sep 17 00:00:00 2001 From: Harsh Shandilya Date: Sat, 15 May 2021 12:53:31 +0530 Subject: Move file reads in RV adapters to a background dispatcher Signed-off-by: Harsh Shandilya --- .../aps/ui/adapters/PasswordItemRecyclerAdapter.kt | 14 ++++++++++---- 1 file changed, 10 insertions(+), 4 deletions(-) (limited to 'app/src/main/java/dev/msfjarvis/aps/ui/adapters') diff --git a/app/src/main/java/dev/msfjarvis/aps/ui/adapters/PasswordItemRecyclerAdapter.kt b/app/src/main/java/dev/msfjarvis/aps/ui/adapters/PasswordItemRecyclerAdapter.kt index a551a8b1..be0267c4 100644 --- a/app/src/main/java/dev/msfjarvis/aps/ui/adapters/PasswordItemRecyclerAdapter.kt +++ b/app/src/main/java/dev/msfjarvis/aps/ui/adapters/PasswordItemRecyclerAdapter.kt @@ -17,12 +17,16 @@ import dev.msfjarvis.aps.R import dev.msfjarvis.aps.data.password.PasswordItem import dev.msfjarvis.aps.util.viewmodel.SearchableRepositoryAdapter import dev.msfjarvis.aps.util.viewmodel.stableId +import kotlinx.coroutines.CoroutineScope +import kotlinx.coroutines.Dispatchers +import kotlinx.coroutines.withContext -open class PasswordItemRecyclerAdapter : +open class PasswordItemRecyclerAdapter(coroutineScope: CoroutineScope) : SearchableRepositoryAdapter( R.layout.password_row_layout, ::PasswordItemViewHolder, - PasswordItemViewHolder::bind + coroutineScope, + PasswordItemViewHolder::bind, ) { fun makeSelectable(recyclerView: RecyclerView) { @@ -48,7 +52,7 @@ open class PasswordItemRecyclerAdapter : private val folderIndicator: AppCompatImageView = itemView.findViewById(R.id.folder_indicator) lateinit var itemDetails: ItemDetailsLookup.ItemDetails - fun bind(item: PasswordItem) { + suspend fun bind(item: PasswordItem) { val parentPath = item.fullPathToParent.replace("(^/)|(/$)".toRegex(), "") val source = if (parentPath.isNotEmpty()) { @@ -62,7 +66,9 @@ open class PasswordItemRecyclerAdapter : if (item.type == PasswordItem.TYPE_CATEGORY) { folderIndicator.visibility = View.VISIBLE val count = - item.file.listFiles { path -> path.isDirectory || path.extension == "gpg" }?.size ?: 0 + withContext(Dispatchers.IO) { + item.file.listFiles { path -> path.isDirectory || path.extension == "gpg" }?.size ?: 0 + } childCount.visibility = if (count > 0) View.VISIBLE else View.GONE childCount.text = "$count" } else { -- cgit v1.2.3