filippodb/SchildiChat-android
1
0
Fork 0
mirror of https://github.com/SchildiChat/SchildiChat-android.git synced 2025-02-09 00:29:00 +01:00
Code Issues Packages Projects Releases Wiki Activity

Call no provider available callback only providers list is empty

Browse Source
This commit is contained in:
Maxime NATUREL 2022-06-03 14:11:52 +02:00
parent b686d30b1c
commit 45d3fe7c07
3 changed files with 33 additions and 21 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
vector/src/main/java/im/vector/app/features/location/LocationSharingService.kt
View File

@ -198,7 +198,7 @@ class LocationSharingService : VectorService(), LocationTracker.Callback {
) )
} }
override fun onLocationProviderIsNotAvailable() { override fun onNoLocationProviderAvailable() {
stopForeground(true) stopForeground(true)
stopSelf() stopSelf()
} }

2
vector/src/main/java/im/vector/app/features/location/LocationSharingViewModel.kt
View File

@ -182,7 +182,7 @@ class LocationSharingViewModel @AssistedInject constructor(
} }
} }
override fun onLocationProviderIsNotAvailable() { override fun onNoLocationProviderAvailable() {
_viewEvents.post(LocationSharingViewEvents.LocationNotAvailableError) _viewEvents.post(LocationSharingViewEvents.LocationNotAvailableError)
} }
} }

50
vector/src/main/java/im/vector/app/features/location/LocationTracker.kt
View File

@ -41,8 +41,15 @@ class LocationTracker @Inject constructor(
private val locationManager = context.getSystemService<LocationManager>() private val locationManager = context.getSystemService<LocationManager>()
interface Callback { interface Callback {
/**
* Called on every location update.
*/
fun onLocationUpdate(locationData: LocationData) fun onLocationUpdate(locationData: LocationData)
fun onLocationProviderIsNotAvailable()
/**
* Called when no location provider is available to request location updates.
*/
fun onNoLocationProviderAvailable()
} }
private val callbacks = mutableListOf<Callback>() private val callbacks = mutableListOf<Callback>()
@ -56,24 +63,24 @@ class LocationTracker @Inject constructor(
@RequiresPermission(anyOf = [Manifest.permission.ACCESS_COARSE_LOCATION, Manifest.permission.ACCESS_FINE_LOCATION]) @RequiresPermission(anyOf = [Manifest.permission.ACCESS_COARSE_LOCATION, Manifest.permission.ACCESS_FINE_LOCATION])
fun start() { fun start() {
Timber.d("## LocationTracker. start()") Timber.d("start()")
if (locationManager == null) { if (locationManager == null) {
callbacks.forEach { it.onLocationProviderIsNotAvailable() } callbacks.forEach { it.onNoLocationProviderAvailable() }
Timber.v("## LocationTracker. LocationManager is not available") Timber.v("LocationManager is not available")
return return
} }
val providers = locationManager.allProviders val providers = locationManager.allProviders
if (providers.isEmpty()) { if (providers.isEmpty()) {
callbacks.forEach { it.onLocationProviderIsNotAvailable() } callbacks.forEach { it.onNoLocationProviderAvailable() }
Timber.v("## LocationTracker. There is no location provider available") Timber.v("There is no location provider available")
} else { } else {
// Take GPS first // Take GPS first
providers.sortedByDescending { getProviderPriority(it) } providers.sortedByDescending { getProviderPriority(it) }
.mapNotNull { provider -> .mapNotNull { provider ->
Timber.d("## LocationTracker. track location using $provider") Timber.d("track location using $provider")
locationManager.requestLocationUpdates( locationManager.requestLocationUpdates(
provider, provider,
@ -87,9 +94,9 @@ class LocationTracker @Inject constructor(
.maxByOrNull { location -> location.time } .maxByOrNull { location -> location.time }
?.let { latestKnownLocation -> ?.let { latestKnownLocation ->
if (BuildConfig.LOW_PRIVACY_LOG_ENABLE) { if (BuildConfig.LOW_PRIVACY_LOG_ENABLE) {
Timber.d("## LocationTracker. lastKnownLocation: $latestKnownLocation") Timber.d("lastKnownLocation: $latestKnownLocation")
} else { } else {
Timber.d("## LocationTracker. lastKnownLocation: ${latestKnownLocation.provider}") Timber.d("lastKnownLocation: ${latestKnownLocation.provider}")
} }
notifyLocation(latestKnownLocation) notifyLocation(latestKnownLocation)
} }
@ -98,7 +105,7 @@ class LocationTracker @Inject constructor(
/** /**
* Compute the priority of the given provider name. * Compute the priority of the given provider name.
* @return an integer representing the priority: the higher the value, the higher the priority is * @return an integer representing the priority: the higher the value, the higher the priority is.
*/ */
private fun getProviderPriority(provider: String): Int = when (provider) { private fun getProviderPriority(provider: String): Int = when (provider) {
LocationManager.FUSED_PROVIDER -> 2 LocationManager.FUSED_PROVIDER -> 2
@ -108,7 +115,7 @@ class LocationTracker @Inject constructor(
@RequiresPermission(anyOf = [Manifest.permission.ACCESS_COARSE_LOCATION, Manifest.permission.ACCESS_FINE_LOCATION]) @RequiresPermission(anyOf = [Manifest.permission.ACCESS_COARSE_LOCATION, Manifest.permission.ACCESS_FINE_LOCATION])
fun stop() { fun stop() {
Timber.d("## LocationTracker. stop()") Timber.d("stop()")
locationManager?.removeUpdates(this) locationManager?.removeUpdates(this)
callbacks.clear() callbacks.clear()
debouncer.cancelAll() debouncer.cancelAll()
@ -139,9 +146,9 @@ class LocationTracker @Inject constructor(
override fun onLocationChanged(location: Location) { override fun onLocationChanged(location: Location) {
if (BuildConfig.LOW_PRIVACY_LOG_ENABLE) { if (BuildConfig.LOW_PRIVACY_LOG_ENABLE) {
Timber.d("## LocationTracker. onLocationChanged: $location") Timber.d("onLocationChanged: $location")
} else { } else {
Timber.d("## LocationTracker. onLocationChanged: ${location.provider}") Timber.d("onLocationChanged: ${location.provider}")
} }
when (location.provider) { when (location.provider) {
@ -152,7 +159,7 @@ class LocationTracker @Inject constructor(
if (hasLocationFromFusedProvider) { if (hasLocationFromFusedProvider) {
hasLocationFromGPSProvider = false hasLocationFromGPSProvider = false
// Ignore this update // Ignore this update
Timber.d("## LocationTracker. ignoring location from ${location.provider}, we have location from fused provider") Timber.d("ignoring location from ${location.provider}, we have location from fused provider")
return return
} else { } else {
hasLocationFromGPSProvider = true hasLocationFromGPSProvider = true
@ -161,7 +168,7 @@ class LocationTracker @Inject constructor(
else -> { else -> {
if (hasLocationFromFusedProvider || hasLocationFromGPSProvider) { if (hasLocationFromFusedProvider || hasLocationFromGPSProvider) {
// Ignore this update // Ignore this update
Timber.d("## LocationTracker. ignoring location from ${location.provider}, we have location from GPS provider") Timber.d("ignoring location from ${location.provider}, we have location from GPS provider")
return return
} }
} }
@ -174,9 +181,9 @@ class LocationTracker @Inject constructor(
private fun notifyLocation(location: Location) { private fun notifyLocation(location: Location) {
if (BuildConfig.LOW_PRIVACY_LOG_ENABLE) { if (BuildConfig.LOW_PRIVACY_LOG_ENABLE) {
Timber.d("## LocationTracker. notify location: $location") Timber.d("notify location: $location")
} else { } else {
Timber.d("## LocationTracker. notify location: ${location.provider}") Timber.d("notify location: ${location.provider}")
} }
val locationData = location.toLocationData() val locationData = location.toLocationData()
@ -185,12 +192,17 @@ class LocationTracker @Inject constructor(
} }
override fun onProviderDisabled(provider: String) { override fun onProviderDisabled(provider: String) {
Timber.d("## LocationTracker. onProviderDisabled: $provider") Timber.d("onProviderDisabled: $provider")
when (provider) { when (provider) {
LocationManager.FUSED_PROVIDER -> hasLocationFromFusedProvider = false LocationManager.FUSED_PROVIDER -> hasLocationFromFusedProvider = false
LocationManager.GPS_PROVIDER -> hasLocationFromGPSProvider = false LocationManager.GPS_PROVIDER -> hasLocationFromGPSProvider = false
} }
callbacks.forEach { it.onLocationProviderIsNotAvailable() }
locationManager?.allProviders
?.takeIf { it.isEmpty() }
?.let {
callbacks.forEach { it.onNoLocationProviderAvailable() }
}
} }
private fun Location.toLocationData(): LocationData { private fun Location.toLocationData(): LocationData {