Remove early return for requestPermission

jennantilla · jinliu9508 · commit 530876b52756 · 2024-02-06T11:26:05.000-05:00
Since isShared now defaults to false, remove early return and add logging to update isShared to true.
diff --git a/OneSignalSDK/onesignal/location/src/main/java/com/onesignal/location/internal/LocationManager.kt b/OneSignalSDK/onesignal/location/src/main/java/com/onesignal/location/internal/LocationManager.kt
@@ -76,7 +76,7 @@ internal class LocationManager(
         var result = false
         withContext(Dispatchers.Main) {
             if (!isShared) {
-                return@withContext false
+                Logging.error("Location permissions must be granted by setting isShared to true")
             }
 
             val hasFinePermissionGranted =

Original file line number	Diff line number	Diff line change
`@@ -76,7 +76,7 @@ internal class LocationManager(`
`76`	`76`	`var result = false`
`77`	`77`	`withContext(Dispatchers.Main) {`
`78`	`78`	`if (!isShared) {`
`79`		`- return@withContext false`
	`79`	`+ Logging.error("Location permissions must be granted by setting isShared to true")`
`80`	`80`	`}`
`81`	`81`
`82`	`82`	`val hasFinePermissionGranted =`