From 22d9b128b926393e5afd75c71b8e089c3ab51bf9 Mon Sep 17 00:00:00 2001 From: Thomas Basler Date: Fri, 4 Aug 2023 23:44:18 +0200 Subject: [PATCH] Fix: Move setting of CMD_PENDING state into the right order --- lib/Hoymiles/src/inverters/HM_Abstract.cpp | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/lib/Hoymiles/src/inverters/HM_Abstract.cpp b/lib/Hoymiles/src/inverters/HM_Abstract.cpp index c40ad44e..7da96929 100644 --- a/lib/Hoymiles/src/inverters/HM_Abstract.cpp +++ b/lib/Hoymiles/src/inverters/HM_Abstract.cpp @@ -64,8 +64,8 @@ bool HM_Abstract::sendAlarmLogRequest(bool force) auto cmd = _radio->prepareCommand(); cmd->setTime(now); cmd->setTargetAddress(serial()); - _radio->enqueCommand(cmd); EventLog()->setLastAlarmRequestSuccess(CMD_PENDING); + _radio->enqueCommand(cmd); return true; } @@ -114,8 +114,8 @@ bool HM_Abstract::sendSystemConfigParaRequest() auto cmd = _radio->prepareCommand(); cmd->setTime(now); cmd->setTargetAddress(serial()); - _radio->enqueCommand(cmd); SystemConfigPara()->setLastLimitRequestSuccess(CMD_PENDING); + _radio->enqueCommand(cmd); return true; } @@ -136,8 +136,8 @@ bool HM_Abstract::sendActivePowerControlRequest(float limit, PowerLimitControlTy auto cmd = _radio->prepareCommand(); cmd->setActivePowerLimit(limit, type); cmd->setTargetAddress(serial()); - _radio->enqueCommand(cmd); SystemConfigPara()->setLastLimitCommandSuccess(CMD_PENDING); + _radio->enqueCommand(cmd); return true; } @@ -162,8 +162,8 @@ bool HM_Abstract::sendPowerControlRequest(bool turnOn) auto cmd = _radio->prepareCommand(); cmd->setPowerOn(turnOn); cmd->setTargetAddress(serial()); - _radio->enqueCommand(cmd); PowerCommand()->setLastPowerCommandSuccess(CMD_PENDING); + _radio->enqueCommand(cmd); return true; } @@ -179,8 +179,8 @@ bool HM_Abstract::sendRestartControlRequest() auto cmd = _radio->prepareCommand(); cmd->setRestart(); cmd->setTargetAddress(serial()); - _radio->enqueCommand(cmd); PowerCommand()->setLastPowerCommandSuccess(CMD_PENDING); + _radio->enqueCommand(cmd); return true; }