From 3cdb077f058a4d39d10253c121cbbeca4c7d2434 Mon Sep 17 00:00:00 2001
From: "J. Nick Koston" <nick@koston.org>
Date: Wed, 7 Jun 2023 20:06:39 -0500
Subject: [PATCH] cleanups

---
 components/ratgdo/ratgdo.cpp | 12 +++++-------
 1 file changed, 5 insertions(+), 7 deletions(-)

diff --git a/components/ratgdo/ratgdo.cpp b/components/ratgdo/ratgdo.cpp
index 761875b..2f7a99c 100644
--- a/components/ratgdo/ratgdo.cpp
+++ b/components/ratgdo/ratgdo.cpp
@@ -189,9 +189,13 @@ namespace ratgdo {
 
         } else if (cmd == 0x281) {
             light ^= 1; // toggle bit
+            ESP_LOGD(TAG, "Light: %d (toggle)", light);
+
         } else if (cmd == 0x84) {
         } else if (cmd == 0x285) {
             motion = 1; // toggle bit
+            ESP_LOGD(TAG, "Motion: %d (toggle)", motion);
+
         }
     }
 
@@ -575,10 +579,7 @@ namespace ratgdo {
         if (this->lightState == LightState::LIGHT_STATE_ON) {
             ESP_LOGD(TAG, "The light is already on");
         } else {
-            toggleLight();
-            // We don't always get the state back so be optimistic
-            this->previousLightState = this->lightState;
-            this->lightState = LightState::LIGHT_STATE_ON;            
+            toggleLight(); 
         }
     }
 
@@ -588,9 +589,6 @@ namespace ratgdo {
             ESP_LOGD(TAG, "The light is already off");
         } else {
             toggleLight();
-            // We don't always get the state back so be optimistic
-            this->previousLightState = this->lightState;
-            this->lightState = LightState::LIGHT_STATE_OFF;
         }
     }