Expected: order will close on defined TP
Actually happening: order closes on same price as filled
Code used:
engine.submitOrder(getLabel(instrument), instrument, IEngine.OrderCommand.SELL, this.orderAmount, 0, 10, 0, tick.getBid() - this.parTakeProfit);
-getLabel - function included in the demo code


@1.3074 TP: 1.3073onMessage: ORDER_SUBMIT_OK - Order submitted order: OPENED SELL 0.001 @ 0.0 OID: 41

onMessage: ORDER_FILL_OK - Order filled order: FILLED SELL 0.001 @ 1.3074 OID: 41

onMessage: ORDER_CLOSE_OK - Order closed order: CLOSED SELL 0.001 @ 1.3074 OID: 41

@1.30717 TP: 1.30707onMessage: ORDER_SUBMIT_OK - Order submitted order: OPENED SELL 0.001 @ 0.0 OID: 42

onMessage: ORDER_FILL_OK - Order filled order: FILLED SELL 0.001 @ 1.30717 OID: 42

onMessage: ORDER_CLOSE_OK - Order closed order: CLOSED SELL 0.001 @ 1.30717 OID: 42

@1.30703 TP: 1.30693onMessage: ORDER_SUBMIT_OK - Order submitted order: OPENED SELL 0.001 @ 0.0 OID: 43

onMessage: ORDER_FILL_OK - Order filled order: FILLED SELL 0.001 @ 1.30703 OID: 43

onMessage: ORDER_CLOSE_OK - Order closed order: CLOSED SELL 0.001 @ 1.30703 OID: 43

Please provide a full example strategy which replicates the case. Please provide a screenshot of the used Historical Tester settings.

I just saw HistoricalTester is a jforex platform component not a jforex-sdk component. Evaluations are from jforex-sdk.

public void onTick(Instrument instrument, ITick tick) throws JFException {
        List<IOrder> openOrders = engine.getOrders(instrument);
        Double ask = tick.getAsk();
        Double bid = tick.getBid();

        if(openOrders.isEmpty()){
            sendBothMarketTP(instrument, engine, tick);
        }
        else{
            //get chan population
            Map<Boolean, Double> chanPopulation = new HashMap<Boolean, Double>();
            for(IOrder order: openOrders){
                Double price = order.getOpenPrice();
                Boolean position = order.isLong();

                if(position){
                    if(Math.abs(price - ask) < this.parChanWidth){
                        chanPopulation.put(position, price);
                    }
                }
                else{
                    if(Math.abs(price - bid) < this.parChanWidth){
                        //two with same position shouldn't be in a chan
                        chanPopulation.put(position, price);
                    }
                }

            }
            //check if anything at all in chan
            if(chanPopulation.isEmpty()){
                sendBothMarketTP(instrument, engine, tick);
            }
            //saturate chan if needed
            else if(chanPopulation.size() == 1){
                Boolean position = chanPopulation.keySet().iterator().next();
                if(position){
                    console.getOut().print(" @" + tick.getBid() + " TP: " + Double.toString(tick.getBid() - this.parTakeProfit));
                    engine.submitOrder(myUtils.getLabel(instrument), instrument, IEngine.OrderCommand.SELL, this.orderAmount, 0, 10, 0, _tick.getBid() - this.parTakeProfit);
                }
                else{
                    console.getOut().print(" @" + tick.getAsk() + " TP: " + Double.toString(tick.getAsk() + this.parTakeProfit));
                    engine.submitOrder(myUtils.getLabel(instrument), instrument, IEngine.OrderCommand.BUY, this.orderAmount, 0, 10, 0, _tick.getAsk() + this.parTakeProfit);
                }

            }
        }

    }


public void sendBothMarketTP(Instrument _instrument, IEngine _engine, ITick _tick) throws JFException{
        _engine.submitOrder(myUtils.getLabel(_instrument), _instrument, IEngine.OrderCommand.BUY, this.orderAmount, 0, 10, 0, _tick.getAsk() + this.parTakeProfit);
        _engine.submitOrder(myUtils.getLabel(_instrument), _instrument, IEngine.OrderCommand.SELL, this.orderAmount, 0, 10, 0, _tick.getBid() - this.parTakeProfit);
    }

Please provide a complete example strategy which replicates the case. If you launch the strategy from the JForex-SDK, please provide the program which launches the strategy.

I modified the my_play that comes with the demo code and attached it.

@1.30647 TP: 1.30657onMessage: ORDER_SUBMIT_OK - Order submitted order: OPENED BUY 0.001 @ 0.0 OID: 4
onMessage: ORDER_FILL_OK - Order filled order: FILLED BUY 0.001 @ 1.30647 OID: 4
onMessage: ORDER_CLOSE_OK - Order closed order: CLOSED BUY 0.001 @ 1.30647 OID: 4

From the wiki and docs this is the preferred way of setting take profit for orders or am I missing something?

We could not replicate those results:

As you see the TP works as expected. So we repeat for the third time - please provide historical tester settings that you achieve such results with.

You're right. In the report the orders are as expected. The message was confusing - it says the order is closed@open_price. Using .getClosedPrice gives the right answer. Tnx.
onMessage: ORDER_FILL_OK - Order filled order: FILLED BUY 0.001 @ 1.31145 closed price:0.0 OID: 2081
onMessage: ORDER_CLOSE_OK - Order closed order: CLOSED BUY 0.001 @ 1.31145 closed price:1.31157 OID: 2081