3.01

Hotfix for bugs with breakeven and hotkey.
EarnForex · Jul 28, 2022 · 765c85e · 765c85e
1 parent 53a5c34
commit 765c85e
Show file tree

Hide file tree

Showing 4 changed files with 31 additions and 13 deletions.
diff --git a/MQL4/Experts/Position Sizer/Position Sizer Trading.mqh b/MQL4/Experts/Position Sizer/Position Sizer Trading.mqh
@@ -437,7 +437,7 @@ void DoBreakEven()
             }
             else if (OrderType() == OP_SELL)
             {
-                double BE = NormalizeDouble(OrderOpenPrice() - sets.TrailingStopPoints * _Point, _Digits);
+                double BE = NormalizeDouble(OrderOpenPrice() - sets.BreakEvenPoints * _Point, _Digits);
                 if ((Ask <= BE) && ((OrderOpenPrice() < OrderStopLoss()) || (OrderStopLoss() == 0))) // Only move to breakeven if the current stop-loss is higher (or zero).
                 {
                     // Write Open price to the SL field.

diff --git a/MQL4/Experts/Position Sizer/Position Sizer.mq4 b/MQL4/Experts/Position Sizer/Position Sizer.mq4
@@ -5,8 +5,8 @@
 //+------------------------------------------------------------------+
 #property copyright "EarnForex.com"
 #property link      "https://www.earnforex.com/metatrader-expert-advisors/Position-Sizer/"
-#property version   "3.00"
-string    Version = "3.00";
+#property version   "3.01"
+string    Version = "3.01";
 #property strict
 
 #property description "Calculates risk-based position size for your account."
@@ -111,14 +111,18 @@ bool Dont_Move_the_Panel_to_Default_Corner_X_Y;
 uint LastRecalculationTime = 0;
 bool StopLossLineIsBeingMoved = false;
 bool TakeProfitLineIsBeingMoved = false;
-uchar MainKey = 0;
-bool CtrlRequired = false;
-bool ShiftRequired = false;
+uchar MainKey;
+bool CtrlRequired;
+bool ShiftRequired;
 bool NeedToCheckToggleScaleOffOn;
 int PrevChartWidth;
 
 int OnInit()
 {
+    CtrlRequired = false;
+    ShiftRequired = false;
+    MainKey = 0;
+
     ExtDialog = new CPositionSizeCalculator;
     PrevChartWidth = 0;
 
@@ -316,9 +320,14 @@ int OnInit()
                 if (keys[i] == "SHIFT") ShiftRequired = true;
                 else if (keys[i] == "CTRL") CtrlRequired = true;
             }
+            StringToUpper(keys[n - 1]);
             MainKey = (uchar)StringGetCharacter(keys[n - 1], 0);
         }
-        else MainKey = (uchar)StringGetCharacter(keys[0], 0); // Just a single key.
+        else
+        {
+            StringToUpper(keys[0]);
+            MainKey = (uchar)StringGetCharacter(keys[0], 0); // Just a single key.
+        }
     }
 
     if (!EventSetTimer(1)) Print("Error setting timer: ", GetLastError());

diff --git a/MQL5/Experts/Position Sizer/Position Sizer Trading.mqh b/MQL5/Experts/Position Sizer/Position Sizer Trading.mqh
diff --git a/MQL5/Experts/Position Sizer/Position Sizer.mq5 b/MQL5/Experts/Position Sizer/Position Sizer.mq5
@@ -5,8 +5,8 @@
 //+------------------------------------------------------------------+
 #property copyright "EarnForex.com"
 #property link      "https://www.earnforex.com/metatrader-expert-advisors/Position-Sizer/"
-#property version   "3.00"
-string    Version = "3.00";
+#property version   "3.01"
+string    Version = "3.01";
 
 #property description "Calculates risk-based position size for your account."
 #property description "Allows trade execution based the calculation results.\r\n"
@@ -122,12 +122,16 @@ bool Dont_Move_the_Panel_to_Default_Corner_X_Y = true;
 uint LastRecalculationTime = 0;
 bool StopLossLineIsBeingMoved = false;
 bool TakeProfitLineIsBeingMoved = false;
-uchar MainKey = 0;
-bool CtrlRequired = false;
-bool ShiftRequired = false;
+uchar MainKey;
+bool CtrlRequired;
+bool ShiftRequired;
 
 int OnInit()
 {
+    CtrlRequired = false;
+    ShiftRequired = false;
+    MainKey = 0;
+
     ExtDialog = new CPositionSizeCalculator;
 
     MathSrand(GetTickCount() + 293029); // Used by CreateInstanceId() in Dialog.mqh (standard library). Keep the second number unique across other panel indicators/EAs.
@@ -323,9 +327,14 @@ int OnInit()
                 if (keys[i] == "SHIFT") ShiftRequired = true;
                 else if (keys[i] == "CTRL") CtrlRequired = true;
             }
+            StringToUpper(keys[n - 1]);
             MainKey = (uchar)StringGetCharacter(keys[n - 1], 0);
         }
-        else MainKey = (uchar)StringGetCharacter(keys[0], 0); // Just a single key.
+        else
+        {
+            StringToUpper(keys[0]);
+            MainKey = (uchar)StringGetCharacter(keys[0], 0); // Just a single key.
+        }
     }
 
     if (!EventSetTimer(1)) Print("Error setting timer: ", GetLastError());