mobile/home-assistant-android
mobile/home-assistant-android
1
0
Fork 0
mirror of https://github.com/home-assistant/android synced 2024-07-22 19:04:20 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Breaking Change: Fix mobile data roaming sensor after target SDK change (#2978)

Browse source 
Fix mobile data roaming sensor after target SDK change
This commit is contained in:
Daniel Shokouhi 2022-10-19 18:06:27 -07:00 committed by GitHub
parent 1cee64c768
commit 07845fb797
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 10 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
app/src/main/java/io/homeassistant/companion/android/sensors/MobileDataManager.kt
View file

@ -1,7 +1,9 @@
package io.homeassistant.companion.android.sensors
import android.Manifest
import android.content.Context
import android.content.pm.PackageManager
import android.os.Build
import android.provider.Settings
import android.provider.Settings.Global.getInt
import android.telephony.TelephonyManager
@ -43,7 +45,9 @@ class MobileDataManager : SensorManager {
}
override fun requiredPermissions(sensorId: String): Array<String> {
return arrayOf()
return if (sensorId == mobileDataRoaming.id)
arrayOf(Manifest.permission.READ_PHONE_STATE)
else arrayOf()
}
override fun hasSensor(context: Context): Boolean {
@ -69,7 +73,11 @@ class MobileDataManager : SensorManager {
var enabled = false
val telephonyManager = context.applicationContext.getSystemService<TelephonyManager>()
if (telephonyManager?.simState == TelephonyManager.SIM_STATE_READY) {
enabled = getInt(context.contentResolver, settingKey, 0) == 1
enabled = if (sensor.id == mobileDataRoaming.id && Build.VERSION.SDK_INT >= Build.VERSION_CODES.Q)
telephonyManager.isDataRoamingEnabled
else {
getInt(context.contentResolver, settingKey, 0) == 1
}
}
onSensorUpdated(
context,