@datarailsshared/dr_renderer 1.2.363 → 1.2.365

package/src/novix_renderer.js

@@ -376,14 +376,13 @@ let initNovixRenderer = function($, window, document, Handsontable){
                         //data columns
                         if (pvtData.colKeys.length > 0) {
 
-                            const formattedColKeys = pvtData.getFormattedColKeys(colKeys);
                             var aux = rowData.length;
                             var startCol = aux;
-                            var prior = formattedColKeys[0][nn];
+                            var prior = colKeys[0][nn];
                             var toMerge = 0;
-                            for (var cc = 0; cc < formattedColKeys.length; cc++) {
+                            for (var cc = 0; cc < colKeys.length; cc++) {
 
-                                var textHeader = formattedColKeys[cc][nn];
+                                var textHeader = colKeys[cc][nn];
 
                                 if (nn < pvtData.colAttrs.length - 1 && prior != textHeader) {
                                     rowData.push(textHeader);
@@ -457,9 +456,8 @@ let initNovixRenderer = function($, window, document, Handsontable){
                 // DATA
 
                 var mergeDic = {};
-                const formattedRowKeys = pvtData.getFormattedRowKeys(rowKeys);
 
-                for (var rr = 0; rr < formattedRowKeys.length; rr++) {
+                for (var rr = 0; rr < rowKeys.length; rr++) {
 
                     var rowData = [];
 
@@ -468,29 +466,29 @@ let initNovixRenderer = function($, window, document, Handsontable){
                             mergeDic[cc] = {
                                 toMerge: 1,
                                 row: 0,
-                                priorText: formattedRowKeys[rr][cc]
+                                priorText: rowKeys[rr][cc]
                             };
                         }
                     }
 
                     //column for row atts
                     for (var cc = 0; cc < pvtData.rowAttrs.length; cc++) {
-                        rowData.push(formattedRowKeys[rr][cc]);
-                        if (formattedRowKeys[rr][cc] === subtotal) {
+                        rowData.push(rowKeys[rr][cc]);
+                        if (rowKeys[rr][cc] === subtotal) {
                             addMergeCell(rr + pvtData.colAttrs.length + 1, cc, 1, dataSource.fixedColumns - cc, true, true);
-                        } else if (cc === formattedRowKeys[rr].length - 1 && hasColumnAttr) {
+                        } else if (cc === rowKeys[rr].length - 1 && hasColumnAttr) {
                             addMergeCell(rr + pvtData.colAttrs.length + 1, cc, 1, 2, true, true);
                         }
                         // merge cell (last column)
                         if (opts.chartOptions.table_options.show_column_total && rr != 0 && mergeDic[cc]) { // check for merge dic
-                            if (formattedRowKeys[rr][cc] == mergeDic[cc].priorText) {
+                            if (rowKeys[rr][cc] == mergeDic[cc].priorText) {
                                 mergeDic[cc].toMerge++;
-                            } else if (cc < formattedRowKeys[rr].length - 1) {
+                            } else if (cc < rowKeys[rr].length - 1) {
                                 //create merge cell for this column
                                 addMergeCell(mergeDic[cc].row + pvtData.colAttrs.length + 1, cc, mergeDic[cc].toMerge, 1);
                                 mergeDic[cc].row = rr;
                                 mergeDic[cc].toMerge = 1;
-                                mergeDic[cc].priorText = formattedRowKeys[rr][cc];
+                                mergeDic[cc].priorText = rowKeys[rr][cc];
                             }
                         }
                     }

package/src/pivottable.js

@@ -1,5 +1,3 @@
-const helpers = require('./dr-renderer-helpers');
-
 // from pivottable@2.23.0
 let initPivotTable = function($, window, document) {
     var indexOf = [].indexOf || function(item) { for (var i = 0, l = this.length; i < l; i++) { if (i in this && this[i] === item) return i; } return -1; },
@@ -579,7 +577,6 @@ let initPivotTable = function($, window, document) {
         l_mapping = {};
         for (i in order) {
             x = order[i];
-            if (order[i] === null) x = '[null]';
             mapping[x] = i;
             if (typeof x === "string") {
                 l_mapping[x.toLowerCase()] = i;
@@ -687,30 +684,13 @@ let initPivotTable = function($, window, document) {
                 return true;
             });
             this.tree = {};
-            this.insights = [];
-
-            this.isKeysSortingDoneOnBackendSide = opts.keysObject && typeof opts.keysObject === 'object' && helpers.backendSortingKeysAreNotEmpty(opts.keysObject);
-            if (this.isKeysSortingDoneOnBackendSide) {
-                this.rowKeys = opts.keysObject.row_keys;
-                this.colKeys = opts.keysObject.col_keys;
-                // TODO: add also for breakdown sort object when BE story is ready.
-            } else {
-                this.rowKeys = [];
-                this.colKeys = [];
-            }
-
+            this.rowKeys = [];
+            this.colKeys = [];
             this.rowTotals = {};
             this.colTotals = {};
             this.allTotal = this.aggregator(this, [], []);
             this.sorted = false;
             this.dateValuesDictionary = opts.dateValuesDictionary;
-            this.sortByValueAttrs = opts.sortByValueAttrs || [];
-            this.colFormats = opts.colFormats || [];
-            this.rowFormats = opts.rowFormats || [];
-            this.isFormattingAxisLabels = opts.rendererOptions && opts.rendererOptions.isFormattingAxisLabels;
-            this.getFormattedColKeys = (keys) => opts.getFormattedColKeys(this, keys);
-            this.getFormattedRowKeys = (keys) => opts.getFormattedRowKeys(this, keys);
-
             PivotData.forEachRecord(this.input, this.derivedAttributes, (function(_this) {
                 return function(record) {
                     if (_this.filter(record)) {
@@ -870,16 +850,12 @@ let initPivotTable = function($, window, document) {
         };
 
         PivotData.prototype.getColKeys = function() {
-            if (!this.isKeysSortingDoneOnBackendSide) {
-                this.sortKeys();
-            }
+            this.sortKeys();
             return this.colKeys;
         };
 
         PivotData.prototype.getRowKeys = function() {
-            if (!this.isKeysSortingDoneOnBackendSide) {
-                this.sortKeys();
-            }
+            this.sortKeys();
             return this.rowKeys;
         };