mobile/home-assistant-android
mobile/home-assistant-android
1
0
Fork 0
mirror of https://github.com/home-assistant/android synced 2024-10-06 16:19:38 +00:00
Code Issues Packages Projects Releases Wiki Activity

Only update geocode sensor if accuracy and time are valid, like location updates (#2802)

Browse source
This commit is contained in:
Daniel Shokouhi 2022-08-19 17:34:33 -07:00 committed by GitHub
parent 06fe9569cc
commit da8263dc9c
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 12 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
app/src/full/java/io/homeassistant/companion/android/sensors/GeocodeSensorManager.kt
View file

@ -76,10 +76,20 @@ class GeocodeSensorManager : SensorManager {
?: DEFAULT_MINIMUM_ACCURACY
sensorDao.add(SensorSetting(geocodedLocation.id, SETTING_ACCURACY, minAccuracy.toString(), SensorSettingType.NUMBER))
if (location.accuracy <= minAccuracy)
if (location.accuracy <= minAccuracy) {
address = Geocoder(context)
.getFromLocation(location.latitude, location.longitude, 1)
.firstOrNull()
} else {
Log.w(TAG, "Skipping geocoded update as accuracy was not met: ${location.accuracy}")
return@addOnSuccessListener
}
val now = System.currentTimeMillis()
if (now - location.time > 300000) {
Log.w(TAG, "Skipping geocoded update due to old timestamp ${location.time} compared to $now")
return@addOnSuccessListener
}
} catch (e: Exception) {
Log.e(TAG, "Failed to get geocoded location", e)
}
@ -99,7 +109,7 @@ class GeocodeSensorManager : SensorManager {
)
}.orEmpty()
var prettyAddress = address?.getAddressLine(0)
val prettyAddress = address?.getAddressLine(0)
HighAccuracyLocationService.updateNotificationAddress(context, location, if (!prettyAddress.isNullOrEmpty()) prettyAddress else context.getString(commonR.string.unknown_address))