SampleOnOrderUpdate

[DaveS]

The following is a scrap of code from the OnExecution method.

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

The comment does not seem to match the conditional logic.

In any event it seems to me that this conditional could never become true since it is contained within another conditional that has 'execution.Order.OrderState == OrderState.PartFilled' OR'd with other contitions.

Am I missing something?

[NinjaTrader_JoshP]

DaveS,

It works as intended as is. The overarching if-statement has many possible OrderStates. You want to reset the entryOrder to null on all OrderStates except the PartFilled one.

[DaveS]

Yes, sorry, I misread the outer 'if' and that lead to a wrong expectation for the inner one - doh!