fixed method "resetLayer" in terms of deleting layer labels

[viatkinviatkin]

Changing the layer settings triggers the base-layer/leafletOptionsDidChange handler, which executes
_resetLayer->_destroyLayer->_removeLayerFromLeafletContainer,
removing the previous leafletLayer instance from the leafletContainer

this.get('_labelsLayer') of labels is a linked separate layer with this.get('_leafletObject')
and will be removed only when this.get('labelSettings.signMapObjects') === true

But this.get('labelSettings.signMapObjects') can be changed BEFORE _resetLayer, which will leave this.get('_labelsLayer') in the leafletContainer.

It is necessary to remove this.get('_labelsLayer') from the leafletContainer whenever _removeLayerFromLeafletContainer is called

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
0.0% Duplication

[DaryaNeko]

Так, ну во-первых, если уж мы озаботились удалением надписей при изменении флага signMapObjects, то как минимум надо это сделать и для additionalZoomLabel (см. выше)

При этом, у нас есть еще и отдельный метод _removeLabelsFromLeafletContainer.
Возможно, разумнее логику по скрытию надписей перенести в него, полностью убрав из _removeLayerFromLeafletContainer. А в метод _destroyLayer добавить вызов _removeLabelsFromLeafletContainer
Возможно нужен какой-то параметр, ибо оба метода _remove... вызываются еще и при отключении видимости слоя. и при включении видимости мы хотим снова увидеть объекты и надписи, которые уже были на карте. Но ты слой не просто убираешь с карты, ты его очищаешь (проверь этот момент, но по-моему надписи обратно не включатся с твоим изменением, если они не в additionalZoomLabel были)

[DaryaNeko]

тут опечатка была просто?

[viatkinviatkin]

Да