[tesla] Fix update timestamp and null exception for offline state (op…

…enhab#15756) Signed-off-by: Bill Forsyth <git@billforsyth.net>
querdenker2k · Oct 21, 2023 · 33986c3 · 33986c3
1 parent ee4c862
commit 33986c3
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/...binding.tesla/src/main/java/org/openhab/binding/tesla/internal/TeslaBindingConstants.java b/...binding.tesla/src/main/java/org/openhab/binding/tesla/internal/TeslaBindingConstants.java
@@ -99,6 +99,7 @@ public enum EventKeys {
 
     public static final String CHANNEL_CHARGE = "charge";
     public static final String CHANNEL_COMBINED_TEMP = "combinedtemp";
+    public static final String CHANNEL_EVENTSTAMP = "eventstamp";
 
     // thing configurations
     public static final String CONFIG_ALLOWWAKEUP = "allowWakeup";

diff --git a/...g.tesla/src/main/java/org/openhab/binding/tesla/internal/handler/TeslaVehicleHandler.java b/...g.tesla/src/main/java/org/openhab/binding/tesla/internal/handler/TeslaVehicleHandler.java
@@ -54,6 +54,7 @@
 import org.openhab.binding.tesla.internal.throttler.QueueChannelThrottler;
 import org.openhab.binding.tesla.internal.throttler.Rate;
 import org.openhab.core.io.net.http.WebSocketFactory;
+import org.openhab.core.library.types.DateTimeType;
 import org.openhab.core.library.types.DecimalType;
 import org.openhab.core.library.types.IncreaseDecreaseType;
 import org.openhab.core.library.types.OnOffType;
@@ -834,6 +835,7 @@ public void parseAndUpdate(String request, String payLoad, String result) {
         try {
             if (request != null && result != null && !"null".equals(result)) {
                 updateStatus(ThingStatus.ONLINE);
+                updateState(CHANNEL_EVENTSTAMP, new DateTimeType());
                 // first, update state objects
                 if ("queryVehicle".equals(request)) {
                     if (vehicle != null) {
@@ -844,8 +846,8 @@ public void parseAndUpdate(String request, String payLoad, String result) {
                         return;
                     }
 
-                    if (vehicle != null && "asleep".equals(vehicle.state)) {
-                        logger.debug("Vehicle is asleep.");
+                    if (vehicle != null && ("asleep".equals(vehicle.state) || "offline".equals(vehicle.state))) {
+                        logger.debug("Vehicle is {}", vehicle.state);
                         return;
                     }