RS Post Settings: Add search to author dialog

WordPress/src/main/java/org/wordpress/android/ui/postsrs/PostRsListActivity.kt

@@ -1,6 +1,5 @@
 package org.wordpress.android.ui.postsrs
 
-import android.app.Activity
 import android.content.Context
 import android.content.Intent
 import android.os.Bundle
@@ -33,7 +32,7 @@ class PostRsListActivity : BaseAppCompatActivity() {
     private val settingsLauncher = registerForActivityResult(
         ActivityResultContracts.StartActivityForResult()
     ) { result ->
-        if (result.resultCode == Activity.RESULT_OK) {
+        if (result.resultCode == RESULT_OK) {
             viewModel.refreshAllTabs()
         }
     }

WordPress/src/main/java/org/wordpress/android/ui/postsrs/PostRsSettingsActivity.kt

@@ -52,16 +52,7 @@ class PostRsSettingsActivity : BaseAppCompatActivity() {
         super.onCreate(savedInstanceState)
         registerMediaPickerLauncher()
         registerTermSelectionLaunchers()
-
-        WindowInsetsControllerCompat(
-            window, window.decorView
-        ).apply {
-            hide(WindowInsetsCompat.Type.statusBars())
-            systemBarsBehavior =
-                WindowInsetsControllerCompat
-                    .BEHAVIOR_SHOW_TRANSIENT_BARS_BY_SWIPE
-        }
-
+        hideStatusBar()
         observeEvents()
 
         setContent {
@@ -101,6 +92,8 @@ class PostRsSettingsActivity : BaseAppCompatActivity() {
                     onFeaturedImageRemoved =
                         viewModel::onFeaturedImageRemoved,
                     onLoadMoreAuthors = viewModel::loadMoreAuthors,
+                    onAuthorSearchQueryChanged =
+                        viewModel::onAuthorSearchQueryChanged,
                     onSaveClicked = viewModel::onSaveClicked,
                     onDismissDialog = viewModel::onDismissDialog,
                     onDiscardConfirmed = viewModel::onDiscardConfirmed,
@@ -109,6 +102,17 @@ class PostRsSettingsActivity : BaseAppCompatActivity() {
         }
     }
 
+    private fun hideStatusBar() {
+        WindowInsetsControllerCompat(
+            window, window.decorView
+        ).apply {
+            hide(WindowInsetsCompat.Type.statusBars())
+            systemBarsBehavior =
+                WindowInsetsControllerCompat
+                    .BEHAVIOR_SHOW_TRANSIENT_BARS_BY_SWIPE
+        }
+    }
+
     private fun observeEvents() {
         lifecycleScope.launch {
             repeatOnLifecycle(Lifecycle.State.STARTED) {

WordPress/src/main/java/org/wordpress/android/ui/postsrs/PostRsSettingsUiState.kt

@@ -27,6 +27,8 @@ data class PostRsSettingsUiState(
     val siteAuthors: List<AuthorInfo> = emptyList(),
     val isLoadingMoreAuthors: Boolean = false,
     val canLoadMoreAuthors: Boolean = false,
+    val authorSearchQuery: String = "",
+    val isSearchingAuthors: Boolean = false,
     val canEditAuthor: Boolean = false,
     val password: String? = null,
     val authorName: FieldState = FieldState.Empty,

WordPress/src/main/java/org/wordpress/android/ui/postsrs/PostRsSettingsViewModel.kt

@@ -7,7 +7,9 @@ import androidx.lifecycle.viewModelScope
 import dagger.hilt.android.lifecycle.HiltViewModel
 import kotlinx.coroutines.CancellationException
 import kotlinx.coroutines.Dispatchers
+import kotlinx.coroutines.Job
 import kotlinx.coroutines.channels.Channel
+import kotlinx.coroutines.delay
 import kotlinx.coroutines.flow.MutableStateFlow
 import kotlinx.coroutines.flow.StateFlow
 import kotlinx.coroutines.flow.asStateFlow
@@ -20,6 +22,7 @@ import org.wordpress.android.fluxc.model.SiteModel
 import org.wordpress.android.fluxc.network.rest.wpapi.rs.WpApiClientProvider
 import org.wordpress.android.ui.mysite.SelectedSiteRepository
 import org.wordpress.android.ui.postsrs.data.PostRsRestClient
+import org.wordpress.android.ui.postsrs.data.PostRsRestClient.Companion.AUTHORS_PER_PAGE
 import org.wordpress.android.util.AppLog
 import org.wordpress.android.util.NetworkUtilsWrapper
 import org.wordpress.android.viewmodel.ResourceProvider
@@ -77,6 +80,7 @@ class PostRsSettingsViewModel @Inject constructor(
 
     private var lastPost: AnyPostWithEditContext? = null
     private var nextAuthorPageParams: UserListParams? = null
+    private var authorSearchJob: Job? = null
 
     init {
         if (site == null) {
@@ -357,6 +361,8 @@ class PostRsSettingsViewModel @Inject constructor(
         val current = _uiState.value
         val authorName = current.siteAuthors
             .firstOrNull { it.id == authorId }?.name
+        authorSearchJob?.cancel()
+        nextAuthorPageParams = null
         _uiState.update {
             it.copy(
                 editedAuthor = authorId.takeIf { ea ->
@@ -367,7 +373,11 @@ class PostRsSettingsViewModel @Inject constructor(
                 } else {
                     it.authorName
                 },
-                dialogState = DialogState.None
+                dialogState = DialogState.None,
+                authorSearchQuery = "",
+                isSearchingAuthors = false,
+                siteAuthors = emptyList(),
+                canLoadMoreAuthors = false,
             )
         }
     }
@@ -663,9 +673,84 @@ class PostRsSettingsViewModel @Inject constructor(
         }
     }
 
+    fun onAuthorSearchQueryChanged(query: String) {
+        _uiState.update {
+            it.copy(authorSearchQuery = query)
+        }
+        authorSearchJob?.cancel()
+        val trimmed = query.trim()
+        if (trimmed.isNotEmpty() &&
+            trimmed.length < MIN_AUTHOR_SEARCH_LENGTH
+        ) {
+            return
+        }
+        authorSearchJob = viewModelScope.launch {
+            delay(AUTHOR_SEARCH_DEBOUNCE_MS)
+            _uiState.update {
+                it.copy(isSearchingAuthors = true)
+            }
+            searchAuthors(trimmed)
+        }
+    }
+
+    @Suppress("TooGenericExceptionCaught")
+    private suspend fun searchAuthors(query: String) {
+        val currentSite = site ?: return
+        try {
+            val params = UserListParams(
+                search = query.ifEmpty { null },
+                perPage = AUTHORS_PER_PAGE,
+            )
+            val page = withContext(Dispatchers.IO) {
+                restClient.fetchSiteAuthors(
+                    currentSite, params
+                )
+            }
+            nextAuthorPageParams = page.nextPageParams
+            _uiState.update {
+                it.copy(
+                    siteAuthors = page.authors,
+                    isSearchingAuthors = false,
+                    canLoadMoreAuthors =
+                        page.nextPageParams != null,
+                )
+            }
+        } catch (e: CancellationException) {
+            throw e
+        } catch (e: Exception) {
+            AppLog.e(
+                AppLog.T.POSTS,
+                "Failed to search authors",
+                e
+            )
+            _uiState.update {
+                it.copy(isSearchingAuthors = false)
+            }
+            _snackbarMessages.trySend(
+                SnackbarMessage(
+                    PostRsErrorUtils.friendlyErrorMessage(
+                        e = e,
+                        resourceProvider =
+                            resourceProvider,
+                        networkUtilsWrapper =
+                            networkUtilsWrapper,
+                    )
+                )
+            )
+        }
+    }
+
     fun onDismissDialog() {
+        authorSearchJob?.cancel()
+        nextAuthorPageParams = null
         _uiState.update {
-            it.copy(dialogState = DialogState.None)
+            it.copy(
+                dialogState = DialogState.None,
+                authorSearchQuery = "",
+                isSearchingAuthors = false,
+                siteAuthors = emptyList(),
+                canLoadMoreAuthors = false,
+            )
         }
     }
 
@@ -1055,6 +1140,8 @@ class PostRsSettingsViewModel @Inject constructor(
 
     companion object {
         const val EXTRA_POST_ID = "extra_post_id"
+        private const val AUTHOR_SEARCH_DEBOUNCE_MS = 500L
+        private const val MIN_AUTHOR_SEARCH_LENGTH = 3
         private const val ERROR_NO_SITE =
             "No site selected"
     }

WordPress/src/main/java/org/wordpress/android/ui/postsrs/data/PostRsRestClient.kt

@@ -361,7 +361,7 @@ class PostRsRestClient @Inject constructor(
         }
 
     companion object {
-        private const val AUTHORS_PER_PAGE: UInt = 20u
+        internal const val AUTHORS_PER_PAGE: UInt = 20u
         private const val PER_PAGE = 100u
     }
 }