[hue] fix npe (#16619)

Signed-off-by: Andrew Fiddian-Green <software@whitebear.ch>
openhab · Apr 6, 2024 · 203be93 · 203be93
1 parent aebbbdf
commit 203be93
Showing 1 changed file with 9 additions and 5 deletions.
diff --git a/...enhab.binding.hue/src/main/java/org/openhab/binding/hue/internal/api/dto/clip2/Power.java b/...enhab.binding.hue/src/main/java/org/openhab/binding/hue/internal/api/dto/clip2/Power.java
@@ -13,6 +13,7 @@
 package org.openhab.binding.hue.internal.api.dto.clip2;
 
 import org.eclipse.jdt.annotation.NonNullByDefault;
+import org.eclipse.jdt.annotation.Nullable;
 import org.openhab.binding.hue.internal.api.dto.clip2.enums.BatteryStateType;
 import org.openhab.core.library.types.DecimalType;
 import org.openhab.core.library.types.OnOffType;
@@ -27,15 +28,18 @@
  */
 @NonNullByDefault
 public class Power {
-    private @NonNullByDefault({}) @SerializedName("battery_state") String batteryState;
+    private @Nullable @SerializedName("battery_state") String batteryState;
     private @SerializedName("battery_level") int batteryLevel;
 
     public BatteryStateType getBatteryState() {
-        try {
-            return BatteryStateType.valueOf(batteryState.toUpperCase());
-        } catch (IllegalArgumentException e) {
-            return BatteryStateType.CRITICAL;
+        String batteryState = this.batteryState;
+        if (batteryState != null) {
+            try {
+                return BatteryStateType.valueOf(batteryState.toUpperCase());
+            } catch (IllegalArgumentException e) {
+            }
         }
+        return BatteryStateType.CRITICAL;
     }
 
     public int getBatteryLevel() {