fix: include program id for TE dimensions

src/modules/layout/axisGetDimensionIds.js

@@ -1,7 +1,7 @@
 import { AXIS } from './axis.js'
 import { dimensionGetId } from './dimensionGetId.js'
 
-export const axisGetDimensionIds = (axis) =>
+export const axisGetDimensionIds = (axis, outputType) =>
     AXIS.isValid(axis)
-        ? axis.map((dimension) => dimensionGetId(dimension))
+        ? axis.map((dimension) => dimensionGetId(dimension, outputType))
         : AXIS.defaultValue

src/modules/layout/dimensionGetId.js

@@ -1,8 +1,14 @@
-import { DIMENSION_PROP_ID, DIMENSION_PROP_PROGRAM_STAGE } from './dimension.js'
+import { formatDimension } from '../../api/analytics/utils.js'
+import {
+    DIMENSION_PROP_ID,
+    DIMENSION_PROP_PROGRAM_STAGE,
+    DIMENSION_PROP_PROGRAM,
+} from './dimension.js'
 
-export const dimensionGetId = (dimension) =>
-    dimension[DIMENSION_PROP_PROGRAM_STAGE.name]?.id
-        ? `${dimension[DIMENSION_PROP_PROGRAM_STAGE.name].id}.${
-              dimension[DIMENSION_PROP_ID.name]
-          }`
-        : dimension[DIMENSION_PROP_ID.name]
+export const dimensionGetId = (dimension, outputType) =>
+    formatDimension({
+        dimension: dimension[DIMENSION_PROP_ID.name],
+        programId: dimension[DIMENSION_PROP_PROGRAM.name]?.id,
+        programStageId: dimension[DIMENSION_PROP_PROGRAM_STAGE.name]?.id,
+        outputType,
+    })

src/modules/layout/layoutFilterDimensions.js

@@ -8,7 +8,10 @@ export const layoutFilterDimensions = (layout, dimensionIds) => {
     DEFAULT_AXIS_IDS.forEach((axisId) => {
         if (AXIS.isValid(filteredLayout[axisId])) {
             filteredLayout[axisId] = filteredLayout[axisId].filter(
-                (dimension) => !idArray.includes(dimensionGetId(dimension))
+                (dimension) =>
+                    !idArray.includes(
+                        dimensionGetId(dimension, layout.outputType)
+                    )
             )
         }
     })

src/modules/layout/layoutGetAxisIdDimensionIdsObject.js

@@ -4,7 +4,7 @@ import { axisGetDimensionIds } from './axisGetDimensionIds.js'
 export const layoutGetAxisIdDimensionIdsObject = (layout) =>
     DEFAULT_AXIS_IDS.reduce((obj, axisId) => {
         if (AXIS.isValid(layout[axisId])) {
-            obj[axisId] = axisGetDimensionIds(layout[axisId])
+            obj[axisId] = axisGetDimensionIds(layout[axisId], layout.outputType)
         }
 
         return obj

src/modules/layout/layoutGetDimensionIdItemIdsObject.js

@@ -4,6 +4,9 @@ import { layoutGetAllDimensions } from './layoutGetAllDimensions.js'
 
 export const layoutGetDimensionIdItemIdsObject = (layout) =>
     layoutGetAllDimensions(layout).reduce((obj, dimension) => {
-        obj[dimensionGetId(dimension)] = dimensionGetItemIds(dimension)
+        obj[dimensionGetId(dimension)] = dimensionGetItemIds(
+            dimension,
+            layout.outputType
+        )
         return obj
     }, {})

src/modules/layout/layoutHasDynamicDimension.js

@@ -7,7 +7,8 @@ export const layoutHasDynamicDimension = (layout) => {
 
     return Boolean(
         layoutGetAllDimensions(layout).find(
-            (dimension) => !fixedIds.includes(dimensionGetId(dimension))
+            (dimension) =>
+                !fixedIds.includes(dimensionGetId(dimension, layout.outputType))
         )
     )
 }