prehistoric-systems/omnivore
1
0
Fork 0
mirror of https://github.com/omnivore-app/omnivore.git synced 2026-03-11 08:54:26 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Always sort labels when presenting them

Browse source
This commit is contained in:
Jackson Harper 2022-12-22 15:46:39 +08:00
parent eb026f2bfb
commit 9e35fee17c
1 changed files with 8 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
apple/OmnivoreKit/Sources/App/Views/Labels/LabelsViewModel.swift
View file

@ -12,6 +12,10 @@ import Views
@Published var showCreateLabelModal = false
@Published var labelSearchFilter = ""
func setLabels(_ labels: [LinkedItemLabel]) {
self.labels = labels.sorted { $0.unwrappedName.trimmingCharacters(in: .whitespaces) < $1.unwrappedName.trimmingCharacters(in: .whitespaces) }
}
func loadLabels(
dataService: DataService,
item: LinkedItem? = nil,
@ -21,7 +25,7 @@ import Views
if let labelIDs = try? await dataService.labels() {
dataService.viewContext.performAndWait {
self.labels = labelIDs.compactMap { dataService.viewContext.object(with: $0) as? LinkedItemLabel }
setLabels(labelIDs.compactMap { dataService.viewContext.object(with: $0) as? LinkedItemLabel })
}
let selLabels = initiallySelectedLabels ?? item?.sortedLabels ?? []
for label in labels {
@ -44,7 +48,7 @@ import Views
if let labelIDs = try? await dataService.labels() {
dataService.viewContext.performAndWait {
self.labels = labelIDs.compactMap { dataService.viewContext.object(with: $0) as? LinkedItemLabel }
setLabels(labelIDs.compactMap { dataService.viewContext.object(with: $0) as? LinkedItemLabel })
}
let selLabels = highlight.labels ?? []
for label in labels {
@ -69,7 +73,7 @@ import Views
if fetchedLabels?.count == 0 {
await fetchLabelsFromNetwork(dataService: dataService)
} else {
labels = fetchedLabels ?? []
setLabels(fetchedLabels ?? [])
unselectedLabels = fetchedLabels ?? []
}
}
@ -82,7 +86,7 @@ import Views
labelIDs.compactMap { dataService.viewContext.object(with: $0) as? LinkedItemLabel }
}
labels = fetchedLabels
setLabels(fetchedLabels)
unselectedLabels = fetchedLabels
}