@dhis2/analytics 27.0.0 → 28.0.0

package/build/cjs/components/FileMenu/__tests__/utils.spec.js

@@ -89,92 +89,37 @@ describe('utils', () => {
     });
   });
   describe('preparePayloadForSave', () => {
-    const mockEngine = {
-      query: jest.fn()
-    };
-    beforeEach(() => {
-      jest.clearAllMocks();
-    });
-    it('fetches subscribers and adds them to the visualization', async () => {
-      const visualization = {
-        id: '123',
-        type: 'BAR',
-        name: 'Existing Name',
-        description: 'Existing Description'
-      };
-      mockEngine.query.mockResolvedValue({
-        ao: {
-          subscribers: ['user1', 'user2']
-        }
-      });
-      const result = await (0, _utils.preparePayloadForSave)({
-        visualization,
-        engine: mockEngine
-      });
-      expect(mockEngine.query).toHaveBeenCalledWith({
-        ao: {
-          resource: 'visualizations',
-          id: expect.any(Function),
-          params: {
-            fields: 'subscribers'
-          }
-        }
-      }, {
-        variables: {
-          id: '123'
-        }
-      });
-      expect(result.subscribers).toEqual(['user1', 'user2']);
-    });
-    it('sets the name to the provided name', async () => {
+    it('sets the name to the provided name', () => {
       const visualization = {
         id: '123',
         type: 'MAP',
         name: 'Existing name'
       };
       const name = 'New Name';
-      mockEngine.query.mockResolvedValue({
-        ao: {
-          subscribers: []
-        }
-      });
-      const result = await (0, _utils.preparePayloadForSave)({
+      const result = (0, _utils.preparePayloadForSave)({
         visualization,
-        name,
-        engine: mockEngine
+        name
       });
       expect(result.name).toBe(name);
     });
-    it('sets the name to the existing name if no new name is provided', async () => {
+    it('sets the name to the existing name if no new name is provided', () => {
       const visualization = {
         id: '123',
         type: 'LINE_LIST',
         name: 'Existing Name'
       };
-      mockEngine.query.mockResolvedValue({
-        ao: {
-          subscribers: []
-        }
-      });
-      const result = await (0, _utils.preparePayloadForSave)({
-        visualization,
-        engine: mockEngine
+      const result = (0, _utils.preparePayloadForSave)({
+        visualization
       });
       expect(result.name).toBe('Existing Name');
     });
-    it('sets the name to a default value if no name is provided', async () => {
+    it('sets the name to a default value if no name is provided', () => {
       const visualization = {
         id: '123',
         type: 'BAR'
       };
-      mockEngine.query.mockResolvedValue({
-        ao: {
-          subscribers: []
-        }
-      });
-      const result = await (0, _utils.preparePayloadForSave)({
-        visualization,
-        engine: mockEngine
+      const result = (0, _utils.preparePayloadForSave)({
+        visualization
       });
       const expectedName = `Untitled Bar, ${new Date().toLocaleDateString(undefined, {
         year: 'numeric',
@@ -183,55 +128,37 @@ describe('utils', () => {
       })}`;
       expect(result.name).toBe(expectedName);
     });
-    it('sets the description to the provided description', async () => {
+    it('sets the description to the provided description', () => {
       const visualization = {
         id: '123',
         type: 'YEAR_OVER_YEAR_LINE',
         description: 'Existing Description'
       };
       const description = 'New Description';
-      mockEngine.query.mockResolvedValue({
-        ao: {
-          subscribers: []
-        }
-      });
-      const result = await (0, _utils.preparePayloadForSave)({
+      const result = (0, _utils.preparePayloadForSave)({
         visualization,
-        description,
-        engine: mockEngine
+        description
       });
       expect(result.description).toBe(description);
     });
-    it('keeps the existing description if no new description is provided', async () => {
+    it('keeps the existing description if no new description is provided', () => {
       const visualization = {
         id: '123',
         type: 'COLUMN',
         description: 'Existing Description'
       };
-      mockEngine.query.mockResolvedValue({
-        ao: {
-          subscribers: []
-        }
-      });
-      const result = await (0, _utils.preparePayloadForSave)({
-        visualization,
-        engine: mockEngine
+      const result = (0, _utils.preparePayloadForSave)({
+        visualization
       });
       expect(result.description).toBe('Existing Description');
     });
-    it('sets the description to undefined if no description is provided and none exists', async () => {
+    it('sets the description to undefined if no description is provided and none exists', () => {
       const visualization = {
         id: '123',
         type: 'BAR'
       };
-      mockEngine.query.mockResolvedValue({
-        ao: {
-          subscribers: []
-        }
-      });
-      const result = await (0, _utils.preparePayloadForSave)({
-        visualization,
-        engine: mockEngine
+      const result = (0, _utils.preparePayloadForSave)({
+        visualization
       });
       expect(result.description).toBeUndefined();
     });

package/build/cjs/components/FileMenu/utils.js

@@ -65,38 +65,12 @@ const preparePayloadForSaveAs = _ref => {
   return visualization;
 };
 exports.preparePayloadForSaveAs = preparePayloadForSaveAs;
-const getSubscriberQuery = type => ({
-  ao: {
-    resource: (0, _visTypes.getApiEndpointByVisType)(type),
-    id: _ref2 => {
-      let {
-        id
-      } = _ref2;
-      return id;
-    },
-    params: {
-      fields: 'subscribers'
-    }
-  }
-});
-const apiFetchAOSubscribers = (dataEngine, id, type) => {
-  return dataEngine.query(getSubscriberQuery(type), {
-    variables: {
-      id
-    }
-  });
-};
-const preparePayloadForSave = async _ref3 => {
+const preparePayloadForSave = _ref2 => {
   let {
     visualization,
     name,
-    description,
-    engine
-  } = _ref3;
-  const {
-    ao
-  } = await apiFetchAOSubscribers(engine, visualization.id, visualization.type);
-  visualization.subscribers = ao.subscribers;
+    description
+  } = _ref2;
   visualization.name = name || visualization.name || _d2I18n.default.t('Untitled {{visualizationType}}, {{date}}', {
     visualizationType: (0, _visTypes.getDisplayNameByVisType)(visualization.type),
     date: new Date().toLocaleDateString(undefined, {
@@ -106,6 +80,8 @@ const preparePayloadForSave = async _ref3 => {
     })
   });
   visualization.description = description !== undefined ? description : visualization.description;
+  delete visualization.displayName;
+  delete visualization.displayDescription;
   return visualization;
 };
 exports.preparePayloadForSave = preparePayloadForSave;

package/build/cjs/modules/pivotTable/PivotTableEngine.js

@@ -250,10 +250,10 @@ class PivotTableEngine {
         rawValue = (0, _parseValue.parseValue)(rawValue);
         switch (this.visualization.numberType) {
           case _pivotTableConstants.NUMBER_TYPE_ROW_PERCENTAGE:
-            renderedValue = rawValue / this.percentageTotals[row].value;
+            rawCell.pctValue = renderedValue = rawValue / this.percentageTotals[row].value;
             break;
           case _pivotTableConstants.NUMBER_TYPE_COLUMN_PERCENTAGE:
-            renderedValue = rawValue / this.percentageTotals[column].value;
+            rawCell.pctValue = renderedValue = rawValue / this.percentageTotals[column].value;
             break;
           default:
             break;
@@ -265,6 +265,19 @@ class PivotTableEngine {
       rawCell.rawValue = rawValue;
       rawCell.renderedValue = renderedValue;
     }
+
+    // show the original value in the tooltip
+    if ([_pivotTableConstants.NUMBER_TYPE_COLUMN_PERCENTAGE, _pivotTableConstants.NUMBER_TYPE_ROW_PERCENTAGE].includes(this.visualization.numberType)) {
+      rawCell.titleValue = _d2I18n.default.t('Value: {{value}}', {
+        value: (0, _renderValue.renderValue)(rawCell.rawValue, valueType,
+        // force VALUE for formatting the original value
+        {
+          ...this.visualization,
+          numberType: _pivotTableConstants.NUMBER_TYPE_VALUE
+        }),
+        nsSeparator: '^^'
+      });
+    }
     if ([_pivotTableConstants.CELL_TYPE_TOTAL, _pivotTableConstants.CELL_TYPE_SUBTOTAL].includes(rawCell.cellType) && rawCell.rawValue === _pivotTableConstants.AGGREGATE_TYPE_NA) {
       rawCell.titleValue = _d2I18n.default.t('Not applicable');
     }
@@ -957,7 +970,11 @@ class PivotTableEngine {
       if (!valueB || valueB.empty) {
         return 1 * order;
       }
-      if (valueA.valueType === _valueTypes.VALUE_TYPE_NUMBER && valueB.valueType === _valueTypes.VALUE_TYPE_NUMBER) {
+      if (
+      // for percentage strings, use the pctValue (percentage value) in the sort comparison
+      [_pivotTableConstants.NUMBER_TYPE_ROW_PERCENTAGE, _pivotTableConstants.NUMBER_TYPE_COLUMN_PERCENTAGE].includes(this.visualization.numberType)) {
+        return (valueA.pctValue - valueB.pctValue) * order;
+      } else if (valueA.valueType === _valueTypes.VALUE_TYPE_NUMBER && valueB.valueType === _valueTypes.VALUE_TYPE_NUMBER) {
         return (valueA.rawValue - valueB.rawValue) * order;
       }
       return valueA.renderedValue.localeCompare(valueB.renderedValue) * order;