From 37f42e2ad870d03c1ff6d56f0019e6add934905f Mon Sep 17 00:00:00 2001
From: Maurice Parker <mo@vincode.io>
Date: Wed, 12 Jun 2019 08:33:14 -0500
Subject: [PATCH] Added Feedbin add account logic.

---
 iOS/Settings/SettingsAddAccountView.swift     |   2 +-
 iOS/Settings/SettingsFeedbinAccountView.swift | 111 ++++++++++++++++--
 iOS/Settings/SettingsLocalAccountView.swift   |   4 +-
 3 files changed, 106 insertions(+), 11 deletions(-)

diff --git a/iOS/Settings/SettingsAddAccountView.swift b/iOS/Settings/SettingsAddAccountView.swift
index e7ec22319..5b1a8bda5 100644
--- a/iOS/Settings/SettingsAddAccountView.swift
+++ b/iOS/Settings/SettingsAddAccountView.swift
@@ -14,7 +14,7 @@ struct SettingsAddAccountView : View {
 			PresentationButton(SettingsAccountLabelView(accountImage: "accountLocal", accountLabel: "On My Device"),
 							   destination: SettingsLocalAccountView(name: ""))
 			PresentationButton(SettingsAccountLabelView(accountImage: "accountFeedbin", accountLabel: "Feedbin"),
-							   destination: SettingsFeedbinAccountView(email: "", password: ""))
+							   destination: SettingsFeedbinAccountView(viewModel: SettingsFeedbinAccountView.ViewModel()))
 		}
 		.listStyle(.grouped)
 		.navigationBarTitle(Text("Add Account"), displayMode: .inline)
diff --git a/iOS/Settings/SettingsFeedbinAccountView.swift b/iOS/Settings/SettingsFeedbinAccountView.swift
index f62fce8c5..50d425bc7 100644
--- a/iOS/Settings/SettingsFeedbinAccountView.swift
+++ b/iOS/Settings/SettingsFeedbinAccountView.swift
@@ -7,11 +7,17 @@
 //
 
 import SwiftUI
+import Combine
+import Account
+import RSWeb
 
 struct SettingsFeedbinAccountView : View {
-	@State var email: String
-	@State var password: String
-
+	@Environment(\.isPresented) private var isPresented
+	@ObjectBinding var viewModel: ViewModel
+	@State var busy: Bool = false
+	@State var error: Text = Text("")
+	var account: Account? = nil
+	
 	var body: some View {
 		NavigationView {
 			List {
@@ -20,27 +26,116 @@ struct SettingsFeedbinAccountView : View {
 				)  {
 					HStack {
 						Spacer()
-						TextField($email, placeholder: Text("Email"))
+						TextField($viewModel.email, placeholder: Text("Email"))
+							.textContentType(.username)
 						Spacer()
 					}
 					HStack {
 						Spacer()
-						SecureField($password, placeholder: Text("Password"))
+						SecureField($viewModel.password, placeholder: Text("Password"))
 						Spacer()
 					}
 				}
-				Section {
+				Section(footer:
 					HStack {
 						Spacer()
-						Button(action: {}) {
+						error.color(.red)
+						Spacer()
+					}
+					) {
+					HStack {
+						Spacer()
+						Button(action: { self.addAccount() }) {
 							Text("Add Account")
 						}
+						.disabled(!viewModel.isValid)
 						Spacer()
 					}
 				}
 			}
+			.disabled(busy)
 			.listStyle(.grouped)
 			.navigationBarTitle(Text(""), displayMode: .inline)
+			.navigationBarItems(leading:
+				Button(action: { self.dismiss() }) { Text("Cancel") }
+			)
+		}
+	}
+	
+	private func addAccount() {
+		
+		busy = true
+
+		let emailAddress = viewModel.email.trimmingCharacters(in: .whitespaces)
+		let credentials = Credentials.basic(username: emailAddress, password: viewModel.password)
+
+		Account.validateCredentials(type: .feedbin, credentials: credentials) { result in
+			
+			self.busy = false
+			
+			switch result {
+			case .success(let authenticated):
+				
+				if authenticated {
+					
+					var newAccount = false
+					let workAccount: Account
+					if self.account == nil {
+						workAccount = AccountManager.shared.createAccount(type: .feedbin)
+						newAccount = true
+					} else {
+						workAccount = self.account!
+					}
+					
+					do {
+						
+						do {
+							try workAccount.removeBasicCredentials()
+						} catch {}
+						try workAccount.storeCredentials(credentials)
+						
+						if newAccount {
+							workAccount.refreshAll() { result in }
+						}
+						
+						self.dismiss()
+						
+					} catch {
+						self.error = Text("Keychain error while storing credentials.")
+					}
+					
+				} else {
+					self.error = Text("Invalid email/password combination.")
+				}
+				
+			case .failure:
+				self.error = Text("Network error. Try again later.")
+			}
+			
+		}
+		
+	}
+	
+	private func dismiss() {
+		isPresented?.value = false
+	}
+	
+	class ViewModel: BindableObject {
+		let didChange = PassthroughSubject<ViewModel, Never>()
+
+		var email: String = "" {
+			didSet {
+				didChange.send(self)
+			}
+		}
+		var password: String = "" {
+			didSet {
+				didChange.send(self)
+			}
+		}
+		
+		var isValid: Bool {
+			return !email.isEmpty && !password.isEmpty
 		}
 	}
 	
@@ -49,7 +144,7 @@ struct SettingsFeedbinAccountView : View {
 #if DEBUG
 struct SettingsFeedbinAccountView_Previews : PreviewProvider {
     static var previews: some View {
-        SettingsFeedbinAccountView(email: "", password: "")
+        SettingsFeedbinAccountView(viewModel: SettingsFeedbinAccountView.ViewModel())
     }
 }
 #endif
diff --git a/iOS/Settings/SettingsLocalAccountView.swift b/iOS/Settings/SettingsLocalAccountView.swift
index 04c91c200..e39edcec4 100644
--- a/iOS/Settings/SettingsLocalAccountView.swift
+++ b/iOS/Settings/SettingsLocalAccountView.swift
@@ -10,8 +10,8 @@ import SwiftUI
 import Account
 
 struct SettingsLocalAccountView : View {
-	@State var name: String
 	@Environment(\.isPresented) private var isPresented
+	@State var name: String
 
     var body: some View {
 		NavigationView {
@@ -37,7 +37,7 @@ struct SettingsLocalAccountView : View {
 			}
 			.listStyle(.grouped)
 			.navigationBarTitle(Text(""), displayMode: .inline)
-			.navigationBarItems(trailing: Button(action: { self.dismiss() }) { Text("Cancel") } )
+			.navigationBarItems(leading: Button(action: { self.dismiss() }) { Text("Cancel") } )
 		}
 	}