Merge pull request #14604 from woocommerce/woomob-1146-woo-poshistorical-orders-orders-searching

[WOOMOB-1146][Woo POS][Orders] Add search functionality to Woo POS orders

Author: kidinov

WooCommerce/src/main/kotlin/com/woocommerce/android/ui/woopos/orders/WooPosOrdersDataSource.kt

@@ -16,6 +16,11 @@ sealed class LoadOrdersResult {
     data class Error(val message: String) : LoadOrdersResult()
 }
 
+sealed class SearchOrdersResult {
+    data class Success(val orders: List<Order>) : SearchOrdersResult()
+    data class Error(val message: String) : SearchOrdersResult()
+}
+
 class WooPosOrdersDataSource @Inject constructor(
     private val restClient: OrderRestClient,
     private val selectedSite: SelectedSite,
@@ -25,31 +30,48 @@ class WooPosOrdersDataSource @Inject constructor(
     companion object {
         const val POS_ORDERS_PAGE_SIZE = 25
     }
+
     fun loadOrders(): Flow<LoadOrdersResult> = flow {
         val cached = ordersCache.getAll()
         if (cached.isNotEmpty()) {
             emit(LoadOrdersResult.SuccessCache(cached))
         }
 
-        val result = restClient.fetchOrders(
-            site = selectedSite.get(),
-            count = POS_ORDERS_PAGE_SIZE,
-            page = 1,
-            orderBy = OrderBy.DATE,
-            sortOrder = OrderRestClient.SortOrder.DESCENDING,
-            statusFilter = null,
-            createdVia = "pos-rest-api"
-        )
+        val result = fetchOrdersFromRemote(searchQuery = null, page = 1)
 
         if (result.isError) {
-            emit(LoadOrdersResult.Error(result.error.message))
+            emit(LoadOrdersResult.Error(result.error?.message ?: "Unknown error"))
         } else {
             val mapped = result.orders.toAppModels()
             ordersCache.setAll(mapped)
-            emit(LoadOrdersResult.SuccessRemote(result.orders.toAppModels()))
+            emit(LoadOrdersResult.SuccessRemote(mapped))
         }
     }
 
+    suspend fun searchOrders(searchQuery: String): SearchOrdersResult {
+        val result = fetchOrdersFromRemote(searchQuery = searchQuery, page = 1)
+
+        return if (result.isError) {
+            SearchOrdersResult.Error(result.error?.message ?: "Unknown error")
+        } else {
+            SearchOrdersResult.Success(result.orders.toAppModels())
+        }
+    }
+
+    private suspend fun fetchOrdersFromRemote(
+        page: Int,
+        searchQuery: String?
+    ) = restClient.fetchOrders(
+        site = selectedSite.get(),
+        count = POS_ORDERS_PAGE_SIZE,
+        page = page,
+        orderBy = OrderBy.DATE,
+        sortOrder = OrderRestClient.SortOrder.DESCENDING,
+        statusFilter = null,
+        createdVia = "pos-rest-api",
+        searchQuery = searchQuery,
+    )
+
     fun clearCache() = ordersCache.clear()
 
     private suspend fun List<OrderEntity>.toAppModels(): List<Order> = map {

WooCommerce/src/main/kotlin/com/woocommerce/android/ui/woopos/orders/WooPosOrdersScreen.kt

@@ -12,7 +12,10 @@ import androidx.compose.foundation.layout.Spacer
 import androidx.compose.foundation.layout.fillMaxHeight
 import androidx.compose.foundation.layout.fillMaxSize
 import androidx.compose.foundation.layout.fillMaxWidth
+import androidx.compose.foundation.layout.height
+import androidx.compose.foundation.layout.heightIn
 import androidx.compose.foundation.layout.padding
+import androidx.compose.foundation.layout.statusBarsPadding
 import androidx.compose.foundation.lazy.LazyColumn
 import androidx.compose.foundation.lazy.items
 import androidx.compose.material.ExperimentalMaterialApi
@@ -30,9 +33,14 @@ import androidx.compose.ui.res.stringResource
 import androidx.compose.ui.semantics.selected
 import androidx.compose.ui.semantics.semantics
 import androidx.compose.ui.text.font.FontWeight
+import androidx.compose.ui.unit.dp
+import androidx.constraintlayout.compose.ConstraintLayout
 import androidx.hilt.navigation.compose.hiltViewModel
 import com.woocommerce.android.R
 import com.woocommerce.android.ui.woopos.common.composeui.WooPosPreview
+import com.woocommerce.android.ui.woopos.common.composeui.component.WooPosSearchInput
+import com.woocommerce.android.ui.woopos.common.composeui.component.WooPosSearchInputState
+import com.woocommerce.android.ui.woopos.common.composeui.component.WooPosSearchUIEvent
 import com.woocommerce.android.ui.woopos.common.composeui.component.WooPosText
 import com.woocommerce.android.ui.woopos.common.composeui.component.WooPosToolbar
 import com.woocommerce.android.ui.woopos.common.composeui.designsystem.WooPosSpacing
@@ -41,6 +49,8 @@ import com.woocommerce.android.ui.woopos.common.composeui.designsystem.WooPosTyp
 import com.woocommerce.android.ui.woopos.home.items.WooPosPullToRefreshState
 import com.woocommerce.android.ui.woopos.root.navigation.WooPosNavigationEvent
 
+private val WOO_POS_ORDERS_TOOLBAR_HEIGHT = 56.dp
+
 @OptIn(ExperimentalMaterialApi::class)
 @Composable
 fun WooPosOrdersScreen(
@@ -59,6 +69,7 @@ fun WooPosOrdersScreen(
             onRefresh = viewModel::onRefresh,
             isRefreshing = state.pullToRefreshState == WooPosPullToRefreshState.Refreshing,
             onOrderSelected = viewModel::onOrderSelected,
+            onSearchEvent = viewModel::onSearchEvent,
             modifier = Modifier
                 .weight(0.3f)
                 .fillMaxHeight()
@@ -83,13 +94,48 @@ private fun OrdersList(
     onRefresh: () -> Unit,
     isRefreshing: Boolean,
     onOrderSelected: (Long) -> Unit,
+    onSearchEvent: (WooPosSearchUIEvent) -> Unit,
     modifier: Modifier = Modifier
 ) {
     Column(modifier = modifier) {
-        WooPosToolbar(
-            titleText = stringResource(R.string.woopos_orders_title),
-            onBackClicked = onBackClicked,
-        )
+        ConstraintLayout(
+            modifier = Modifier
+                .fillMaxWidth()
+                .heightIn(min = WOO_POS_ORDERS_TOOLBAR_HEIGHT),
+        ) {
+            val (toolbar, searchInput) = createRefs()
+
+            if (state.searchInputState is WooPosSearchInputState.Closed) {
+                WooPosToolbar(
+                    titleText = stringResource(R.string.woopos_orders_title),
+                    onBackClicked = onBackClicked,
+                    modifier = Modifier.constrainAs(toolbar) {
+                        start.linkTo(parent.start)
+                        top.linkTo(parent.top)
+                        bottom.linkTo(parent.bottom)
+                    }
+                )
+            }
+            WooPosSearchInput(
+                state = state.searchInputState,
+                onEvent = onSearchEvent,
+                modifier = Modifier
+                    .statusBarsPadding()
+                    .constrainAs(searchInput) {
+                        if (state.searchInputState is WooPosSearchInputState.Open) {
+                            start.linkTo(parent.start)
+                            end.linkTo(parent.end)
+                            width = androidx.constraintlayout.compose.Dimension.fillToConstraints
+                        } else {
+                            end.linkTo(parent.end)
+                        }
+                        top.linkTo(parent.top)
+                        bottom.linkTo(parent.bottom)
+                    }
+            )
+        }
+
+        Spacer(modifier = Modifier.height(WooPosSpacing.Small.value))
 
         val pullRefreshState = rememberPullRefreshState(
             refreshing = isRefreshing,

WooCommerce/src/main/kotlin/com/woocommerce/android/ui/woopos/orders/WooPosOrdersState.kt

@@ -1,6 +1,7 @@
 package com.woocommerce.android.ui.woopos.orders
 
 import androidx.compose.runtime.Immutable
+import com.woocommerce.android.ui.woopos.common.composeui.component.WooPosSearchInputState
 import com.woocommerce.android.ui.woopos.home.items.WooPosPaginationState
 import com.woocommerce.android.ui.woopos.home.items.WooPosPullToRefreshState
 
@@ -16,30 +17,33 @@ data class OrderItemViewState(
 @Immutable
 sealed class WooPosOrdersState {
     abstract val pullToRefreshState: WooPosPullToRefreshState
+    abstract val searchInputState: WooPosSearchInputState
 
     @Immutable
     data class Content(
         val items: List<OrderItemViewState>,
         override val pullToRefreshState: WooPosPullToRefreshState,
+        override val searchInputState: WooPosSearchInputState,
         val paginationState: WooPosPaginationState,
         val selectedOrderId: Long?
     ) : WooPosOrdersState()
 
     @Immutable
     data class Error(
         val message: String,
+        override val searchInputState: WooPosSearchInputState
     ) : WooPosOrdersState() {
         override val pullToRefreshState: WooPosPullToRefreshState = WooPosPullToRefreshState.Disabled
     }
 
     @Immutable
-    data object Loading : WooPosOrdersState() {
+    data class Loading(override val searchInputState: WooPosSearchInputState) : WooPosOrdersState() {
         override val pullToRefreshState: WooPosPullToRefreshState = WooPosPullToRefreshState.Disabled
     }
 
     @Immutable
     data class Empty(
-        override val pullToRefreshState: WooPosPullToRefreshState =
-            WooPosPullToRefreshState.Enabled
+        override val pullToRefreshState: WooPosPullToRefreshState = WooPosPullToRefreshState.Enabled,
+        override val searchInputState: WooPosSearchInputState,
     ) : WooPosOrdersState()
 }