Dry Contact Limit sensor improvements (ratgdo#398)

Use the limit sensors to determine if door action should be ignored

Set door position when limit sensors are triggered

Author: breakingspell

components/ratgdo/dry_contact.cpp

@@ -90,12 +90,12 @@ namespace ratgdo {
 
         void DryContact::door_action(DoorAction action)
         {
-            if (action == DoorAction::OPEN && this->door_state_ != DoorState::CLOSED) {
-                ESP_LOGW(TAG, "The door is not closed. Ignoring door action: %s", DoorAction_to_string(action));
+            if (action == DoorAction::OPEN && this->open_limit_reached_) {
+                ESP_LOGW(TAG, "The door is already fully open. Ignoring door action: %s", LOG_STR_ARG(DoorAction_to_string(action)));
                 return;
             }
-            if (action == DoorAction::CLOSE && this->door_state_ != DoorState::OPEN) {
-                ESP_LOGW(TAG, "The door is not open. Ignoring door action: %s", DoorAction_to_string(action));
+            if (action == DoorAction::CLOSE && this->close_limit_reached_) {
+                ESP_LOGW(TAG, "The door is already fully closed. Ignoring door action: %s", LOG_STR_ARG(DoorAction_to_string(action)));
                 return;
             }
 

components/ratgdo/ratgdo.cpp

@@ -923,6 +923,7 @@ namespace ratgdo {
         dry_contact_open_sensor_ = dry_contact_open_sensor;
         dry_contact_open_sensor_->add_on_state_callback([this](bool sensor_value) {
             this->protocol_->set_open_limit(sensor_value);
+            this->door_position = 1.0;
         });
     }
 
@@ -931,6 +932,7 @@ namespace ratgdo {
         dry_contact_close_sensor_ = dry_contact_close_sensor;
         dry_contact_close_sensor_->add_on_state_callback([this](bool sensor_value) {
             this->protocol_->set_close_limit(sensor_value);
+            this->door_position = 0.0;
         });
     }