Merge pull request #17247 from apache/fix/tmp-symbol-z

fix(line): set default z/zlevel for temporary symbol to avoid warnings.
apache · Jun 20, 2022 · 26fbb75 · 26fbb75
2 parents 0f9f16c + 6212121
commit 26fbb75
Show file tree

Hide file tree

Showing 4 changed files with 39 additions and 5 deletions.
diff --git a/src/chart/line/LineView.ts b/src/chart/line/LineView.ts
@@ -947,8 +947,8 @@ class LineView extends ChartView {
                 if (this._clipShapeForSymbol && !this._clipShapeForSymbol.contain(x, y)) {
                     return;
                 }
-                const zlevel = seriesModel.get('zlevel');
-                const z = seriesModel.get('z');
+                const zlevel = seriesModel.get('zlevel') || 0;
+                const z = seriesModel.get('z') || 0;
                 symbol = new SymbolClz(data, dataIndex);
                 symbol.x = x;
                 symbol.y = y;

diff --git a/test/line-case.html b/test/line-case.html
diff --git a/test/runTest/actions/__meta__.json b/test/runTest/actions/__meta__.json
diff --git a/test/runTest/actions/line-case.json b/test/runTest/actions/line-case.json