What is  wrong in this code ?

I have an indicator that crosses an average of the indicator as a buy/sell signal , in this case going short.

The closing of  this position is done by the reverse cross. (see picture test)

However the execution in backtest shows that the position is covered randomness without taking in account the “closing condition” ; worse it opens a new short without satisfying the “condition”…

Where is the problem in this code?

Any suggestion is welcome.

// Definition of code parameters
DEFPARAM CumulateOrders = False // Cumulating positions deactivated

// Prevents the system from placing new orders on specified days of the week
daysForbiddenEntry = OpenDayOfWeek = 6 OR OpenDayOfWeek = 0

// Conditions to enter long positions
indicator1 = DPO[8](close)
indicator2 = Average[5](DPO[8](close))
c1 = (indicator1 CROSSES OVER indicator2)

//IF c1 AND not daysForbiddenEntry THEN
//BUY 1 SHARES AT MARKET
//ENDIF

// Conditions to enter short positions
indicator3 = DPO[8](close)
indicator4 = Average[5](DPO[8](close))
c2 = (indicator3[1] CROSSES UNDER indicator4[1])
//c3 = (indicator3[1] CROSSES OVER indicator4[1])
IF c2 AND not daysForbiddenEntry THEN
SELLSHORT 1 SHARES AT MARKET
ENDIF

IF c1 THEN
EXITSHORT AT MARKET
ENDIF

TEST-1.jpg