[sdk][andr] Don't use PowerManager if not available (#35)

* [sdk][andr] Don't use PowerManager if not available * Add comment
bitdriftlabs · Sep 10, 2024 · d888daa · d888daa
1 parent 261815b
commit d888daa
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/platform/jvm/capture/src/main/kotlin/io/bitdrift/capture/events/common/PowerMonitor.kt b/platform/jvm/capture/src/main/kotlin/io/bitdrift/capture/events/common/PowerMonitor.kt
@@ -13,7 +13,9 @@ import android.os.PowerManager
 import androidx.annotation.RequiresApi
 
 internal class PowerMonitor(context: Context) {
-    val powerManager = context.getSystemService(Context.POWER_SERVICE) as PowerManager
+    // Customers have reported encountering certain devices (particularly the Caterpillar S48C phone)
+    // where this returns null on Android 8.1.0. Even though it should always be available on API level >= 21.
+    val powerManager = context.getSystemService(Context.POWER_SERVICE) as? PowerManager
 
     @RequiresApi(Build.VERSION_CODES.Q)
     private val thermalStatusMap = hashMapOf(
@@ -32,6 +34,6 @@ internal class PowerMonitor(context: Context) {
     }
 
     fun isPowerSaveModeEnabledAttribute(): Pair<String, String> {
-        return Pair("_low_power_enabled", if (powerManager.isPowerSaveMode) "1" else "0")
+        return Pair("_low_power_enabled", if (powerManager?.isPowerSaveMode == true) "1" else "0")
     }
 }
diff --git a/...vm/capture/src/main/kotlin/io/bitdrift/capture/events/device/DeviceStateListenerLogger.kt b/...vm/capture/src/main/kotlin/io/bitdrift/capture/events/device/DeviceStateListenerLogger.kt
@@ -66,7 +66,7 @@ internal class DeviceStateListenerLogger(
         context.registerComponentCallbacks(this)
 
         if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.Q) {
-            powerMonitor.powerManager.addThermalStatusListener(executor, thermalCallback)
+            powerMonitor.powerManager?.addThermalStatusListener(executor, thermalCallback)
         }
     }
 
@@ -75,7 +75,7 @@ internal class DeviceStateListenerLogger(
         context.unregisterComponentCallbacks(this)
 
         if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.Q) {
-            powerMonitor.powerManager.removeThermalStatusListener(thermalCallback)
+            powerMonitor.powerManager?.removeThermalStatusListener(thermalCallback)
         }
     }