Merge pull request #19642 from apache/fix/pie-endAngle-emptyCircle

fix(pie): fix `endAngle` is not applied on the empty circle

Author: Ovilia

src/chart/pie/PieView.ts

@@ -33,7 +33,7 @@ import { setLabelLineStyle, getLabelLineStatesModels } from '../../label/labelGu
 import { setLabelStyle, getLabelStatesModels } from '../../label/labelStyle';
 import { getSectorCornerRadius } from '../helper/sectorHelper';
 import { saveOldStyle } from '../../animation/basicTransition';
-import { getBasicPieLayout } from './pieLayout';
+import { getBasicPieLayout, getSeriesLayoutData } from './pieLayout';
 
 /**
  * Piece of pie including Sector, Label, LabelLine
@@ -260,8 +260,9 @@ class PieView extends ChartView {
         }
         // when all data are filtered, show lightgray empty circle
         if (data.count() === 0 && seriesModel.get('showEmptyCircle')) {
+            const layoutData = getSeriesLayoutData(seriesModel);
             const sector = new graphic.Sector({
-                shape: getBasicPieLayout(seriesModel, api)
+                shape: extend(getBasicPieLayout(seriesModel, api), layoutData)
             });
             sector.useStyle(seriesModel.getModel('emptyCircleStyle').getItemStyle());
             this._emptyCircleSector = sector;

src/chart/pie/pieLayout.ts

@@ -25,6 +25,7 @@ import ExtensionAPI from '../../core/ExtensionAPI';
 import PieSeriesModel from './PieSeries';
 import { SectorShape } from 'zrender/src/graphic/shape/Sector';
 import { normalizeArcAngles } from 'zrender/src/core/PathProxy';
+import { makeInner } from '../../util/model';
 
 const PI2 = Math.PI * 2;
 const RADIAN = Math.PI / 180;
@@ -127,6 +128,11 @@ export default function pieLayout(
 
         [startAngle, endAngle] = angles;
 
+        const layoutData = getSeriesLayoutData(seriesModel);
+        layoutData.startAngle = startAngle;
+        layoutData.endAngle = endAngle;
+        layoutData.clockwise = clockwise;
+
         const angleRange = Math.abs(endAngle - startAngle);
 
         // In the case some sector angle is smaller than minAngle
@@ -224,8 +230,8 @@ export default function pieLayout(
                             actualEndAngle = actualStartAngle;
                         }
                         else {
-                             actualStartAngle = startAngle + dir * idx * angle + halfPadAngle;
-                             actualEndAngle = startAngle + dir * (idx + 1) * angle - halfPadAngle;
+                            actualStartAngle = startAngle + dir * idx * angle + halfPadAngle;
+                            actualEndAngle = startAngle + dir * (idx + 1) * angle - halfPadAngle;
                         }
 
                         layout.startAngle = actualStartAngle;
@@ -263,3 +269,9 @@ export default function pieLayout(
         }
     });
 }
+
+export const getSeriesLayoutData = makeInner<{
+    startAngle: number
+    endAngle: number
+    clockwise: boolean
+}, PieSeriesModel>();