[BUG] Indicator.override is assigning _prevIndicator to a deep clone making shouldUpdate not working

[char101]

In

KLineChart/src/component/Indicator.ts

Line 385 in 6c839d8

this._prevIndicator = { ...clone(currentOthers), result }

_prevIndicator is a deep clone, while in

KLineChart/src/component/Indicator.ts

Line 344 in 6c839d8

prev.figures !== current.figures ||

attribute like figures are compared using identity. Thus any update to the indicator like toggling visible value will always result in an update.

I think _prevIndicator should be a shallow clone

this._prevIndicator = { ...this }

[char101]

The same logic applies to Overlay which compares extendData and styles which are objects.

KLineChart/src/component/Overlay.ts

Line 276 in 6c839d8

this._prevOverlay = clone(this)

KLineChart/src/component/Overlay.ts

Line 346 in 6c839d8

this._prevOverlay.extendData !== this.extendData ||

[char101]

I was mistaken. Since the override is merged using merge, figures and calcParams which are arrays are overwritten.