Skip to content

Commit f47c114

Browse files
jennantillajinliu9508
authored andcommitted
Update LocationManager
- Change log to warn vs. error - Access and save to constant PREFS_OS_LOCATION_SHARED
1 parent 530876b commit f47c114

File tree

1 file changed

+5
-3
lines changed
  • OneSignalSDK/onesignal/location/src/main/java/com/onesignal/location/internal

1 file changed

+5
-3
lines changed

OneSignalSDK/onesignal/location/src/main/java/com/onesignal/location/internal/LocationManager.kt

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,8 @@ import com.onesignal.common.AndroidUtils
55
import com.onesignal.common.threading.suspendifyOnThread
66
import com.onesignal.core.internal.application.IApplicationService
77
import com.onesignal.core.internal.preferences.IPreferencesService
8+
import com.onesignal.core.internal.preferences.PreferenceOneSignalKeys
9+
import com.onesignal.core.internal.preferences.PreferenceStores
810
import com.onesignal.core.internal.startup.IStartableService
911
import com.onesignal.debug.LogLevel
1012
import com.onesignal.debug.internal.logging.Logging
@@ -25,12 +27,12 @@ internal class LocationManager(
2527
private val _locationPermissionController: LocationPermissionController,
2628
private val _prefs: IPreferencesService,
2729
) : ILocationManager, IStartableService, ILocationPermissionChangedHandler {
28-
private var _isShared: Boolean = _prefs.getBool("OneSignal", "PREFS_OS_LOCATION_SHARED", false)!!
30+
private var _isShared: Boolean = _prefs.getBool(PreferenceStores.ONESIGNAL, PreferenceOneSignalKeys.PREFS_OS_LOCATION_SHARED, false)!!
2931
override var isShared
3032
get() = _isShared
3133
set(value) {
3234
Logging.debug("LocationManager.setIsShared(value: $value)")
33-
_prefs.saveBool("OneSignal", "PREFS_OS_LOCATION_SHARED", value)
35+
_prefs.saveBool(PreferenceStores.ONESIGNAL, PreferenceOneSignalKeys.PREFS_OS_LOCATION_SHARED, value)
3436
_isShared = value
3537

3638
onLocationPermissionChanged(value)
@@ -76,7 +78,7 @@ internal class LocationManager(
7678
var result = false
7779
withContext(Dispatchers.Main) {
7880
if (!isShared) {
79-
Logging.error("Location permissions must be granted by setting isShared to true")
81+
Logging.warn("Requesting location permission, but location sharing must also be enabled by setting isShared to true")
8082
}
8183

8284
val hasFinePermissionGranted =

0 commit comments

Comments
 (0)