@datarailsshared/dr_renderer 1.2.363 → 1.2.365

package/.circleci/config.yml

@@ -20,7 +20,7 @@ jobs:
           name: Prepare publish version
           command: |
             postfix=""
-            if [[ "${CIRCLE_BRANCH}" != master ]] ; then
+            if [[ "${CIRCLE_BRANCH}" != prod ]] ; then
               postfix="-$CIRCLE_BRANCH"
             fi
 
@@ -70,7 +70,7 @@ workflows:
           filters:
             branches:
               only:
-                - /master/
+                - /prod.*/
                 - /tigers/
                 - /bratans/
                 - /dragons/

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@datarailsshared/dr_renderer",
-  "version": "1.2.363",
+  "version": "1.2.365",
   "description": "DataRails charts and tables renderer",
   "keywords": [
     "datarails",

package/src/dr_pivottable.js

@@ -12,10 +12,6 @@ let initDRPivotTable = function($, window, document) {
       document.ReportHippo.user.organization.settings && document.ReportHippo.user.organization.settings.use_new_ux;
     // const numberOfRows = 500; // change to activate the handsontable when num of rows bigger then this.
 
-    const isFlatKeyInPivotKeys = function(keys, flatKey) {
-        return keys.some(key => key.join(delim) === flatKey);
-    };
-
     DRPivotData = (function(superClass) {
         extend(DRPivotData, superClass);
 
@@ -95,7 +91,6 @@ let initDRPivotTable = function($, window, document) {
 
         DRPivotData.prototype.arrSort = function(attrs) {
             var a, sortersArr;
-            const sortByValueAttrs = this.sortByValueAttrs;
             sortersArr = (function() {
                 var l, len1, results;
                 results = [];
@@ -105,21 +100,11 @@ let initDRPivotTable = function($, window, document) {
                 }
                 return results;
             }).call(this);
-
             return function(a, b) {
                 var comparison, i, sorter;
                 for (i in sortersArr) {
-                    const index = parseInt(i);
-                    sorter = sortersArr[index];
-                    if (sortByValueAttrs.indexOf(attrs[index]) !== -1) {
-
-                        // For case current Field attrs[index] is sorted by value we are concatenating values passed to sorter function
-                        // Concatenation is done from first field in a block (first axis or first series) until current field index.
-                        // Cause for this case sorting will be as list of these concatenated strings (which is prepared in getSortingByValueOrderList)
-                        comparison = sorter(a.slice(0, index + 1).join(','), b.slice(0, index + 1).join(','));
-                    } else {
-                        comparison = sorter(a[index], b[index]);
-                    }
+                    sorter = sortersArr[i];
+                    comparison = sorter(a[i], b[i]);
                     if (comparison !== 0) {
                         return comparison;
                     }
@@ -158,15 +143,7 @@ let initDRPivotTable = function($, window, document) {
             return key;
         };
 
-        DRPivotData.prototype.getInsight = function(colKey, rowKey) {
-            const insightInfo = _.find(this.insights, insight => insight.colKey === colKey && insight.rowKey === rowKey);
-            return _.get(insightInfo, 'insight', null);
-        }
-
         DRPivotData.prototype.processRecord = function(record, useTotalsCalculation) {
-            const insight = record.insight;
-            delete record.insight;
-            
             if (useTotalsCalculation) {
                 if (!this.notFirst) {
                     this.keysLength = _.filter(_.keys(record), key => !_.includes(['data_types', 'formats', 'values_formats'], key)).length - 1;
@@ -203,10 +180,10 @@ let initDRPivotTable = function($, window, document) {
                 let flatColKey = colKey.join(delim);
 
                 if (this.keysLength === rowKey.length + colKey.length) {
-                    if (!this.isKeysSortingDoneOnBackendSide && !isFlatKeyInPivotKeys(this.rowKeys, flatRowKey)) {
+                    if (!this.rowKeys.some(rKey => rKey.join(delim) === flatRowKey)) {
                         this.rowKeys.push(rowKey);
                     }
-                    if (!this.isKeysSortingDoneOnBackendSide && !isFlatKeyInPivotKeys(this.colKeys, flatColKey)) {
+                    if (!this.colKeys.some(cKey => cKey.join(delim) === flatColKey)) {
                         this.colKeys.push(colKey);
                     }
                 }
@@ -230,14 +207,6 @@ let initDRPivotTable = function($, window, document) {
                     this.tree[flatRowKey][flatColKey] = this.aggregator(this, rowKey, colKey);
                     this.tree[flatRowKey][flatColKey].push(record);
                 }
-
-                if (insight && (flatRowKey || flatColKey)) {
-                    this.insights.push({
-                        colKey: flatColKey,
-                        rowKey: flatRowKey,
-                        insight: insight,
-                    });
-                }
                 return;
             }
 
@@ -261,19 +230,6 @@ let initDRPivotTable = function($, window, document) {
                 };
             })(this));
 
-            if (insight) {
-                const flatRowKeyForInsight = rowKey.join(delim);
-                const flatColKeyForInsight = colKey.join(delim);
-
-                if (flatRowKeyForInsight || flatColKeyForInsight) {
-                    this.insights.push({
-                        colKey: flatColKeyForInsight,
-                        rowKey: flatRowKeyForInsight,
-                        insight: insight,
-                    });
-                }
-            }
-
             m = rowKey.length - 1;
             n = colKey.length - 1;
             if (m < 0 || n < 0) {
@@ -515,11 +471,8 @@ let initDRPivotTable = function($, window, document) {
             return resultsArr;
         } else {
             let tooMuch = false;
-            const show_more_than_thousand_rows = _.get(opts, 'chartOptions.table_options.show_more_than_thousand_rows', false);
-            if (pvtData &&
-                (pvtData.rowKeys.length > 1000 || pvtData.colKeys.length > 500) &&
-                (!opts.show_more_function_cols || !opts.show_more_function_cols) &&
-                !show_more_than_thousand_rows) {
+            if (pvtData && (pvtData.rowKeys.length > 1000 || pvtData.colKeys.length > 500) &&
+                (!opts.show_more_function_cols || !opts.show_more_function_cols)) {
                 pvtData.rowKeys = [];
                 pvtData.colKeys = [];
                 tooMuch = true;
@@ -923,7 +876,7 @@ let initDRPivotTable = function($, window, document) {
             return string_val;
         };
 
-        processKeys = function(keysArr, className, dimention, attrs, formattedKeys) {
+        processKeys = function(keysArr, className, dimention, attrs) {
             for (let i = 0; i < keysArr.length; i++) {
                 const additionalField = _.find(additionalFieldsList, {key: keysArr[i][0]})
 
@@ -944,7 +897,7 @@ let initDRPivotTable = function($, window, document) {
                 lastCol = 0;
             }
             rMark = [];
-            th = createElement("th", className, valueNoDashes(encodeHtmlEntities(formattedKeys[0][0]) + getSubtotalInBrackets(keysArr, className, lastCol, lastRow, 0, 0)));
+            th = createElement("th", className, valueNoDashes(encodeHtmlEntities(keysArr[0][0]) + getSubtotalInBrackets(keysArr, className, lastCol, lastRow, 0, 0)));
             key = [];
             key.push(keysArr[0][0]);
             nodePos = 0;
@@ -964,7 +917,7 @@ let initDRPivotTable = function($, window, document) {
             rMark[0] = node;
             c = 1;
             while (c <= lastCol) {
-                th = createElement("th", className, valueNoDashes(encodeHtmlEntities(formattedKeys[0][c]) + getSubtotalInBrackets(keysArr, className, lastCol, lastRow, 0, c)));
+                th = createElement("th", className, valueNoDashes(encodeHtmlEntities(keysArr[0][c]) + getSubtotalInBrackets(keysArr, className, lastCol, lastRow, 0, c)));
                 key = key.slice();
                 key.push(keysArr[0][c]);
                 ++nodePos;
@@ -1030,7 +983,7 @@ let initDRPivotTable = function($, window, document) {
                     }
 
 
-                    th = createElement("th", className, valueNoDashes(encodeHtmlEntities(formattedKeys[r][c]) + getSubtotalInBrackets(keysArr, className, lastCol, lastRow, r, c)));
+                    th = createElement("th", className, valueNoDashes(encodeHtmlEntities(keysArr[r][c]) + getSubtotalInBrackets(keysArr, className, lastCol, lastRow, r, c)));
                     ++nodePos;
                     node = {
                         node: nodePos,
@@ -2454,7 +2407,7 @@ let initDRPivotTable = function($, window, document) {
             return Array.from(wrapper.querySelectorAll(selectString));
         }
 
-        main = function(rowAttrs, rowKeys, colAttrs, colKeys, pivotData) {
+        main = function(rowAttrs, rowKeys, colAttrs, colKeys) {
             var c,rowspan, colHeaderCols, colHeaderHeaders, colHeaders, h, k, l, len, len1, result, rowHeaderHeaders, rowHeaderRows, rowHeaders, tbody, thead, tr;
             rowHeaders = [];
             colHeaders = [];
@@ -2463,12 +2416,10 @@ let initDRPivotTable = function($, window, document) {
             colHeaderHeaders = [];
             colHeaderCols = [];
             if (rowAttrs.length > 0 && rowKeys.length > 0) {
-                const formattedKeys = pivotData.getFormattedRowKeys(rowKeys);
-                rowHeaders = processKeys(rowKeys, "pvtRowLabel", 'rows', rowAttrs, formattedKeys);
+                rowHeaders = processKeys(rowKeys, "pvtRowLabel", 'rows', rowAttrs);
             }
             if (colAttrs.length > 0 && colKeys.length > 0) {
-                const formattedKeys = pivotData.getFormattedColKeys(colKeys);
-                colHeaders = processKeys(colKeys, "pvtColLabel", 'cols', colAttrs, formattedKeys);
+                colHeaders = processKeys(colKeys, "pvtColLabel", 'cols', colAttrs);
             }
             var tableClasses = ['pvtTable'];
             if (opts.chartOptions.table_options.use_new_table_design) {
@@ -2613,7 +2564,7 @@ let initDRPivotTable = function($, window, document) {
 
             return wrapper;
         };
-        return main(rowAttrs, rowKeys, colAttrs, colKeys, pivotData);
+        return main(rowAttrs, rowKeys, colAttrs, colKeys);
     };
     // $.pivotUtilities.subtotal_renderers = SubtotalRenderer;
     $.pivotUtilities.subtotal_renderers = NovixRenderer;