StopLossMultiplier updated even when displayed and disabled

[samuelion]

Hello!

First of all, thank you very much for this great tool!
I’ve just discovered it and I really like it!

In the tutorial, it says:
"SL multiplier — multiplier for ATR-based stop-loss. The ATR value is multiplied by this value to produce the actual stop-loss distance. Set to zero to disable ATR-based stop-loss. ATR options are visible only if ShowATROptions input parameter is set to Yes."
However, setting this parameter to 0 only disables it "temporarily".

• When the SL is set in pips, manually moving the SL restores the SL multiplier value (not critical as both are distances from the entry price and are similar concept with just different units but a bit strange).
• But when the SL is set to a specific price (i.e. fixed), manually moving the SL still restores the SL multiplier. This turns the SL back into a variable distance from the entry, even though I had set the multiplier to 0 to disable the ATR-based stop-loss. Is this expected or am I misunderstanding something?

The same behaviour occurs for TP.

Edit :
Was just curious => I think I would have added a check (StopLossMultiplier != 0) in MainViewOnStopLossFieldValueChanged => if (InputShowAtrOptions).

I'm sorry, not so familiar with github =).

Something I found also that may be interested to check : if spread check / spread add to stoploss is enabled, I would also have added it when doing breakeven (and even add few pips to avoid loss but I may open another issue or even try to propose something =) )

Thanks for your help =) and thanks for your huge contribution !

[aamoraru]

Thanks for reporting! This will be fixed in the next version.

[samuelion]

I also update on my side :
private void StopLossLineMoved(object sender, ChartLineMovedEventArgs e)
...
if (Model is { IsAtrModeActive: true, StopLossMultiplier: not 0 })

and did the same for target price =).
Seems to help regarding my issue =).