Break Even in OnOrderUpdate

[daxtrading]

I am using OnExecution to trigger submission of stop/target orders.
When I raise the stop-loss order to breakeven in OnBarUpdate nothing happens.
I know it has to do with the code below (from the SampleOnOrderUpdate) that shows only the EntryLong order. When I comment the lines for ExitShort order everything works just fine.

if (entryOrder != null && entryOrder == execution.Order)
{
if (execution.Order.OrderState == OrderState.Filled || execution.Order.OrderState == OrderState.PartFilled || (execution.Order.OrderState == OrderState.Cancelled && execution.Order.Filled > 0))
{
// Stop-Loss order 10 ticks below our entry price
stopOrder = ExitLongStop(0, true, execution.Order.Filled, execution.Order.AvgFillPrice-(10*TickSize), "StopEL1", "EL1");

// Stop-Loss order 10 ticks above our entry price
stopOrder = ExitShortStop(0, true, execution.Order.Filled, execution.Order.AvgFillPrice+(10*TickSize), "StopES1", "ES1");

// Target order 50 ticks above our entry price
targetOrder = ExitLongLimit(0, true, execution.Order.Filled, execution.Order.AvgFillPrice + 50 * TickSize, "TargetEL1", "EL1");

// Target order 50 ticks below our entry price
targetOrder = ExitShortLimit(0, true, execution.Order.Filled, execution.Order.AvgFillPrice - 50 * TickSize, "TargetES1", "ES1");

// Resets the entryOrder object to null after the order has been filled or partially filled
if (execution.Order.OrderState != OrderState.PartFilled)
{
entryOrder = null;
}
}
}

[NinjaTrader_VinceB]

Maarten, this is very likely a coding issue in your script – as you’re using the same IOrder object for the Long and Short side here, I would recommend using a dedicated object per side.

Also, please ensure the reset parts as outlined in the sample script are working in your code as expected as well, best is to print all states associated with the object to debug your custom code modifications.

For tips on debugging, please check this tip out:

http://www.ninjatrader.com/support/forum/showthread.php?t=3418

[daxtrading]

I already thought about using a dedicated object per side and that fixed it.
Thanks for your reply.