Return completion data when requesting temporary full accuracy permis…

…sions (#20) * Returns the completion closure's error or the updated accuracyAuthorization as a future * Callback with AccuracyAuthorization * Small tweaks * infer Co-authored-by: Stephen Celis <[email protected]>
pointfreeco · Mar 22, 2022 · 95d45d7 · 95d45d7
1 parent 198e936
commit 95d45d7
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 6 deletions.
diff --git a/Sources/ComposableCoreLocation/Interface.swift b/Sources/ComposableCoreLocation/Interface.swift
@@ -294,9 +294,7 @@ public struct LocationManager {
 
   public var requestWhenInUseAuthorization: () -> Effect<Never, Never>
 
-  public var requestTemporaryFullAccuracyAuthorization:
-    (String)
-      -> Effect<Never, Never>
+  public var requestTemporaryFullAccuracyAuthorization: (String) -> Effect<Never, Error>
 
   public var set: (Properties) -> Effect<Never, Never>
 

diff --git a/Sources/ComposableCoreLocation/Live.swift b/Sources/ComposableCoreLocation/Live.swift
@@ -103,13 +103,21 @@ extension LocationManager {
         }
       },
       requestTemporaryFullAccuracyAuthorization: { purposeKey in
-        .fireAndForget {
+        .run { subscriber in
           #if (compiler(>=5.3) && !(os(macOS) || targetEnvironment(macCatalyst))) || compiler(>=5.3.1)
             if #available(iOS 14.0, tvOS 14.0, watchOS 7.0, macOS 11.0, macCatalyst 14.0, *) {
-              manager
-                .requestTemporaryFullAccuracyAuthorization(withPurposeKey: purposeKey)
+              manager.requestTemporaryFullAccuracyAuthorization(
+                withPurposeKey: purposeKey
+              ) { error in
+                subscriber.send(completion: error.map { .failure(.init($0)) } ?? .finished)
+              }
+            } else {
+              subscriber.send(completion: .finished)
             }
+          #else
+            subscriber.send(completion: .finished)
           #endif
+          return AnyCancellable {}
         }
       },
       set: { properties in