Indicator -> Automated Trade - Keeps firing - page 2

Ok, i'll work out how to do that later tonight. Need to take kids to practice now.

Thanks very much for your help guys. Ended up being a stupid mistake on my part and not having the spelling right. I can now see why it was failing, it's not returning an empty_value when not correct, rather a 1 (true) when it is valid. So my test (below) was incorrect in any case.

I'll correct this if statement later tonight. Can I convert to a boolean like in python or is it old school test like (var == 1.0)?

   double ArrowUP=iCustom(Symbol(), 0, indicator1, 0, 1);  
   double ArrowDN=iCustom(Symbol(), 0, indicator1, 1, 1);  
   
   double HistoUP=iCustom(Symbol(), 0, indicator2, 0, 1);  
   double HistoDN=iCustom(Symbol(), 0, indicator2, 1, 1);  
   
   Comment("ArrowUp: ", ArrowUP, "\nArrowDn: ", ArrowDN, "\nHistoUp: ", HistoUP, "\nHistoDn: ", HistoDN);
   
   // Incorrect test below
   if ((ArrowDN != EMPTY_VALUE ) && (HistoDN != EMPTY_VALUE ) && (CheckMoneyForTrade(Symbol(), lots, 1))) 
   {
      Comment("Sell - ", TimeToStr(TimeCurrent(),TIME_SECONDS));
   }
   
   if ((ArrowUP != EMPTY_VALUE ) && (HistoUP != EMPTY_VALUE ) && (CheckMoneyForTrade(Symbol(), lots, 0))) 
   {
      Comment("Buy - ", TimeToStr(TimeCurrent(),TIME_SECONDS));
   }

2147483647 is EMPTY_VALUE, so your comparison is valid. Check your bar index (i.e. last parameter of iCustom()) instead.

bool(EMTPY_VALUE) and bool(1) will give you true...

Aaah apologies, still trying to get used to this forum layout. I'm used to VB4.x sites.

I've got orderopen code in now but getting this error: Trade Disabled so will look at that today. It's not disabled as i have another EA on a different chart that is doing orders fine and autotrade is enabled.

Also, when using iCustom to pull buffer data from an indicator, is that indicator no longer required to be attached to the chart? For example, can i simply use the EA without the indicator now on chart?

1. The indicator is on the chart, just invisible as EAs have no eyes.
2. Unless the indicator has been made a resource of the EA, it needs to be installed.
3. You can just embed the other indicator(s) inside your indicator/EA. Add the CI(s) to your code as a resource.
             Use the publicly released code - Expert Advisors and Automated Trading - MQL5 programming forum
             Resources - MQL4 programs - MQL4 Reference

   Be aware that using resources is 40x times slower than using CIs directly.
             A custom indicator as a resource - MQL4 programming forum 2019.11.26

Ok, so when you say that the indicator is on the chart, does this hold true even if I do not manually add it to the chart? Is the EA when loading what loads the indicator? Sorry for asking the obvious, just wondering whether what I am doing atm is redundant - 

1. Add indicator

2. Add EA (which uses iCustom and the indicator)

Or is it a case of, I only need to add the EA as by adding the EA, it will also add the indicator to the chart.