Remove dataview method does not stop fetching data from dataChanged event

[oriolbx]

Context

The remove dataview event, removes the dataview from the client element, but it does not stop fetching data when there is the dataChanged event.

You need to use the disable() method to stop fetching data from the dataview after remove it from the client.

This leads to confusion when using these methods on CARTO.js to deal with dataviews.

Steps to Reproduce

1. Create a map with one dataview that contains the dataChanged event
2. Apply the removeDataview method, the dataview is removed from the client, but it still keeps fetching data when you move around the map or change the zoom level.
3. If you add the disable() method, then the dataview stops being updated when the map position or zoom changes.

Expected result

The dataview should stop being updated once it is removed from the client using the removeDataview() method.

There should be no need to add the disable() method after applying the removeDataview() method.

I'm adding an HTML example with this issue:

test_rm_dataview.html.zip

cc @Jesus89

[Jesus89]

Thanks for the feedback!

This should be an easy one. Just adding dataview.disable() to the client.removeDataview() function:

carto.js/src/api/v4/client.js

Line 295 in 1c362ce

Client.prototype.removeDataview = function (dataview) {

[rubenmoya]

Fixed in version 4.0.14