From 8dcf4966fbbb6f470bcda1f8cd4ce7329047ecf2 Mon Sep 17 00:00:00 2001
From: Maurice Parker <mo@vincode.io>
Date: Sat, 15 Jun 2019 16:03:41 -0500
Subject: [PATCH] Port Feedbin credential update too SwfitUI.

---
 iOS/Settings/SettingsDetailAccountView.swift  | 31 ++++++++----
 iOS/Settings/SettingsFeedbinAccountView.swift | 48 +++++++++++++------
 2 files changed, 57 insertions(+), 22 deletions(-)

diff --git a/iOS/Settings/SettingsDetailAccountView.swift b/iOS/Settings/SettingsDetailAccountView.swift
index 75f4fe1fe..dab8f30ce 100644
--- a/iOS/Settings/SettingsDetailAccountView.swift
+++ b/iOS/Settings/SettingsDetailAccountView.swift
@@ -9,10 +9,12 @@
 import SwiftUI
 import Combine
 import Account
+import RSWeb
 
 struct SettingsDetailAccountView : View {
 	@ObjectBinding var viewModel: ViewModel
 	@State private var verifyDelete = false
+	@State private var showFeedbinCredentials = false
 	
     var body: some View {
 		List {
@@ -26,15 +28,19 @@ struct SettingsDetailAccountView : View {
 					Text("Active")
 				}
 			}
-			Section {
-				HStack {
-					Spacer()
-					Button(action: {
-						
-					}) {
-						Text("Credentials")
+			if viewModel.isCreditialsAvailable {
+				Section {
+					HStack {
+						Spacer()
+						Button(action: {
+							self.showFeedbinCredentials = true
+						}) {
+							Text("Credentials")
+						}
+						.presentation(showFeedbinCredentials ? feedbinCredentialsModal : nil)
+						.onDisappear() { self.showFeedbinCredentials = false }
+						Spacer()
 					}
-					Spacer()
 				}
 			}
 			if viewModel.isDeletable {
@@ -62,6 +68,11 @@ struct SettingsDetailAccountView : View {
 
 	}
 	
+	var feedbinCredentialsModal: Modal {
+		let feedbinViewModel = SettingsFeedbinAccountView.ViewModel(account: viewModel.account)
+		return Modal(SettingsFeedbinAccountView(viewModel: feedbinViewModel))
+	}
+	
 	class ViewModel: BindableObject {
 		let didChange = PassthroughSubject<ViewModel, Never>()
 		let account: Account
@@ -94,6 +105,10 @@ struct SettingsDetailAccountView : View {
 			}
 		}
 		
+		var isCreditialsAvailable: Bool {
+			return account.type != .onMyMac
+		}
+		
 		var isDeletable: Bool {
 			return AccountManager.shared.defaultAccount != account
 		}
diff --git a/iOS/Settings/SettingsFeedbinAccountView.swift b/iOS/Settings/SettingsFeedbinAccountView.swift
index 50d425bc7..c867c94d2 100644
--- a/iOS/Settings/SettingsFeedbinAccountView.swift
+++ b/iOS/Settings/SettingsFeedbinAccountView.swift
@@ -16,8 +16,7 @@ struct SettingsFeedbinAccountView : View {
 	@ObjectBinding var viewModel: ViewModel
 	@State var busy: Bool = false
 	@State var error: Text = Text("")
-	var account: Account? = nil
-	
+
 	var body: some View {
 		NavigationView {
 			List {
@@ -25,15 +24,15 @@ struct SettingsFeedbinAccountView : View {
 					SettingsAccountLabelView(accountImage: "accountFeedbin", accountLabel: "Feedbin").padding()
 				)  {
 					HStack {
-						Spacer()
-						TextField($viewModel.email, placeholder: Text("Email"))
-							.textContentType(.username)
-						Spacer()
+						Text("Email:")
+						Divider()
+						TextField($viewModel.email)
+						.textContentType(.username)
 					}
 					HStack {
-						Spacer()
-						SecureField($viewModel.password, placeholder: Text("Password"))
-						Spacer()
+						Text("Password:")
+						Divider()
+						SecureField($viewModel.password)
 					}
 				}
 				Section(footer:
@@ -46,7 +45,11 @@ struct SettingsFeedbinAccountView : View {
 					HStack {
 						Spacer()
 						Button(action: { self.addAccount() }) {
-							Text("Add Account")
+							if viewModel.isUpdate {
+								Text("Update Account")
+							} else {
+								Text("Add Account")
+							}
 						}
 						.disabled(!viewModel.isValid)
 						Spacer()
@@ -65,7 +68,8 @@ struct SettingsFeedbinAccountView : View {
 	private func addAccount() {
 		
 		busy = true
-
+		error = Text("")
+		
 		let emailAddress = viewModel.email.trimmingCharacters(in: .whitespaces)
 		let credentials = Credentials.basic(username: emailAddress, password: viewModel.password)
 
@@ -80,11 +84,11 @@ struct SettingsFeedbinAccountView : View {
 					
 					var newAccount = false
 					let workAccount: Account
-					if self.account == nil {
+					if self.viewModel.account == nil {
 						workAccount = AccountManager.shared.createAccount(type: .feedbin)
 						newAccount = true
 					} else {
-						workAccount = self.account!
+						workAccount = self.viewModel.account!
 					}
 					
 					do {
@@ -122,6 +126,18 @@ struct SettingsFeedbinAccountView : View {
 	
 	class ViewModel: BindableObject {
 		let didChange = PassthroughSubject<ViewModel, Never>()
+		var account: Account? = nil
+		
+		init() {
+		}
+		
+		init(account: Account) {
+			self.account = account
+			if case .basic(let username, let password) = try? account.retrieveBasicCredentials() {
+				self.email = username
+				self.password = password
+			}
+		}
 
 		var email: String = "" {
 			didSet {
@@ -134,6 +150,10 @@ struct SettingsFeedbinAccountView : View {
 			}
 		}
 		
+		var isUpdate: Bool {
+			return account != nil
+		}
+		
 		var isValid: Bool {
 			return !email.isEmpty && !password.isEmpty
 		}
@@ -144,7 +164,7 @@ struct SettingsFeedbinAccountView : View {
 #if DEBUG
 struct SettingsFeedbinAccountView_Previews : PreviewProvider {
     static var previews: some View {
-        SettingsFeedbinAccountView(viewModel: SettingsFeedbinAccountView.ViewModel())
+		SettingsFeedbinAccountView(viewModel: SettingsFeedbinAccountView.ViewModel())
     }
 }
 #endif