Tweak target series getter for visualMap, collection target series each time.

Otherwise, setOption at the second time will get wrong target series if it not specified.

Author: 100pah

src/component/visualMap/ContinuousModel.js

@@ -39,7 +39,6 @@ define(function(require) {
         optionUpdated: function (newOption, isInit) {
             ContinuousModel.superApply(this, 'optionUpdated', arguments);
 
-            this.resetTargetSeries();
             this.resetExtent();
 
             this.resetVisual(function (mappingOption) {

src/component/visualMap/PiecewiseModel.js

@@ -81,7 +81,6 @@ define(function(require) {
              */
             this._pieceList = [];
 
-            this.resetTargetSeries();
             this.resetExtent();
 
             /**

src/component/visualMap/VisualMapModel.js

@@ -64,7 +64,8 @@ define(function(require) {
             zlevel: 0,
             z: 4,
 
-            seriesIndex: null,       // 所控制的series indices，默认所有有value的series.
+            seriesIndex: 'all',     // 'all' or null/undefined: all series.
+                                    // A number or an array of number: the specified series.
 
                                     // set min: 0, max: 200, only for campatible with ec2.
                                     // In fact min max should not have default value.
@@ -181,26 +182,31 @@ define(function(require) {
             );
         },
 
-
         /**
          * @protected
+         * @return {Array.<number>} An array of series indices.
          */
-        resetTargetSeries: function () {
-            var thisOption = this.option;
-            var allSeriesIndex = thisOption.seriesIndex == null;
-            thisOption.seriesIndex = allSeriesIndex
-                ? [] : modelUtil.normalizeToArray(thisOption.seriesIndex);
+        getTargetSeriesIndices: function () {
+            var optionSeriesIndex = this.option.seriesIndex;
+            var seriesIndices = [];
+
+            if (optionSeriesIndex == null || optionSeriesIndex === 'all') {
+                this.ecModel.eachSeries(function (seriesModel, index) {
+                    seriesIndices.push(index);
+                });
+            }
+            else {
+                seriesIndices = modelUtil.normalizeToArray(optionSeriesIndex);
+            }
 
-            allSeriesIndex && this.ecModel.eachSeries(function (seriesModel, index) {
-                thisOption.seriesIndex.push(index);
-            });
+            return seriesIndices;
         },
 
         /**
          * @public
          */
         eachTargetSeries: function (callback, context) {
-            zrUtil.each(this.option.seriesIndex, function (seriesIndex) {
+            zrUtil.each(this.getTargetSeriesIndices(), function (seriesIndex) {
                 callback.call(context, this.ecModel.getSeriesByIndex(seriesIndex));
             }, this);
         },

test/bar.html

@@ -24,11 +24,6 @@
             }
         </style>
 
-        <h1>
-            <input type="button" onclick="testHelper.setURLParam(['cn', 'en'], 'cn')" value="CN"/>
-            <input type="button" onclick="testHelper.setURLParam(['cn', 'en'], 'en')" value="EN"/>
-        </h1>
-
         <div id="main" class="chart"></div>
         <script>