fxdaytrader wrote:See latest version attached, I think the sl-stuff works now. Set the new external variables as you need them (sl-section).

I think there was one bloop in the tp-section:

Code: Select all

if (OpenTrades > 0) take = lp;

in the original version it should be > 1, but as we have the option to open multiple trades now (maintrade, reentry, stacks) I changed that to

Code: Select all

 if (TradeComment==StackTradeComment) take = lp;

Do not forget to set LoadCustomMASettings=false, otherwise you load differen ma-settings (as I use them on one account for testing) ...

Bob and Shelley Again.v1o.PendingOrders.fxdaytrader.mq4

fxdaytrader wrote:but no errors as well?

fxdaytrader wrote:oha, it opened 2 orders at the same time?

Try the following please:

go to the code into the function bool TooClose()

find the line (line 3 or 4 there):

Code: Select all

   if (OrdersHistoryTotal() == 0) return(false);

direct under that line insert the following:

Code: Select all

//fxdaytrader, check for pending orders  
if (UsePendingStopOrders) {
 for (int cd = OrdersTotal() - 1; cd >= 0; cd--) {
  if (!OrderSelect(cd, SELECT_BY_POS) ) continue;
  if (OrderMagicNumber() != MagicNumber) continue;
  if (OrderSymbol() != Symbol()) continue;
  if (OrderType() != OP_BUYSTOP) continue;
  if (OrderType() != OP_SELLSTOP) continue;
      
  if (TimeCurrent() - OrderOpenTime() < (PendingOrderHighLowTF * 60)) return(true);
  
 }//end for (int cd = OrdersTotal()
}//end if (UsePendingStopOrders) {
//end fxdaytrader

Not 100% sure, but I think this helps