bug fix in scripting comparison

Author: facontidavide

include/behaviortree_cpp/scripting/operators.hpp

@@ -384,7 +384,7 @@ struct ExprComparison : ExprBase
                (lhs_v.isNumber() && rhs_v.isString()))
       {
         auto lv = lhs_v.cast<double>();
-        auto rv = lhs_v.cast<double>();
+        auto rv = rhs_v.cast<double>();
         if (!SwitchImpl(lv, rv, ops[i]))
         {
           return False;

tests/script_parser_test.cpp

@@ -228,6 +228,10 @@ TEST(ParserTest, Equations)
 
   EXPECT_EQ(GetResult(" y == x  &&  x == 3 ").cast<int>(), 0);
   EXPECT_EQ(GetResult(" y == x  ||  x == 3 ").cast<int>(), 1);
+
+  // we expect string to be casted to number
+  EXPECT_EQ(GetResult(" par1:='3'; par2:=3; par1==par2").cast<int>(), 1);
+  EXPECT_EQ(GetResult(" par1:='3'; par2:=4; par1!=par2").cast<int>(), 1);
 }