diff --git a/Modules/Sources/Browse/Logic/Models/Document.swift b/Modules/Sources/Browse/Logic/Models/Document.swift index 46df5eb..913a310 100644 --- a/Modules/Sources/Browse/Logic/Models/Document.swift +++ b/Modules/Sources/Browse/Logic/Models/Document.swift @@ -8,7 +8,7 @@ import Foundation -struct Document { +struct Document: FileSystemItem { // MARK: Properties @@ -35,7 +35,3 @@ struct Document { } } - -// MARK: - FileSystemIdIdentifiable - -extension Document: FileSystemItemIdentifiable {} diff --git a/Modules/Sources/Browse/Logic/Models/Folder.swift b/Modules/Sources/Browse/Logic/Models/Folder.swift index b089e36..63c22ea 100644 --- a/Modules/Sources/Browse/Logic/Models/Folder.swift +++ b/Modules/Sources/Browse/Logic/Models/Folder.swift @@ -6,7 +6,7 @@ // Copyright © 2022 Röck+Cöde. All rights reserved. // -public struct Folder { +public struct Folder: FileSystemItem { // MARK: Properties @@ -24,11 +24,3 @@ public struct Folder { } } - -// MARK: - FileSystemIdIdentifiable - -extension Folder: FileSystemItemIdentifiable {} - -// MARK: - Equatable - -extension Folder: Equatable {} diff --git a/Modules/Sources/Browse/Logic/Protocols/FileSystemItem.swift b/Modules/Sources/Browse/Logic/Protocols/FileSystemItem.swift index d4f4d04..69b3931 100644 --- a/Modules/Sources/Browse/Logic/Protocols/FileSystemItem.swift +++ b/Modules/Sources/Browse/Logic/Protocols/FileSystemItem.swift @@ -6,11 +6,7 @@ // Copyright © 2022 Röck+Cöde. All rights reserved. // -protocol FileSystemItem { +protocol FileSystemItem: Identifiable, Hashable, Equatable { var id: String { get } var name: String { get } } - -// MARK: - Type aliases - -typealias FileSystemItemIdentifiable = FileSystemItem & Identifiable & Hashable diff --git a/Modules/Sources/Browse/Logic/Use Cases/DeleteItemUseCase.swift b/Modules/Sources/Browse/Logic/Use Cases/DeleteItemUseCase.swift new file mode 100644 index 0000000..505353b --- /dev/null +++ b/Modules/Sources/Browse/Logic/Use Cases/DeleteItemUseCase.swift @@ -0,0 +1,45 @@ +// +// DeleteItemUseCase.swift +// Browse +// +// Created by Javier Cicchelli on 16/12/2022. +// Copyright © 2022 Röck+Cöde. All rights reserved. +// + +import APIService +import DependencyInjection +import Dependencies + +struct DeleteItemUseCase { + + // MARK: Properties + + let apiService: APIService + + // MARK: Functions + + func callAsFunction( + id: String, + username: String, + password: String + ) async throws { + try await apiService.deleteItem( + id: id, + credentials: .init( + username: username, + password: password + ) + ) + } + +} + +// MARK: - Initialisers + +extension DeleteItemUseCase { + init() { + @Dependency(\.apiService) var apiService + + self.init(apiService: apiService) + } +} diff --git a/Modules/Sources/Browse/Logic/Use Cases/GetDataUseCase.swift b/Modules/Sources/Browse/Logic/Use Cases/GetDataUseCase.swift index 5b87e28..daee2de 100644 --- a/Modules/Sources/Browse/Logic/Use Cases/GetDataUseCase.swift +++ b/Modules/Sources/Browse/Logic/Use Cases/GetDataUseCase.swift @@ -13,9 +13,9 @@ import Foundation struct GetDataUseCase { - // MARK: Dependencies + // MARK: Properties - @Dependency(\.apiService) private var apiService + let apiService: APIService // MARK: Functions @@ -34,3 +34,13 @@ struct GetDataUseCase { } } + +// MARK: - Initialisers + +extension GetDataUseCase { + init() { + @Dependency(\.apiService) var apiService + + self.init(apiService: apiService) + } +} diff --git a/Modules/Sources/Browse/Logic/Use Cases/GetItemsUseCase.swift b/Modules/Sources/Browse/Logic/Use Cases/GetItemsUseCase.swift index bbd219f..281b6dc 100644 --- a/Modules/Sources/Browse/Logic/Use Cases/GetItemsUseCase.swift +++ b/Modules/Sources/Browse/Logic/Use Cases/GetItemsUseCase.swift @@ -12,9 +12,9 @@ import Dependencies struct GetItemsUseCase { - // MARK: Dependencies + // MARK: Properties - @Dependency(\.apiService) private var apiService + let apiService: APIService // MARK: Functions @@ -22,7 +22,7 @@ struct GetItemsUseCase { id: String, username: String, password: String - ) async throws -> [any FileSystemItemIdentifiable] { + ) async throws -> [any FileSystemItem] { let items = try await apiService.getItems( id: id, credentials: .init( @@ -32,7 +32,7 @@ struct GetItemsUseCase { ) return items - .compactMap { item -> any FileSystemItemIdentifiable in + .compactMap { item -> any FileSystemItem in if item.isDirectory { return Folder( id: item.id, @@ -51,3 +51,13 @@ struct GetItemsUseCase { } } + +// MARK: - Initialisers + +extension GetItemsUseCase { + init() { + @Dependency(\.apiService) var apiService + + self.init(apiService: apiService) + } +} diff --git a/Modules/Sources/Browse/Resources/en.lproj/Localizable.strings b/Modules/Sources/Browse/Resources/en.lproj/Localizable.strings index 6cef50f..5336bc7 100644 --- a/Modules/Sources/Browse/Resources/en.lproj/Localizable.strings +++ b/Modules/Sources/Browse/Resources/en.lproj/Localizable.strings @@ -25,6 +25,16 @@ "message.type_not_supported.text.second" = "Please be patient while the support for this type of document is being built by our development team."; "message.type_not_supported.button.text" = "Go back to folder"; +// DeleteItemViewModifier + +"delete_item.action_sheet.title" = "Delete an item"; +"delete_item.action_sheet.message %@" = "You are about to delete an item named \"%@\" from this folder.\n\nAre you sure you wish to proceed?"; +"delete_item.action_sheet.button.ok" = "Yes, please delete it."; +"delete_item.action_sheet.button.cancel" = "No, I reconsidered."; +"delete_item.system_alert.title" = "..."; +"delete_item.system_alert.message" = "..."; +"delete_item.system_alert.button.dismiss" = "..."; + // BrowseView "browse.toolbar_item.menu.add_actions.text" = "Add file and/or folder"; diff --git a/Modules/Sources/Browse/UI/Components/DocumentItem.swift b/Modules/Sources/Browse/UI/Components/DocumentItem.swift index 73b9f48..f895f2d 100644 --- a/Modules/Sources/Browse/UI/Components/DocumentItem.swift +++ b/Modules/Sources/Browse/UI/Components/DocumentItem.swift @@ -13,7 +13,7 @@ struct DocumentItem: View { // MARK: Properties - let item: FileSystemItem + let item: any FileSystemItem let select: ActionClosure let download: ActionClosure let delete: ActionClosure diff --git a/Modules/Sources/Browse/UI/Components/FolderItem.swift b/Modules/Sources/Browse/UI/Components/FolderItem.swift index 40733f4..fc1811b 100644 --- a/Modules/Sources/Browse/UI/Components/FolderItem.swift +++ b/Modules/Sources/Browse/UI/Components/FolderItem.swift @@ -13,7 +13,7 @@ struct FolderItem: View { // MARK: Properties - let item: FileSystemItem + let item: any FileSystemItem let select: ActionClosure let delete: ActionClosure diff --git a/Modules/Sources/Browse/UI/Extensions/View+ViewModifiers.swift b/Modules/Sources/Browse/UI/Extensions/View+ViewModifiers.swift index 7b199c9..ce602dc 100644 --- a/Modules/Sources/Browse/UI/Extensions/View+ViewModifiers.swift +++ b/Modules/Sources/Browse/UI/Extensions/View+ViewModifiers.swift @@ -6,6 +6,7 @@ // Copyright © 2022 Röck+Cöde. All rights reserved. // +import DataModels import SwiftUI extension View { @@ -20,4 +21,14 @@ extension View { destination: { destination } )) } + + func delete( + item: Binding<(any FileSystemItem)?>, + deleted: @escaping ActionClosure + ) -> some View { + modifier(DeleteItemViewModifier( + item: item, + deleted: deleted + )) + } } diff --git a/Modules/Sources/Browse/UI/View Modifiers/DeleteItemViewModifier.swift b/Modules/Sources/Browse/UI/View Modifiers/DeleteItemViewModifier.swift new file mode 100644 index 0000000..aa1e6ab --- /dev/null +++ b/Modules/Sources/Browse/UI/View Modifiers/DeleteItemViewModifier.swift @@ -0,0 +1,131 @@ +// +// DeleteItemViewModifier.swift +// Browse +// +// Created by Javier Cicchelli on 16/12/2022. +// + +import DataModels +import KeychainStorage +import SwiftUI + +struct DeleteItemViewModifier: ViewModifier { + + // MARK: Storages + + @KeychainStorage(key: .KeychainStorage.account) private var account: Account? + + // MARK: States + + @State private var showErrorAlert: Bool = false + + // MARK: Bindings + + @Binding var item: (any FileSystemItem)? + + // MARK: Properties + + let deleted: ActionClosure + + private let deleteItem: DeleteItemUseCase = .init() + + // MARK: Body + + func body(content: Content) -> some View { + content + .actionSheet(isPresented: showDeletionConfirmation) { + ActionSheet( + title: Text( + "delete_item.action_sheet.title", + bundle: .module + ), + message: Text( + "delete_item.action_sheet.message \(itemName)", + bundle: .module + ), + buttons: [ + .destructive(Text( + "delete_item.action_sheet.button.ok", + bundle: .module + )) { + Task { + await removeItem() + } + }, + .cancel(Text( + "delete_item.action_sheet.button.cancel", + bundle: .module + )) { + item = nil + }, + ] + ) + } + .alert(isPresented: $showErrorAlert) { + Alert( + title: Text( + "delete_item.system_alert.title", + bundle: .module + ), + message: Text( + "delete_item.system_alert.message", + bundle: .module + ), + dismissButton: .cancel(Text( + "delete_item.system_alert.button.cancel", + bundle: .module + )) { + item = nil + } + ) + } + } + +} + +// MARK: - Helpers + +private extension DeleteItemViewModifier { + + // MARK: Computed + + var itemName: String { + item?.name ?? .Constants.noName + } + + var showDeletionConfirmation: Binding { + .init { item != nil } set: { _ in } + } + + // MARK: Functions + + func removeItem() async { + guard + let id = item?.id, + let account + else { + showErrorAlert = true + return + } + + do { + try await deleteItem( + id: id, + username: account.username, + password: account.password + ) + + deleted() + } catch { + showErrorAlert = true + } + } +} + +// MARK: - String+Constants + +private extension String { + enum Constants { + static let noName = "no name" + } +} diff --git a/Modules/Sources/Browse/UI/Views/BrowseView.swift b/Modules/Sources/Browse/UI/Views/BrowseView.swift index 74821b9..067521c 100644 --- a/Modules/Sources/Browse/UI/Views/BrowseView.swift +++ b/Modules/Sources/Browse/UI/Views/BrowseView.swift @@ -19,8 +19,9 @@ public struct BrowseView: View { // MARK: States @State private var status: ViewStatus = .loading - @State private var items: [any FileSystemItemIdentifiable] = [] + @State private var items: [any FileSystemItem] = [] @State private var stack: Stack? + @State private var itemToDelete: (any FileSystemItem)? // MARK: Properties @@ -31,7 +32,7 @@ public struct BrowseView: View { private let login: ActionClosure private let getItems: GetItemsUseCase = .init() - + // MARK: Initialisers public init( @@ -60,6 +61,11 @@ public struct BrowseView: View { showProfile: showProfile ) } + .delete(item: $itemToDelete) { + Task { + await updateItems() + } + } .task(id: folder) { await loadItems() } @@ -112,13 +118,13 @@ private extension BrowseView { // MARK: Functions @ViewBuilder func makeFolderItem( - for item: any FileSystemItemIdentifiable + for item: any FileSystemItem ) -> some View { if let folder = item as? Folder { FolderItem(item: item) { stack = .browse(folder) } delete: { - // TODO: delete the item id from the backend. + itemToDelete = item } .navigate( to: BrowseView( @@ -137,7 +143,7 @@ private extension BrowseView { } @ViewBuilder func makeDocumentItem( - for item: any FileSystemItemIdentifiable + for item: any FileSystemItem ) -> some View { if let document = item as? Document { DocumentItem(item: item) { @@ -145,7 +151,7 @@ private extension BrowseView { } download: { // TODO: download the item id from the backend. } delete: { - // TODO: delete the item id from the backend. + itemToDelete = item } .navigate( to: DocumentView( @@ -190,6 +196,11 @@ private extension BrowseView { status = .error } } + + func updateItems() async { + items = items.filter { $0.id != itemToDelete?.id } + itemToDelete = nil + } } // MARK: - Previews