Make SettingsStore functions non-static, re-implement them as

subscripts and make settingsStore an environmentObject

Fixes: https://github.com/maxgoedjen/secretive/pull/536#discussion_r1508446459
This commit is contained in:
Paul Heidekrüger 2024-04-05 09:24:35 +02:00
parent 46022962b8
commit a5a3f1cec9
2 changed files with 45 additions and 40 deletions

View File

@ -16,6 +16,7 @@ struct Secretive: App {
}() }()
private let agentStatusChecker = AgentStatusChecker() private let agentStatusChecker = AgentStatusChecker()
private let justUpdatedChecker = JustUpdatedChecker() private let justUpdatedChecker = JustUpdatedChecker()
private let settingsStore = SettingsStore()
@AppStorage("defaultsHasRunSetup") var hasRunSetup = false @AppStorage("defaultsHasRunSetup") var hasRunSetup = false
@State private var showingSetup = false @State private var showingSetup = false
@ -27,6 +28,7 @@ struct Secretive: App {
.environmentObject(storeList) .environmentObject(storeList)
.environmentObject(Updater(checkOnLaunch: hasRunSetup)) .environmentObject(Updater(checkOnLaunch: hasRunSetup))
.environmentObject(agentStatusChecker) .environmentObject(agentStatusChecker)
.environmentObject(settingsStore)
.onAppear { .onAppear {
if !hasRunSetup { if !hasRunSetup {
showingSetup = true showingSetup = true

View File

@ -8,17 +8,20 @@
import Foundation import Foundation
class SettingsStore { class SettingsStore: ObservableObject {
static let service = "com.maxgoedjen.Secretive" let service = "com.maxgoedjen.Secretive"
} }
extension SettingsStore { extension SettingsStore {
static func set(key: String, value: String) -> Bool { subscript(key: String) -> String? {
let valueData = value.data(using: String.Encoding.utf8)! set(value) {
guard let valueData = value?.data(using: String.Encoding.utf8)! else {
return
}
if let keyVal = get(key: key) { if let keyVal = self[key] {
if keyVal == value { if keyVal == value {
return true return
} }
let updateQuery: [String: Any] = [kSecClass as String: kSecClassGenericPassword, let updateQuery: [String: Any] = [kSecClass as String: kSecClassGenericPassword,
@ -29,7 +32,7 @@ extension SettingsStore {
let status = SecItemUpdate(updateQuery as CFDictionary, attributes as CFDictionary) let status = SecItemUpdate(updateQuery as CFDictionary, attributes as CFDictionary)
guard status == errSecSuccess else { guard status == errSecSuccess else {
print("Couldn't update item in keychain. " + status.description) print("Couldn't update item in keychain. " + status.description)
return false return
} }
} else { } else {
let addquery: [String: Any] = [kSecClass as String: kSecClassGenericPassword, let addquery: [String: Any] = [kSecClass as String: kSecClassGenericPassword,
@ -40,13 +43,12 @@ extension SettingsStore {
let status = SecItemAdd(addquery as CFDictionary, nil) let status = SecItemAdd(addquery as CFDictionary, nil)
guard status == errSecSuccess else { guard status == errSecSuccess else {
print("Couldn't add item to keychain. " + status.description) print("Couldn't add item to keychain. " + status.description)
return false return
} }
} }
return true
} }
static func get(key: String) -> String? { get {
let getquery: [String: Any] = [kSecClass as String: kSecClassGenericPassword, let getquery: [String: Any] = [kSecClass as String: kSecClassGenericPassword,
kSecAttrAccount as String: key, kSecAttrAccount as String: key,
kSecAttrServer as String: service, kSecAttrServer as String: service,
@ -57,4 +59,5 @@ extension SettingsStore {
return status == errSecSuccess ? String(decoding: item as! Data, as: UTF8.self) : nil return status == errSecSuccess ? String(decoding: item as! Data, as: UTF8.self) : nil
} }
}
} }