I understand how this can be confusing, but that's how the AP is intended to work. The idea is that the user enters the value first and only then ticks the checkbox. Having a zero value in that condition is a totally valid situation.1- I open the EA, no options are selected.
2- The I have chosen the first three option in the actions tab
a) Close 100%
b) Delete all pending orders
c) Disable autotrading
3- Then I have open four orders. Because of the spread they were all negative around -100 USD.
4- Then I have Condition tab I have chosen "Floating point rises to currency units:"
and I have not changed the value from 0 to any other value.
Then in the same moment the EA has closed all the four orders.
In the MT4 experts tab the following message appeared;
Account Protector AUDCAD M1: CONDITIONS IS MET: Floating loss rose to 0.00 USD
In my opinion
the following condition should be added (in the file Account Protector.mqh function CAccountProtector::CheckAllConditions()):
// Floating loss rose to <Actual number> <currency ISO code>.
if ((!DisableFloatLossRiseCurr) &&
(sets.doubleLossQuanUnits != 0) && // This additional validation should be done , exclude the case when it is zero
(floating_profit <= -sets.doubleLossQuanUnits))
Besides this situation all the others variables fields in the Conditions should be changed to include this new condition. To only consider it when the respective value is not zero.
Email address (and other related settings) are controlled via MetaTrader's options: Tools->Options->Email.1- The send email is not working, there is not place to add the destination email address. Why ?
I'm not sure what exactly you mean here, but you might want to look at the DoNotDisableConditions and DoNotDisableActions input parameters.2- I would be nice if there was an option to save the configurations the user prefers.
Because every time I want to use it all the options I have selected before disappeared, specially when I have selected the "Disable autotrading" option
and I press the Auto Trading button back to on.
You mean something like a global "Enable/Disable" button? I will consider adding this. Thanks!3- I don't know if it makes sense, but it could be useful and option to trigger the EA only when the user wants, that is the user setups all the
EA options and only when he wants the EA will kickoff and execute the actions according to the configurations. Otherwise the EA triggers immediatly
just before you have finished all the setup steps, especially if you don't have left the conditions setup as the last step.