fix(plugin-chart-echarts): exclude extra metrics from stacked totals

superset-frontend/plugins/plugin-chart-echarts/src/Timeseries/transformProps.ts

@@ -304,18 +304,19 @@ export default function transformProps(
     xAxisLabel = verboseMap[xAxisLabel];
   }
   const isHorizontal = orientation === OrientationType.Horizontal;
+  const extraMetricLabels = extractExtraMetrics(chartProps.rawFormData).map(
+    getMetricLabel,
+  );
   const { totalStackedValues, thresholdValues } = extractDataTotalValues(
     rebasedData,
     {
       stack,
       percentageThreshold,
       xAxisCol: xAxisLabel,
       legendState,
+      extraMetricLabels,
     },
   );
-  const extraMetricLabels = extractExtraMetrics(chartProps.rawFormData).map(
-    getMetricLabel,
-  );
 
   const isMultiSeries = groupBy.length || metrics?.length > 1;
   const xAxisDataType = dataTypes?.[xAxisLabel] ?? dataTypes?.[xAxisOrig];

superset-frontend/plugins/plugin-chart-echarts/src/utils/series.ts

@@ -420,20 +420,33 @@ export function extractDataTotalValues(
     percentageThreshold: number;
     xAxisCol: string;
     legendState?: LegendState;
+    extraMetricLabels?: string[];
   },
 ): {
   totalStackedValues: number[];
   thresholdValues: number[];
 } {
   const totalStackedValues: number[] = [];
   const thresholdValues: number[] = [];
-  const { stack, percentageThreshold, xAxisCol, legendState } = opts;
+  const {
+    stack,
+    percentageThreshold,
+    xAxisCol,
+    legendState,
+    extraMetricLabels = [],
+  } = opts;
+  // Extra metrics, such as a sort metric, are present in the data but are not
+  // rendered as series, so they must not contribute to the stacked total.
+  const extraMetricLabelsSet = new Set(extraMetricLabels);
   if (stack) {
     data.forEach(datum => {
       const values = Object.keys(datum).reduce((prev, curr) => {
         if (curr === xAxisCol) {
           return prev;
         }
+        if (extraMetricLabelsSet.has(curr)) {
+          return prev;
+        }
         if (legendState && !legendState[curr]) {
           return prev;
         }

superset-frontend/plugins/plugin-chart-echarts/test/utils/series.test.ts

@@ -28,6 +28,7 @@ import { GenericDataType } from '@apache-superset/core/common';
 import {
   calculateLowerLogTick,
   dedupSeries,
+  extractDataTotalValues,
   extractGroupbyLabel,
   extractSeries,
   extractShowValueIndexes,
@@ -1681,3 +1682,49 @@ test('getAreaScaledSymbolSize handles degenerate extents and bad values', () =>
     midAreaSize,
   );
 });
+
+describe('extractDataTotalValues', () => {
+  const data: DataRecord[] = [
+    { __timestamp: 1704067200000, A: 32, B: 0, Sort: 2 },
+  ];
+
+  test('excludes extra metrics from the stacked total', () => {
+    const { totalStackedValues } = extractDataTotalValues(data, {
+      stack: true,
+      percentageThreshold: 0,
+      xAxisCol: '__timestamp',
+      extraMetricLabels: ['Sort'],
+    });
+    expect(totalStackedValues).toEqual([32]);
+  });
+
+  test('includes every metric column when there are no extra metrics', () => {
+    const { totalStackedValues } = extractDataTotalValues(data, {
+      stack: true,
+      percentageThreshold: 0,
+      xAxisCol: '__timestamp',
+    });
+    expect(totalStackedValues).toEqual([34]);
+  });
+
+  test('still honors legendState when excluding extra metrics', () => {
+    const { totalStackedValues } = extractDataTotalValues(data, {
+      stack: true,
+      percentageThreshold: 0,
+      xAxisCol: '__timestamp',
+      legendState: { A: false, B: true },
+      extraMetricLabels: ['Sort'],
+    });
+    expect(totalStackedValues).toEqual([0]);
+  });
+
+  test('derives thresholdValues from the total excluding extra metrics', () => {
+    const { thresholdValues } = extractDataTotalValues(data, {
+      stack: true,
+      percentageThreshold: 50,
+      xAxisCol: '__timestamp',
+      extraMetricLabels: ['Sort'],
+    });
+    expect(thresholdValues).toEqual([16]);
+  });
+});