Skip to content

Commit

Permalink
Update LocationManager
Browse files Browse the repository at this point in the history
- Change log to warn vs. error
- Access and save to constant PREFS_OS_LOCATION_SHARED
  • Loading branch information
jennantilla committed Dec 27, 2023
1 parent fc842b9 commit 8b0b74b
Showing 1 changed file with 5 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,8 @@ import com.onesignal.common.AndroidUtils
import com.onesignal.common.threading.suspendifyOnThread
import com.onesignal.core.internal.application.IApplicationService
import com.onesignal.core.internal.preferences.IPreferencesService
import com.onesignal.core.internal.preferences.PreferenceOneSignalKeys
import com.onesignal.core.internal.preferences.PreferenceStores
import com.onesignal.core.internal.startup.IStartableService
import com.onesignal.debug.LogLevel
import com.onesignal.debug.internal.logging.Logging
Expand All @@ -25,12 +27,12 @@ internal class LocationManager(
private val _locationPermissionController: LocationPermissionController,
private val _prefs: IPreferencesService,
) : ILocationManager, IStartableService, ILocationPermissionChangedHandler {
private var _isShared: Boolean = _prefs.getBool("OneSignal", "PREFS_OS_LOCATION_SHARED", false)!!
private var _isShared: Boolean = _prefs.getBool(PreferenceStores.ONESIGNAL, PreferenceOneSignalKeys.PREFS_OS_LOCATION_SHARED, false)!!
override var isShared
get() = _isShared
set(value) {
Logging.debug("LocationManager.setIsShared(value: $value)")
_prefs.saveBool("OneSignal", "PREFS_OS_LOCATION_SHARED", value)
_prefs.saveBool(PreferenceStores.ONESIGNAL, PreferenceOneSignalKeys.PREFS_OS_LOCATION_SHARED, value)
_isShared = value

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

val hasFinePermissionGranted =
Expand Down

0 comments on commit 8b0b74b

Please sign in to comment.