fix bug

simatic-ax · Oct 18, 2024 · 5a152c6 · 5a152c6
1 parent 0d587c2
commit 5a152c6
Show file tree

Hide file tree

Showing 3 changed files with 1 addition and 8 deletions.
diff --git a/src/Actuator/TimeBasedActuator.st b/src/Actuator/TimeBasedActuator.st
@@ -23,8 +23,6 @@ NAMESPACE Simatic.Ax.axftcmlib
             IF (Q_ToWorkPosition = NULL) THEN 
                 THIS.SetError(WORD#16#8101);
                 RETURN;
-            ELSE
-                THIS.SetError(FALSE);
             END_IF;
             _tonOnDuration(signal := Q_ToWorkPosition.IsOn(), Duration := OnDuration);
             CASE _state OF

diff --git a/src/Actuator/TwoWayActuator.st b/src/Actuator/TwoWayActuator.st
@@ -22,8 +22,6 @@ NAMESPACE Simatic.Ax.axftcmlib
             IF (Q_ToWorkPosition = NULL) THEN
                 THIS.SetError(WORD#16#8101);
                 RETURN;
-            ELSE
-                THIS.SetError(FALSE);
             END_IF;
             CASE _state OF
                 // Switch all outputs off, when actuator is idle in home position

diff --git a/test/TwoWayActuatorTest.st b/test/TwoWayActuatorTest.st
@@ -44,10 +44,7 @@ NAMESPACE Simatic.Ax.axftcmlib
             busy := cmd.Busy();
             IF NOT(busy) THEN
                 done := cmd.Done();
-            ELSE 
-                busy := TRUE;
-            END_IF;  
-            busy := TRUE;
+            END_IF; 
             AxUnit.Assert.Equal(expected := TRUE, actual := busy);
             AxUnit.Assert.Equal(expected := FALSE, actual := done);
             AxUnit.Assert.Equal(expected := TRUE, actual := _qToWork.IsOn());