@sis-cc/dotstatsuite-components 14.2.1 → 14.2.4

package/lib/rules/src/preparators/getReportingYearStart.js

@@ -43,8 +43,10 @@ var getReportingYearStart = exports.getReportingYearStart = function getReportin
 
   var _R$match = R.match(/[\d]{2}/g, reportYearStart),
       _R$match2 = (0, _slicedToArray3.default)(_R$match, 2),
-      month = _R$match2[0],
-      day = _R$match2[1];
+      _R$match2$ = _R$match2[0],
+      month = _R$match2$ === undefined ? '01' : _R$match2$,
+      _R$match2$2 = _R$match2[1],
+      day = _R$match2$2 === undefined ? '01' : _R$match2$2;
 
   return { month: month, day: day };
 };

package/lib/rules/src/table/units/getUnitsSeries.js

@@ -16,10 +16,10 @@ function _interopRequireWildcard(obj) { if (obj && obj.__esModule) { return obj;
 */
 
 var getIndex = R.path(['value', 'index']);
-var getUnits = function getUnits(keys, acc, units) {
-  return R.pick(keys, R.mergeWith(function (a, b) {
+var getUnits = function getUnits(acc, units) {
+  return R.mergeWith(function (a, b) {
     return getIndex(a) === getIndex(b) ? a : null;
-  }, acc, units));
+  }, acc, units);
 };
 
 var getUnitsSeries = exports.getUnitsSeries = function getUnitsSeries(observations) {
@@ -31,11 +31,11 @@ var getUnitsSeries = exports.getUnitsSeries = function getUnitsSeries(observatio
   return R.map(function (serie) {
     return R.reduce(function (acc, units) {
       var unitsKeys = R.keys(units);
-      if (R.isEmpty(acc.unitsKeys)) return { units: getUnits(unitsKeys, acc.units, units), unitsKeys: unitsKeys };
+      if (R.isEmpty(acc.unitsKeys)) return { units: units, unitsKeys: unitsKeys };
       var communKeys = R.intersection(acc.unitsKeys, unitsKeys);
       return {
-        units: getUnits(communKeys, acc.units, units),
-        unitsKeys: unitsKeys
+        units: getUnits(R.pick(communKeys, acc.units), R.pick(communKeys, units)),
+        unitsKeys: communKeys
       };
     }, { unitsKeys: [], units: {} }, serie).units;
   })(series);

package/lib/rules2/src/hierarchiseDimensionWithNativeHierarchy.js

@@ -13,12 +13,15 @@ function _interopRequireWildcard(obj) { if (obj && obj.__esModule) { return obj;
 
 var hierarchiseDimensionWithNativeHierarchy = exports.hierarchiseDimensionWithNativeHierarchy = function hierarchiseDimensionWithNativeHierarchy(dimension) {
   var values = R.sortBy(R.prop('__indexPosition'), dimension.values || []);
-  var grouped = R.groupBy(R.propOr('#ROOT', 'parent'), values);
+  var indexed = R.indexBy(R.prop('id'), values);
+  var grouped = R.groupBy(function (val) {
+    return val.parent && R.has(val.parent, indexed) ? val.parent : '#ROOT';
+  }, values);
 
   var getChildren = function getChildren(parents) {
     var childs = R.propOr([], R.last(parents) || '#ROOT', grouped);
     return R.reduce(function (acc, child) {
-      var refined = R.assoc('parents', parents, child);
+      var refined = R.pipe(R.assoc('parents', parents), R.assoc('parent', R.last(parents)))(child);
       var children = getChildren(R.append(child.id, parents));
       return R.concat(acc, R.prepend(refined, children));
     }, [], childs);

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@sis-cc/dotstatsuite-components",
   "description": "Set components based on React.",
-  "version": "14.2.1",
+  "version": "14.2.4",
   "main": "lib/index.js",
   "author": "OECD",
   "license": "MIT",

package/src/rules/src/preparators/getReportingYearStart.js

@@ -11,6 +11,6 @@ export const getReportingYearStart = (sdmxJson) => {
     return id === REPORTING_YEAR_START_DAY || id === REPYEARSTART;
   }, attributes) || {};
   const reportYearStart = R.pathOr('--01-01', ['values', 0, 'name'], reportYearStartAttr);
-  const [month, day] = R.match(/[\d]{2}/g, reportYearStart);
+  const [month = '01', day = '01'] = R.match(/[\d]{2}/g, reportYearStart);
   return ({ month, day });
 };

package/src/rules/src/table/units/getUnitsSeries.js

@@ -5,7 +5,7 @@ import * as R from 'ramda';
 */
 
 const getIndex = R.path(['value', 'index']);
-const getUnits = (keys, acc, units) => R.pick(keys, R.mergeWith((a, b) => getIndex(a) === getIndex(b) ? a : null, acc, units))
+const getUnits = (acc, units) => R.mergeWith((a, b) => getIndex(a) === getIndex(b) ? a : null, acc, units)
 
 export const getUnitsSeries = (observations) => {
   let series = {};
@@ -20,13 +20,12 @@ export const getUnitsSeries = (observations) => {
     serie => R.reduce(
       (acc, units) => {
         const unitsKeys = R.keys(units);
-        if (R.isEmpty(acc.unitsKeys)) return { units: getUnits(unitsKeys, acc.units, units), unitsKeys }
+        if (R.isEmpty(acc.unitsKeys)) return { units, unitsKeys }
         const communKeys = R.intersection(acc.unitsKeys, unitsKeys);
-        return { 
-          units: getUnits(communKeys, acc.units, units),
-          unitsKeys
-        }
-       ;
+        return {
+          units: getUnits(R.pick(communKeys, acc.units), R.pick(communKeys, units)),
+          unitsKeys: communKeys,
+        };
       }, { unitsKeys: [], units: {} }, serie).units
   )(series);
 };

package/src/rules2/src/hierarchiseDimensionWithNativeHierarchy.js

@@ -2,13 +2,17 @@ import * as R from 'ramda';
 
 export const hierarchiseDimensionWithNativeHierarchy = (dimension) => {
   const values = R.sortBy(R.prop('__indexPosition'), dimension.values || []);
-  const grouped = R.groupBy(R.propOr('#ROOT', 'parent'), values);
+  const indexed = R.indexBy(R.prop('id'), values);
+  const grouped = R.groupBy(val => val.parent && R.has(val.parent, indexed) ? val.parent : '#ROOT', values);
 
   const getChildren = (parents) => {
     const childs = R.propOr([], R.last(parents) || '#ROOT', grouped);
     return R.reduce(
       (acc, child) => {
-        const refined = R.assoc('parents', parents, child);
+        const refined = R.pipe(
+          R.assoc('parents', parents),
+          R.assoc('parent', R.last(parents)),
+        )(child);
         const children = getChildren(R.append(child.id, parents));
         return R.concat(acc, R.prepend(refined, children));
       },

package/test/getReportingYearStart.test.js

@@ -5,6 +5,22 @@ describe('getReportingYearStart tests', () => {
   it('no data', () => {
     expect(getReportingYearStart()).to.deep.equal({ month: '01', day: '01' });
   });
+  it('unexpected data in REPYEARSTART attr: "0"', () => {
+    const data = {
+      data: {
+        structure: {
+          attributes: {
+            observation: [
+              { "id": "REPYEARSTART",
+                "values": [{ "name": "0", "names": { "en": "0" } }]
+              }
+            ]
+          }
+        }
+      }
+    };
+    expect(getReportingYearStart(data)).to.deep.equal({ month: '01', day: '01' });
+  });
   it('REPORTING_YEAR_START_DAY', () => {
     const dataReportingYearStartDay = {
       data: {

package/test/hierarchiseDimensionWithNativeHierarchy.test.js

@@ -27,6 +27,31 @@ describe('hierarchiseDimensionWithNativeHierarchy tests', () => {
       ]
     };
 
+    expect(hierarchiseDimensionWithNativeHierarchy(dimension)).to.deep.equal(expected);
+  });
+  it ('missing parent test', () => {
+    const dimension = {
+      id: 'test',
+      values: [
+        { id: 'FRA', parent: 'EA', __indexPosition: 101 },
+        { id: 'GER', parent: 'EA', __indexPosition: 100 },
+        { id: 'EA', parent: 'W', __indexPosition: 30 },
+        { id: 'A', parent: 'W', __indexPosition: 20 },
+        { id: 'CHI', parent: undefined, __indexPosition: 0 }
+      ]
+    };
+    
+    const expected = {
+      id: 'test',
+      values: [
+        { id: 'CHI', parent: undefined, parents: [], __indexPosition: 0 },
+        { id: 'A', parent: undefined, parents: [], __indexPosition: 20 },
+        { id: 'EA', parent: undefined, parents: [], __indexPosition: 30 },
+        { id: 'GER', parent: 'EA', parents: ['EA'], __indexPosition: 100 },
+        { id: 'FRA', parent: 'EA', parents: ['EA'], __indexPosition: 101 }
+      ]
+    };
+
     expect(hierarchiseDimensionWithNativeHierarchy(dimension)).to.deep.equal(expected);
   });
 });