@datarailsshared/dr_renderer 1.2.301-dragons → 1.2.302-dragons

package/package.json

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

package/src/highcharts_renderer.js

@@ -4397,25 +4397,42 @@ let getHighchartsRenderer = function ($, document, Highcharts, default_colors, h
         return ret_str;
     };
 
-    highchartsRenderer.getSortingByValueOrderList = function (pivotData, sortingOptions, keysArray, attrs, fieldIndex) {
+    highchartsRenderer.getSortingByValueOrderList = function (pivotData, sortingOptions, keysArray, attrs, fieldIndex, widget) {
         let values_names_arr = [];
         lodash.forEach(keysArray, function (val) {
             const firstArray = [];
             const secondArray = val.slice(0, fieldIndex + 1);
-            let getAggregatorParams = [firstArray, secondArray];
 
-            if (lodash.includes(pivotData.rowAttrs, attrs[fieldIndex])) {
-                getAggregatorParams = lodash.reverse(getAggregatorParams);
-            }
+            let valueForComparison;
+            if (sortingOptions.sort_by === 'variance') {
+                const varianceConfig = widget.options.chartOptions.delta_column;
+                const data = pivotData.input;
+                const varianceField = varianceConfig.field === 'category' ? widget.cols[0] : widget.rows[0];
+                const varianceRowsForCurrentKey = lodash.filter(data, row => 
+                    row[varianceField.name] === varianceConfig.name 
+                        && lodash.every(secondArray, (item, index) => row[attrs[index]] === item)
+                );
+
+                valueForComparison = lodash.reduce(varianceRowsForCurrentKey, (a, d) => a + d[widget.vals[0].name], 0);
+            } else {
+                let getAggregatorParams = [firstArray, secondArray];
+
+                if (lodash.includes(pivotData.rowAttrs, attrs[fieldIndex])) {
+                    getAggregatorParams = lodash.reverse(getAggregatorParams);
+                }
+    
+                let aggregator_subtotal = pivotData.getAggregator(...getAggregatorParams);
 
-            let aggregator_subtotal = pivotData.getAggregator(...getAggregatorParams);
+                if (aggregator_subtotal) {
+                    valueForComparison = aggregator_subtotal.value();
+                }
+            }
 
-            if (aggregator_subtotal) {
-                let value_subtotal = aggregator_subtotal.value();
-                if (sortingOptions && sortingOptions.is_absolute && !isNaN(parseFloat(value_subtotal))) {
-                    value_subtotal = Math.abs(value_subtotal);
+            if (typeof valueForComparison !== 'undefined') {
+                if (sortingOptions && sortingOptions.is_absolute && !isNaN(parseFloat(valueForComparison))) {
+                    valueForComparison = Math.abs(valueForComparison);
                 }
-                values_names_arr.push({name: secondArray.join(','), value: value_subtotal});
+                values_names_arr.push({name: secondArray.join(','), value: valueForComparison});
             }
         });
 
@@ -4447,7 +4464,8 @@ let getHighchartsRenderer = function ($, document, Highcharts, default_colors, h
                     sortingOptions.sorting,
                     pivotData[isColumnSort ? 'colKeys' : 'rowKeys'],
                     pivotData[isColumnSort ? 'colAttrs' : 'rowAttrs'],
-                    fieldIndex
+                    fieldIndex,
+                    widget
                 );
                 return $.pivotUtilities.sortAs(orderedNamesList);
             } else {
@@ -4667,7 +4685,7 @@ let getHighchartsRenderer = function ($, document, Highcharts, default_colors, h
 
                 const sortByValueSettings = lodash.filter(
                     lodash.get(widget, 'options.sortingFields', []),
-                    sortingField => lodash.get(sortingField, 'sorting.sort_by') === 'field_values'
+                    sortingField => lodash.includes(['field_values', 'variance'], lodash.get(sortingField, 'sorting.sort_by'))
                 );
 
                 if (sortByValueSettings.length) {
@@ -7018,18 +7036,6 @@ let getHighchartsRenderer = function ($, document, Highcharts, default_colors, h
                 value_name: 'is_percentage',
                 default_value: false,
                 hidden: true,
-            }, {
-                element_type: 'checkbox',
-                element_label: 'Sort by variance',
-                value_name: 'sort_by_variance',
-                default_value: false,
-                hidden: true
-            }, {
-                element_type: 'checkbox',
-                element_label: 'Sort by absolute variance',
-                value_name: 'sort_by_absolute_variance',
-                default_value: false,
-                hidden: true
             }]
         },
         'delta_column_for_drill_down': {
@@ -7096,18 +7102,6 @@ let getHighchartsRenderer = function ($, document, Highcharts, default_colors, h
                 value_name: 'is_percentage',
                 default_value: false,
                 hidden: true,
-            }, {
-                element_type: 'checkbox',
-                element_label: 'Sort by variance',
-                value_name: 'sort_by_variance',
-                default_value: false,
-                hidden: true
-            }, {
-                element_type: 'checkbox',
-                element_label: 'Sort by absolute variance',
-                value_name: 'sort_by_absolute_variance',
-                default_value: false,
-                hidden: true
             }, {
                 element_type: 'checkbox',
                 element_label: 'Filter zero values',
@@ -8745,61 +8739,9 @@ let getHighchartsRenderer = function ($, document, Highcharts, default_colors, h
         }
 
         /****** END *******/
-
-        if (lodash.has(widget, "options.chartOptions.delta_column") &&
-            widget.options.chartOptions.delta_column.field != '' &&
-            (widget.options.chartOptions.delta_column.sort_by_variance ||
-                widget.options.chartOptions.delta_column.sort_by_absolute_variance) &&
-            widget.rows.length > 0 &&
-            widget.cols.length > 0 &&
-            widget.vals.length
-        ) {
-            let variance_config = widget.options.chartOptions.delta_column;
-            let val_field = widget.vals[0];
-            let field_for_sorting = null;
-            let field_with_variant = null;
-            if (variance_config.field == "series") {
-                field_for_sorting = widget.cols[0];
-                field_with_variant = widget.rows[0];
-            } else if (variance_config.field == "category") {
-                field_for_sorting = widget.rows[0];
-                field_with_variant = widget.cols[0];
-            }
-
-            let data_sorted = lodash.filter(data, function (data_row) {
-                return data_row[field_with_variant.name] == variance_config.name && data_row[field_for_sorting.name] != undefined;
-            });
-
-            const sorting_variance = widget.options.total_value_options && widget.options.total_value_options.sorting_variance === '' ? 'asc' : 'desc';
-
-            if (widget.options.chartOptions.delta_column.sort_by_absolute_variance) {
-                data_sorted = lodash.sortBy(data_sorted, function (o) {
-                    if (sorting_variance === 'desc') {
-                        return Math.abs(o[val_field.name]) * -1;
-                    }
-
-                    return Math.abs(o[val_field.name]) * 1;
-                });
-            } else {
-                data_sorted = lodash.orderBy(data_sorted, [val_field.name], [sorting_variance]);
-            }
-
-            let values_for_sort = lodash.map(data_sorted, function (data_row) {
-                return data_row[field_for_sorting.name];
-            });
-
-            values_for_sort = lodash.uniq(values_for_sort);
-
-            if (values_for_sort.length > 0) {
-                let field = lodash.find(datesFields, {name: field_for_sorting.name});
-                if (field) {
-                    field.values = values_for_sort;
-                    field.sorting = null;
-                } else {
-                    datesFields.push({name: field_for_sorting.name, values: values_for_sort});
-                }
-            }
-        } else if (widget.options && widget.options.sortingValues) {
+        
+        // TODO: Remove. sortingValues looks like lagacy which is not in use neither in webclient nor in renderer
+        if (widget.options && widget.options.sortingValues) {
             var field = lodash.find(datesFields, {name: widget.options.sortingValues.field});
             if (field) {
                 field.values = widget.options.sortingValues.values;