Fix incorrect non determined status hanling as incorrect for one shot…

… request
malcommac · Nov 30, 2020 · 5cf11b5 · 5cf11b5
1 parent 3d16166
commit 5cf11b5
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 1 deletion.
diff --git a/Sources/SwiftLocation/Extensions/CLLocationManager+Extension.swift b/Sources/SwiftLocation/Extensions/CLLocationManager+Extension.swift
@@ -146,6 +146,15 @@ extension CLAuthorizationStatus: CustomStringConvertible {
             return false
         }
     }
+
+    internal var isRejected: Bool {
+        switch self {
+        case .denied, .restricted:
+            return true
+        default:
+            return false
+        }
+    }
 
     public var description: String {
         switch self {

diff --git a/Sources/SwiftLocation/SwiftLocation.swift b/Sources/SwiftLocation/SwiftLocation.swift
@@ -551,7 +551,7 @@ public class LocationManager: LocationManagerDelegate, CustomStringConvertible {
     }
 
     private func failWeakAuthorizationRequests() {
-        guard authorizationStatus.isAuthorized == false else {
+        guard authorizationStatus.isRejected == true else {
             // If we have already the authorization even request with `avoidRequestAuthorization = true`
             // may receive notifications of locations.
             return

diff --git a/...und/SwiftLocationPlayground/View Controllers/Features/Location by GPS/GPSController.swift b/...und/SwiftLocationPlayground/View Controllers/Features/Location by GPS/GPSController.swift
@@ -232,6 +232,7 @@ class GPSController: UIViewController, UITableViewDelegate, UITableViewDataSourc
         let request = SwiftLocation.gpsLocationWith(serviceOptions)
 
         serviceOptions = GPSLocationOptions()
+        serviceOptions.avoidRequestAuthorization = true
         reloadData()
 
         AppDelegate.attachSubscribersToGPS([request])