Updated recipe editing user interface

Nextcloud Cookbook iOS Client/Data/DataModels.swift

@@ -19,7 +19,7 @@ extension Category: Identifiable, Hashable {
 
 struct Recipe: Codable {
     let name: String
-    let keywords: String
+    let keywords: String?
     let dateCreated: String
     let dateModified: String
     let imageUrl: String

Nextcloud Cookbook iOS Client/Data/UserDefaults.swift

@@ -34,10 +34,17 @@ class UserSettings: ObservableObject {
         }
     }
     
+    @Published var defaultCategory: String {
+        didSet {
+            UserDefaults.standard.set(defaultCategory, forKey: "defaultCategory")
+        }
+    }
+    
     init() {
         self.username = UserDefaults.standard.object(forKey: "username") as? String ?? ""
         self.token = UserDefaults.standard.object(forKey: "token") as? String ?? ""
         self.serverAddress = UserDefaults.standard.object(forKey: "serverAddress") as? String ?? ""
         self.onboarding = UserDefaults.standard.object(forKey: "onboarding") as? Bool ?? true
+        self.defaultCategory = UserDefaults.standard.object(forKey: "defaultCategory") as? String ?? ""
     }
 }

Nextcloud Cookbook iOS Client/ViewModels/MainViewModel.swift

@@ -47,13 +47,15 @@ import SwiftUI
     }
     
     /// Try to load the recipe list from store or the server.
+    /// - Warning: The category named '\*' is translated into '\_' for network calls and storage requests in this function. This is necessary for the nextcloud cookbook api.
     /// - Parameters
     ///     - categoryName: The name of the category containing the requested list of recipes.
     ///     - needsUpdate: If true, the recipe will be loaded from the server directly, otherwise it will be loaded from store first.
     func loadRecipeList(categoryName: String, needsUpdate: Bool = false) async {
+        let categoryString = categoryName == "*" ? "_" : categoryName
         if let recipeList: [Recipe] = await loadObject(
-            localPath: "category_\(categoryName).data",
+            localPath: "category_\(categoryString).data",
-            networkPath: .RECIPE_LIST(categoryName: categoryName),
+            networkPath: .RECIPE_LIST(categoryName: categoryString),
             needsUpdate: needsUpdate
         ) {
             recipes[categoryName] = recipeList

Nextcloud Cookbook iOS Client/Views/CategoryDetailView.swift

@@ -31,6 +31,7 @@ struct CategoryDetailView: View {
         }
         .navigationTitle(categoryName == "*" ? "Other" : categoryName)
         .toolbar {
+            ToolbarItem(placement: .topBarLeading) {
                 Menu {
                     Button {
                         print("Downloading all recipes in category \(categoryName) ...")
@@ -46,6 +47,7 @@ struct CategoryDetailView: View {
                     Image(systemName: "ellipsis.circle")
                 }
             }
+        }
         .searchable(text: $searchText, prompt: "Search recipes")
         .task {
             await viewModel.loadRecipeList(categoryName: categoryName)

Nextcloud Cookbook iOS Client/Views/MainView.swift

@@ -23,7 +23,7 @@ struct MainView: View {
                     NavigationLink(value: category) {
                         HStack(alignment: .center) {
                             Image(systemName: "book.closed.fill")
-                            Text(category.name)
+                            Text(category.name == "*" ? "Other" : category.name)
                                 .font(.system(size: 20, weight: .light, design: .serif))
                                 .italic()
                         }.padding(7)
@@ -32,6 +32,7 @@ struct MainView: View {
             }
             .navigationTitle("Cookbooks")
             .toolbar {
+                ToolbarItem(placement: .topBarLeading) {
                     Menu {
                         Button {
                             print("Downloading all recipes ...")
@@ -57,15 +58,15 @@ struct MainView: View {
                     } label: {
                         Image(systemName: "ellipsis.circle")
                     }
-                
+                }
+                ToolbarItem(placement: .topBarTrailing) {
                     NavigationLink( destination: SettingsView(userSettings: userSettings, viewModel: viewModel)) {
                         Image(systemName: "gearshape")
                     }
                 }
-            .navigationDestination(isPresented: $showEditView) {
-                RecipeEditView(viewModel: viewModel, isPresented: $showEditView)
             }
             
+            
         } detail: {
             NavigationStack {
                 if let category = selectedCategory {
@@ -79,12 +80,17 @@ struct MainView: View {
             
         }
         .tint(.nextcloudBlue)
+        .sheet(isPresented: $showEditView) {
+            RecipeEditView(viewModel: viewModel, isPresented: $showEditView)
+        }
         .task {
             await viewModel.loadCategoryList()
         }
         .refreshable {
             await viewModel.loadCategoryList(needsUpdate: true)
         }
+        
+        // TODO: SET DEFAULT CATEGORY
     }
 }
 

Nextcloud Cookbook iOS Client/Views/RecipeDetailView.swift

@@ -73,16 +73,20 @@ struct RecipeDetailView: View {
         .navigationTitle(showTitle ? recipe.name : "")
         .toolbar {
             if let recipeDetail = recipeDetail {
-                NavigationLink {
+                Button {
-                    RecipeEditView(viewModel: viewModel, recipe: recipeDetail, isPresented: $presentEditView, uploadNew: false).tag("RecipeEditView")
+                    presentEditView = true
                 } label: {
                     HStack {
-                        Image(systemName: "pencil")
                         Text("Edit")
                     }
                 }
             }
         }
+        .sheet(isPresented: $presentEditView) {
+            if let recipeDetail = recipeDetail {
+                RecipeEditView(viewModel: viewModel, recipe: recipeDetail, isPresented: $presentEditView, uploadNew: false)
+            }
+        }
         .task {
             recipeDetail = await viewModel.loadRecipeDetail(recipeId: recipe.recipe_id)
             recipeImage = await viewModel.loadImage(recipeId: recipe.recipe_id, thumb: false)

Nextcloud Cookbook iOS Client/Views/RecipeEditView.swift

@@ -10,21 +10,112 @@ import SwiftUI
 import PhotosUI
 
 
+fileprivate enum ErrorMessages: Error {
+    
+    case NO_TITLE,
+         DUPLICATE,
+         UPLOAD_ERROR,
+         CONFIRM_DELETE,
+         GENERIC,
+         CUSTOM(title: LocalizedStringKey, description: LocalizedStringKey)
+    
+    var localizedDescription: LocalizedStringKey {
+        switch self {
+        case .NO_TITLE:
+            return "Please enter a recipe name."
+        case .DUPLICATE:
+            return "A recipe with that name already exists."
+        case .UPLOAD_ERROR:
+            return "Unable to upload your recipe. Please check your internet connection."
+        case .CONFIRM_DELETE:
+            return "This action is not reversible!"
+        case .CUSTOM(title: _, description: let description):
+            return description
+        default:
+            return "An unknown error occured."
+        }
+    }
+    
+    var localizedTitle: LocalizedStringKey {
+        switch self {
+        case .NO_TITLE:
+            return "Missing recipe name."
+        case .DUPLICATE:
+            return "Duplicate recipe."
+        case .UPLOAD_ERROR:
+            return "Network error."
+        case .CONFIRM_DELETE:
+            return "Delete recipe?"
+        case .CUSTOM(title: let title, description: _):
+            return title
+        default:
+            return "Error."
+        }
+    }
+}
+
+
 struct RecipeEditView: View {
     @ObservedObject var viewModel: MainViewModel
     @State var recipe: RecipeDetail = RecipeDetail()
     @Binding var isPresented: Bool
-    
-    @State var image: PhotosPickerItem? = nil
-    @State var times = [Date.zero, Date.zero, Date.zero]
     @State var uploadNew: Bool = true
-    @State var searchText: String = ""
-    @State var keywords: [String] = []
 
-    @State private var alertMessage: String = ""
+    @State private var image: PhotosPickerItem? = nil
+    @State private var times = [Date.zero, Date.zero, Date.zero]
+    @State private var searchText: String = ""
+    @State private var keywords: [String] = []
+    
+    @State private var alertMessage: ErrorMessages = .GENERIC
     @State private var presentAlert: Bool = false
+    @State private var waitingForUpload: Bool = false
     
     var body: some View {
+        VStack {
+            HStack {
+                Button() {
+                    isPresented = false
+                } label: {
+                    Text("Cancel")
+                        .bold()
+                }
+                if !uploadNew {
+                    Menu {
+                        Button {
+                            print("Delete recipe.")
+                            alertMessage = .CONFIRM_DELETE
+                            presentAlert = true
+                        } label: {
+                            Image(systemName: "trash")
+                                .foregroundStyle(.red)
+                            Text("Delete recipe")
+                                .foregroundStyle(.red)
+                        }
+                    } label: {
+                        Image(systemName: "ellipsis.circle")
+                            .font(.title3)
+                            .padding()
+                    }
+                }
+                Spacer()
+                Button() {
+                    if uploadNew {
+                        uploadNewRecipe()
+                    } else {
+                        uploadEditedRecipe()
+                    }
+                } label: {
+                    Text("Upload")
+                        .bold()
+                }
+            }.padding()
+            HStack {
+                Text(recipe.name == "" ? "New recipe" : recipe.name)
+                    .font(.title)
+                    .bold()
+                    .padding()
+                Spacer()
+            }
             Form {
                 TextField("Title", text: $recipe.name)
                 Section {
@@ -86,32 +177,6 @@ struct RecipeEditView: View {
                 EditableListSection(title: "Ingredients", items: $recipe.recipeIngredient)
                 EditableListSection(title: "Tools", items: $recipe.tool)
                 EditableListSection(title: "Instructions", items: $recipe.recipeInstructions)
-        }.navigationTitle("Edit your recipe")
-        .toolbar {
-            Menu {
-                Button {
-                    print("Delete recipe.")
-                    deleteRecipe()
-                    self.isPresented = false
-                } label: {
-                    Image(systemName: "trash")
-                        .foregroundStyle(.red)
-                    Text("Delete recipe")
-                        .foregroundStyle(.red)
-                }
-            } label: {
-                Image(systemName: "ellipsis.circle")
-            }
-            Button() {
-                if uploadNew {
-                    uploadNewRecipe()
-                } else {
-                    uploadEditedRecipe()
-                }
-            } label: {
-                Image(systemName: "icloud.and.arrow.up")
-                Text(uploadNew ? "Upload" : "Update")
-                    .bold()
             }
         }
         .onAppear {
@@ -130,15 +195,22 @@ struct RecipeEditView: View {
                 keywords.append(keyword)
             }
         }
-        .alert(alertMessage, isPresented: $presentAlert) {
+        .alert(alertMessage.localizedTitle, isPresented: $presentAlert) {
-            Button("Ok", role: .cancel) {
+            switch alertMessage {
-                self.isPresented = false
+            case .CONFIRM_DELETE:
+                Button("Cancel", role: .cancel) { }
+                Button("Delete", role: .destructive) {
+                    deleteRecipe()
                 }
+            default:
+                Button("Ok", role: .cancel) { }
+            }
+        } message: {
+            Text(alertMessage.localizedDescription)
         }
     }
     
     func createRecipe() {
-        print(self.recipe.name)
         if let date = Date.toPTRepresentation(date: times[0]) {
             self.recipe.prepTime = date
         }
@@ -148,12 +220,43 @@ struct RecipeEditView: View {
         if let date = Date.toPTRepresentation(date: times[2]) {
             self.recipe.totalTime = date
         }
+        if !self.keywords.isEmpty {
             self.recipe.keywords = self.keywords.joined(separator: ",")
         }
+    }
+    
+    func recipeValid() -> Bool {
+        // Check if the recipe has a name
+        if recipe.name == "" {
+            self.alertMessage = .NO_TITLE
+            self.presentAlert = true
+            return false
+        }
+        // Check if the recipe has a unique name
+        for recipeList in viewModel.recipes.values {
+            for r in recipeList {
+                if r.name
+                    .replacingOccurrences(of: " ", with: "")
+                    .lowercased() ==
+                    recipe.name
+                    .replacingOccurrences(of: " ", with: "")
+                    .lowercased()
+                {
+                    self.alertMessage = .DUPLICATE
+                    self.presentAlert = true
+                    return false
+                }
+            }
+        }
+        
+        return true
+    }
     
     func uploadNewRecipe() {
         print("Uploading new recipe.")
+        waitingForUpload = true
         createRecipe()
+        guard recipeValid() else { return }
         let request = RequestWrapper.customRequest(
             method: .POST,
             path: .NEW_RECIPE,
@@ -165,9 +268,11 @@ struct RecipeEditView: View {
             body: JSONEncoder.safeEncode(self.recipe)
         )
         sendRequest(request)
+        dismissEditView()
     }
     
     func uploadEditedRecipe() {
+        waitingForUpload = true
         print("Uploading changed recipe.")
         guard let recipeId = Int(recipe.id) else { return }
         createRecipe()
@@ -182,6 +287,7 @@ struct RecipeEditView: View {
             body: JSONEncoder.safeEncode(self.recipe)
         )
         sendRequest(request)
+        dismissEditView()
     }
     
     func deleteRecipe() {
@@ -195,6 +301,7 @@ struct RecipeEditView: View {
             ]
         )
         sendRequest(request)
+        dismissEditView()
     }
     
     func sendRequest(_ request: RequestWrapper) {
@@ -204,34 +311,28 @@ struct RecipeEditView: View {
             guard let data = data else { return }
             do {
                 let error = try JSONDecoder().decode(ServerMessage.self, from: data)
-                alertMessage = error.msg
+                DispatchQueue.main.sync {
+                    alertMessage = .CUSTOM(title: "Error.", description: LocalizedStringKey(stringLiteral: error.msg))
                     presentAlert = true
+                    return
+                }
             } catch {
-                self.isPresented = false
+                
+            }
+        }
+    }
+    
+    func dismissEditView() {
+        Task {
+            await self.viewModel.loadCategoryList(needsUpdate: true)
             await self.viewModel.loadRecipeList(categoryName: self.recipe.recipeCategory, needsUpdate: true)
         }
-        }
+        self.isPresented = false
     }
 }
 
 
 
-struct SearchField: View {
-    @State var title: String
-    @State var text: String
-    @State var searchSuggestions: [String]
-    
-    var body: some View {
-        TextField(title, text: $text)
-            .searchSuggestions {
-                ForEach(searchSuggestions, id: \.self) { suggestion in
-                    Text(suggestion).searchCompletion(suggestion)
-                }
-            }
-    }
-}
-
-
 struct EditableListSection: View {
     @State var title: String
     @Binding var items: [String]