Skip to content

Commit 6d7dfa9

Browse files
committed
Invert fetch logic, make products first case
1 parent 9f5c228 commit 6d7dfa9

File tree

3 files changed

+7
-7
lines changed

3 files changed

+7
-7
lines changed

WooCommerce/Classes/POS/Controllers/PointOfSaleItemsController.swift

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -206,9 +206,9 @@ private extension PointOfSaleItemsController {
206206
@MainActor
207207
func fetchItems(pageNumber: Int, appendToExistingItems: Bool = true) async throws -> Bool {
208208
do {
209-
let pagedItems = itemType == .coupons
210-
? try itemProvider.providePointOfSaleCoupons()
211-
: try await itemProvider.providePointOfSaleItems(pageNumber: pageNumber)
209+
let pagedItems = itemType == .products
210+
? try await itemProvider.providePointOfSaleItems(pageNumber: pageNumber)
211+
: try itemProvider.providePointOfSaleCoupons()
212212

213213
let newItems = pagedItems.items
214214
var allItems = appendToExistingItems ? itemsViewState.itemsStack.root.items : []

WooCommerce/Classes/POS/Models/PointOfSaleAggregateModel.swift

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -93,9 +93,10 @@ protocol PointOfSaleAggregateModelProtocol {
9393
@available(iOS 17.0, *)
9494
extension PointOfSaleAggregateModel {
9595
@MainActor
96-
func toggleItemType() async {
97-
await itemsController.toggleItemType()
98-
}
96+
func toggleItemType() async {
97+
await itemsController.toggleItemType()
98+
}
99+
99100
@MainActor
100101
func loadItems(base: ItemListBaseItem) async {
101102
await itemsController.loadItems(base: base)

WooCommerce/Classes/POS/Presentation/ItemListView.swift

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
import SwiftUI
22
import enum Yosemite.POSItem
33
import protocol Yosemite.POSOrderableItem
4-
54
import struct Yosemite.POSCoupon
65

76
@available(iOS 17.0, *)

0 commit comments

Comments
 (0)