fix(model): guard against null data in getDataParams and event handler

src/chart/chord/ChordSeries.ts

@@ -273,6 +273,9 @@ class ChordSeriesModel extends SeriesModel<ChordSeriesOption> {
 
     getDataParams(dataIndex: number, dataType: 'node' | 'edge') {
         const params = super.getDataParams(dataIndex, dataType);
+        if (!this.getData()) {
+            return params;
+        }
         if (dataType === 'node') {
             const nodeData = this.getData();
             const node = this.getGraph().getNodeByIndex(dataIndex);

src/chart/funnel/FunnelSeries.ts

@@ -140,6 +140,9 @@ class FunnelSeriesModel extends SeriesModel<FunnelSeriesOption> {
     // Overwrite
     getDataParams(dataIndex: number): FunnelCallbackDataParams {
         const data = this.getData();
+        if (!data) {
+            return {} as FunnelCallbackDataParams;
+        }
         const params = super.getDataParams(dataIndex) as FunnelCallbackDataParams;
         const valueDim = data.mapDimension('value');
         const sum = data.getSum(valueDim);

src/chart/pie/PieSeries.ts

@@ -186,6 +186,9 @@ class PieSeriesModel extends SeriesModel<PieSeriesOption> {
      */
     getDataParams(dataIndex: number): PieCallbackDataParams {
         const data = this.getData();
+        if (!data) {
+            return {} as PieCallbackDataParams;
+        }
         // update seats when data is changed
         const dataInner = innerData(data);
         let seats = dataInner.seats;

src/chart/sunburst/SunburstSeries.ts

@@ -209,8 +209,12 @@ class SunburstSeriesModel extends SeriesModel<SunburstSeriesOption> {
      */
     getDataParams(dataIndex: number) {
         const params = super.getDataParams.apply(this, arguments as any) as SunburstDataParams;
+        const data = this.getData();
+        if (!data) {
+            return params;
+        }
 
-        const node = this.getData().tree.getNodeByDataIndex(dataIndex);
+        const node = data.tree.getNodeByDataIndex(dataIndex);
         params.treePathInfo = wrapTreePathInfo<SunburstSeriesNodeItemOption['value']>(node, this);
 
         return params;

src/chart/tree/TreeSeries.ts

@@ -242,8 +242,12 @@ class TreeSeriesModel extends SeriesModel<TreeSeriesOption> {
     // Add tree path to tooltip param
     getDataParams(dataIndex: number) {
         const params = super.getDataParams.apply(this, arguments as any) as TreeSeriesCallbackDataParams;
+        const data = this.getData();
+        if (!data) {
+            return params;
+        }
 
-        const node = this.getData().tree.getNodeByDataIndex(dataIndex);
+        const node = data.tree.getNodeByDataIndex(dataIndex);
         params.treeAncestors = wrapTreePathInfo(node, this);
         params.collapsed = !node.isExpand;
 

src/chart/treemap/TreemapSeries.ts

@@ -446,8 +446,12 @@ class TreemapSeriesModel extends SeriesModel<TreemapSeriesOption> {
      */
     getDataParams(dataIndex: number) {
         const params = super.getDataParams.apply(this, arguments as any) as TreemapSeriesCallbackDataParams;
+        const data = this.getData();
+        if (!data) {
+            return params;
+        }
 
-        const node = this.getData().tree.getNodeByDataIndex(dataIndex);
+        const node = data.tree.getNodeByDataIndex(dataIndex);
         params.treeAncestors = wrapTreePathInfo(node, this);
         // compatitable the previous code.
         params.treePathInfo = params.treeAncestors;

src/core/echarts.ts

@@ -1208,9 +1208,14 @@ class ECharts extends Eventful<ECEventDefinition> {
                         const ecData = getECData(parent);
                         if (ecData && ecData.dataIndex != null) {
                             const dataModel = ecData.dataModel || ecModel.getSeriesByIndex(ecData.seriesIndex);
-                            params = (
-                                dataModel && dataModel.getDataParams(ecData.dataIndex, ecData.dataType, el) || {}
-                            ) as ECElementEvent;
+                            try {
+                                params = (
+                                    dataModel && dataModel.getDataParams(ecData.dataIndex, ecData.dataType, el) || {}
+                                ) as ECElementEvent;
+                            }
+                            catch (e) {
+                                params = {} as ECElementEvent;
+                            }
                             return true;
                         }
                         // If element has custom eventData of components

src/model/mixin/dataFormat.ts

@@ -61,6 +61,9 @@ export class DataFormatMixin {
     ): CallbackDataParams {
 
         const data = this.getData(dataType);
+        if (!data) {
+            return {} as CallbackDataParams;
+        }
         const rawValue = this.getRawValue(dataIndex, dataType);
         const rawDataIndex = data.getRawIndex(dataIndex);
         const name = data.getName(dataIndex);