🐛 fix duplicate indicators in indicator-based explorers

owid · Sep 10, 2024 · a3b260f · a3b260f
1 parent 8b53f96
commit a3b260f
Showing 1 changed file with 10 additions and 5 deletions.
diff --git a/explorer/Explorer.tsx b/explorer/Explorer.tsx
@@ -589,13 +589,18 @@ export class Explorer
         // set given variable IDs as dimensions to make Grapher
         // download the data and metadata for these variables
         const dimensions = config.dimensions ?? []
-
-        yVariableIdsList.forEach((yVariableId) => {
-            dimensions.push({
+        const variablesToLoad = yVariableIdsList
+            // Filter out variableIds that are already present in the dimensions array
+            .filter(
+                (yVariableId) =>
+                    !dimensions.some((d) => d.variableId === yVariableId)
+            )
+            .map((yVariableId) => ({
                 variableId: yVariableId,
                 property: DimensionProperty.y,
-            })
-        })
+            }))
+        dimensions.push(...variablesToLoad)
+
         if (xVariableId) {
             const maybeXVariableId = parseIntOrUndefined(xVariableId)
             if (maybeXVariableId !== undefined)