From 3d25ed7994ec604e03acd21a6ac73ba39adfd864 Mon Sep 17 00:00:00 2001 From: Gerald Hansen Date: Sun, 8 Dec 2019 22:27:18 +0100 Subject: [PATCH] Change state values for Worx Landroid sensor (#27453) The obj["state"] contains already several named return states as following: "grass cutting", "trapped recovery", "searching wire", "following wire", "searching home", "home", "idle" And with the batteryChargerState also the "charging" Fixes #455 --- homeassistant/components/worxlandroid/sensor.py | 13 +++---------- 1 file changed, 3 insertions(+), 10 deletions(-) diff --git a/homeassistant/components/worxlandroid/sensor.py b/homeassistant/components/worxlandroid/sensor.py index ad583d6d943..dd74c5b7d17 100644 --- a/homeassistant/components/worxlandroid/sensor.py +++ b/homeassistant/components/worxlandroid/sensor.py @@ -141,16 +141,9 @@ class WorxLandroidSensor(Entity): state = self.get_error(obj) if state is None: - state_obj = obj["settaggi"] + if obj["batteryChargerState"] == "charging": + return obj["batteryChargerState"] - if state_obj[14] == 1: - return "manual-stop" - if state_obj[5] == 1 and state_obj[13] == 0: - return "charging" - if state_obj[5] == 1 and state_obj[13] == 1: - return "charging-complete" - if state_obj[15] == 1: - return "going-home" - return "mowing" + return obj["state"] return state