reset parsing flags during update when data is parsed

Author: junedchhipa

src/apexcharts.js

@@ -440,7 +440,7 @@ export default class ApexCharts {
     }
 
     if (options.series) {
-      this.updateHelpers.resetParsingFlags()
+      this.data.resetParsingFlags()
 
       this.series.resetSeries(false, true, false)
       if (options.series.length && options.series[0].data) {
@@ -482,7 +482,7 @@ export default class ApexCharts {
    * @param {array} series - New series which will override the existing
    */
   updateSeries(newSeries = [], animate = true, overwriteInitialSeries = true) {
-    this.updateHelpers.resetParsingFlags()
+    this.data.resetParsingFlags()
 
     this.series.resetSeries(false)
     this.updateHelpers.revertDefaultAxisMinMax()
@@ -499,6 +499,8 @@ export default class ApexCharts {
    * @param {array} newSerie - New serie which will be appended to the existing series
    */
   appendSeries(newSerie, animate = true, overwriteInitialSeries = true) {
+    this.data.resetParsingFlags()
+
     const newSeries = this.w.config.series.slice()
     newSeries.push(newSerie)
     this.series.resetSeries(false)
@@ -518,8 +520,8 @@ export default class ApexCharts {
   appendData(newData, overwriteInitialSeries = true) {
     let me = this
 
+    me.data.resetParsingFlags()
     me.w.globals.dataChanged = true
-
     me.series.getPreviousPaths()
 
     let newSeries = me.w.config.series.slice()

src/modules/CoreUtils.js

@@ -279,9 +279,15 @@ class CoreUtils {
       axisSeries.forEach((si) => {
         // series may be bare until loaded in realtime
         if (cnf.series[si] && cnf.series[si].group === undefined) {
-          // A series with no group defined will be named after the axis that
-          // referenced it and thus form a group automatically.
-          cnf.series[si].group = 'apexcharts-axis-'.concat(ai.toString())
+          if (cnf.series[si].__apexParsed) {
+            // For parsed series, use sequential counter to avoid duplicates
+            cnf.series[si].group = 'apexcharts-axis-'.concat(si.toString())
+          } else {
+            // For non-parsed series,
+            // A series with no group defined will be named after the axis that
+            // referenced it and thus form a group automatically.
+            cnf.series[si].group = 'apexcharts-axis-'.concat(ai.toString())
+          }
         }
       })
     })

src/modules/Data.js

@@ -600,6 +600,23 @@ export default class Data {
     return this.w
   }
 
+  /**
+   * Reset parsing flags to allow re-parsing of data during updates
+   */
+  resetParsingFlags() {
+    const w = this.w
+    w.globals.dataWasParsed = false
+    w.globals.originalSeries = null
+
+    if (w.config.series) {
+      w.config.series.forEach((serie) => {
+        if (serie.__apexParsed) {
+          delete serie.__apexParsed
+        }
+      })
+    }
+  }
+
   extractPieDataFromSeries(ser) {
     const values = []
     const labels = []

src/modules/helpers/UpdateHelpers.js

@@ -134,22 +134,8 @@ export default class UpdateHelpers {
         this.ctx.series.getPreviousPaths()
       }
 
-      let existingSeries
-
-      // axis charts
-      if (w.globals.axisCharts) {
-        existingSeries = newSeries.map((s, i) => {
-          return this._extendSeries(s, i)
-        })
-
-        if (existingSeries.length === 0) {
-          existingSeries = [{ data: [] }]
-        }
-        w.config.series = existingSeries
-      } else {
-        // non-axis chart (pie/radialbar)
-        w.config.series = newSeries.slice()
-      }
+      this.ctx.data.resetParsingFlags()
+      this.ctx.data.parseData(newSeries)
 
       if (overwriteInitialSeries) {
         w.globals.initialConfig.series = Utils.clone(w.config.series)
@@ -255,22 +241,6 @@ export default class UpdateHelpers {
     return options
   }
 
-  resetParsingFlags() {
-    const gl = this.w.globals
-
-    // Reset parsing flags
-    gl.dataWasParsed = false
-    gl.originalSeries = null
-
-    if (this.w.config.series) {
-      this.w.config.series.forEach((serie) => {
-        if (serie.__apexParsed) {
-          delete serie.__apexParsed
-        }
-      })
-    }
-  }
-
   /**
    * This function reverts the yaxis and xaxis min/max values to what it was when the chart was defined.
    * This function fixes an important bug where a user might load a new series after zooming in/out of previous series which resulted in wrong min/max