I don't think it works to call plot() from a TMF.
Quote
If plot is called several times with the same Name in the same run cycle, only the last data value is plotted.
https://manual.zorro-project.com/plot.htm

More importantly, why are you using the value TradePriceClose when it is impossible to know what TradePriceClose is because your trade is still open? Use priceClose() instead.