single sendmail event sends 50 emails

[jcash]

i have been using sendmail to receive trade and pnl updates for months now with perfect results. after the most recent NT update i now receive 25-50 emails on the first sendmail event of the day, then every sendmail event after just one is sent.

i have a binary bool that prevents the code from being executed a second time:

if (longoda == 0)
{
                    if (Close[0] > Close[1])
                    {
                        longoda = 1;
                        EnterLong(1, "LE");
                        SendMail("[email protected]", "Trade Alert - LE", "  LE:  "+Instrument.FullName+"   "+GetCurrentAsk()+
                        "\r\n  RPNL:  "+Math.Round(daypnl, 2)+"  UPNL:  "+Math.Round(openpnl, 2)+"\r\n  Date/Time:  "+ToDay(Time[0])+"    "+ToTime(Time[0]));
                    }
}

it's annoying erasing all the emails. again this code has served me well for months. it just started happening with the recent update.

did the last NT update have a sendmail change?

[NinjaTrader_Jim]

Hello jcash,

The only noted change I see in the Release Notes page that is relevant to SendMail() has to do with fixing an error when creating a strategy with this method through the Strategy Builder.

I have tested this in an indicator under a doOnce condition, and I only received one email.

I would suggest to add prints where you are placing the SendMail() call so you can determine if the strategy is calling SendMail() multiple times.

Here is the code I have tested with:

private bool doOnce = true;
protected override void OnBarUpdate()
{
	if(State == State.Historical)
		return;
	if(doOnce)
	{
		SendMail("[email protected]", "Hello", "Test");
		doOnce = false;
	}
}

I’ve provided a link covering debugging which you may find helpful.

Debugging: http://ninjatrader.com/support/forum...ead.php?t=3418

Please let me know if I may be of further assistance.

[jcash]

thanks Jim.