@datarailsshared/dr_renderer 1.5.100 → 1.5.103

package/package.json

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

package/src/dr_pivottable.js

@@ -10,8 +10,6 @@ let initDRPivotTable = function($, window, document) {
     const newTableColors = ['rgb(127, 196, 255)', 'rgb(200, 243,243)', 'rgb(247, 161, 173)', 'rgb(255, 237, 178)', 'rgb(221, 239, 255)',
         'rgb(171, 216, 255)', 'rgb(174, 231, 220)', 'rgb(227, 255, 236)', 'rgb(162, 215, 227)', 'rgb(223, 239, 236)'];
 
-    const useNewUx = document.ReportHippo && document.ReportHippo && document.ReportHippo.user &&
-      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.
 
     DRPivotData = (function(superClass) {
@@ -437,7 +435,7 @@ let initDRPivotTable = function($, window, document) {
                 colors = newTableColors;
             }
 
-            var isNewDesign = useNewUx && opts.chartOptions.table_options.use_new_table_design;
+            var isNewDesign = opts.chartOptions && opts.chartOptions.table_options && opts.chartOptions.table_options.use_new_table_design;
             if (value) {
                 var offset = parseInt(value);
                 if (offset) {
@@ -500,12 +498,12 @@ let initDRPivotTable = function($, window, document) {
         };
 
         createTotalValsBolder = function(element){
-            if(opts.chartOptions.table_options.colorize_headers == true && (!useNewUx || !opts.chartOptions.table_options.use_new_table_design)){
+            if(opts.chartOptions.table_options.colorize_headers == true && (!opts.chartOptions.table_options.use_new_table_design)){
                 element.style.setProperty("border-bottom", "3px double #000000", "important");
             }
         };
         createGrandTotalBolder = function(element){
-            if(opts.chartOptions.table_options.colorize_headers == true && (!useNewUx || !opts.chartOptions.table_options.use_new_table_design)){
+            if(opts.chartOptions.table_options.colorize_headers == true && (!opts.chartOptions.table_options.use_new_table_design)){
                 element.style.setProperty("border-top", "3px solid #000000", "important");
             }
         };

package/src/highcharts_renderer.js

@@ -173,7 +173,7 @@ const FEATURES = helpers.FEATURES;
 const TICKS_COUNT = 5;
 
 let getHighchartsRenderer = function ($, document, Highcharts, default_colors, highchartsRenderer,
-                                      DataFormatter, lodash, moment_lib, isNewAngular) {
+                                      DataFormatter, lodash, moment_lib, isNewAngular, envOptions) {
 
     /**  Highcharts initial configuration section **/
     if(!lodash){
@@ -214,6 +214,8 @@ let getHighchartsRenderer = function ($, document, Highcharts, default_colors, h
 
     highchartsRenderer.enabledNewWidgetValueFormatting = false;
     let disableAnimation = false;
+    const environment = envOptions || {};
+    highchartsRenderer.environment = environment;
 
     const _handleComputationalError = (err, onlyOptions) => {
         if (err instanceof BaseRendererError) {
@@ -244,13 +246,14 @@ let getHighchartsRenderer = function ($, document, Highcharts, default_colors, h
     }
 
     highchartsRenderer.hasFeature = function(featureFlagKey) {
-        return lodash.includes(lodash.get(document, 'ReportHippo.user.features', []), featureFlagKey);
+        return lodash.includes(highchartsRenderer.features || [], featureFlagKey);
     }
-
-    if (!!lodash.get(document, 'ReportHippo.user')) {
-        disableAnimation = document.ReportHippo.user.organization && document.ReportHippo.user.organization.settings && document.ReportHippo.user.organization.settings.disable_animation
+    highchartsRenderer.setFeatures = function(features) {
+        highchartsRenderer.features = features || [];
         highchartsRenderer.enabledNewWidgetValueFormatting = highchartsRenderer.hasFeature(FEATURES.ENABLE_NEW_WIDGET_VALUE_FORMATTING);
     }
+    highchartsRenderer.setFeatures(lodash.get(environment, 'features', []));
+    disableAnimation = !!lodash.get(environment, 'disable_animation', false);
 
     // fix issue of use tootip.stickOnContact with tooltip.outside , source: https://github.com/highcharts/highcharts/pull/15960
     // TODO: remove it after it is fixed in highcharts library new version
@@ -964,7 +967,7 @@ let getHighchartsRenderer = function ($, document, Highcharts, default_colors, h
                 tempAr[temp_len - 1] = highchartsRenderer.getFieldName(colAttrs[j]);
             }
 
-            const formattedColKeys = highchartsRenderer.getFormattedColKeys(pivotData, null);
+            const formattedColKeys = highchartsRenderer.getFormattedColKeys(pivotData, null, true);
             tempAr = tempAr.concat(lodash.map(formattedColKeys, function (row) {
                 return row[j];
             }));
@@ -9440,11 +9443,11 @@ let getHighchartsRenderer = function ($, document, Highcharts, default_colors, h
     }
 
     highchartsRenderer.getFiscalYearMonthModifier = function() {
-        return lodash.get(document, 'ReportHippo.user.organization.fiscal_year_starts_from', 1) - 1;
+        return lodash.get(highchartsRenderer, 'environment.fiscal_year_starts_from', 1) - 1;
     }
 
     highchartsRenderer.isFiscalYearBackEnabled = function() {
-        return lodash.get(document, 'ReportHippo.user.organization.fiscal_year_back');
+        return lodash.get(highchartsRenderer, 'environment.fiscal_year_back');
     }
 
     highchartsRenderer.getTimeframeByFormat = function(format) {
@@ -9504,7 +9507,7 @@ let getHighchartsRenderer = function ($, document, Highcharts, default_colors, h
     }
 
     // Method for getting formatted kyes for Axis (cols, rows)
-    highchartsRenderer.getFormattedKey = function (initialKey, pivotData, type) {
+    highchartsRenderer.getFormattedKey = function (initialKey, pivotData, type, forceFormatDates) {
         const isFlatKey = lodash.isString(initialKey) || lodash.isNumber(initialKey);
         if (isFlatKey || lodash.isArray(initialKey)) {
             let values = isFlatKey ? initialKey.toString().split(highchartsRenderer.delimer) : lodash.cloneDeep(initialKey);
@@ -9513,7 +9516,7 @@ let getHighchartsRenderer = function ($, document, Highcharts, default_colors, h
                     const formatInfo = pivotData[type][key] || {};
                     const valueToFloat = parseFloat(value);
                     const isDate = formatInfo.type === 'Date' && (moment_lib(valueToFloat).isValid() || moment_lib(value, formatInfo.format).isValid());
-                    const isDateFormatting = isDate && highchartsRenderer.isFormattingDatesAsOtherAxisTypes();
+                    const isDateFormatting = isDate && (forceFormatDates || highchartsRenderer.isFormattingDatesAsOtherAxisTypes());
                     const isNumberFormatting = !isDate && formatInfo.format && highchartsRenderer.isFormattingNumberAxis(pivotData);
 
                     if (isNumberFormatting || isDateFormatting) {
@@ -9541,13 +9544,13 @@ let getHighchartsRenderer = function ($, document, Highcharts, default_colors, h
         return highchartsRenderer.getFormattedKey(initialValue, pivotData, 'rowFormats')
     }
 
-    highchartsRenderer.getFormattedKeys = function(pivotData, isCols, keys) {
+    highchartsRenderer.getFormattedKeys = function(pivotData, isCols, keys, forceFormatDates) {
         keys = keys || (isCols ? pivotData.getColKeys() : pivotData.getRowKeys());
-        return lodash.map(keys, key => highchartsRenderer.getFormattedKey(key, pivotData, isCols ? 'colFormats' : 'rowFormats'));
+        return lodash.map(keys, key => highchartsRenderer.getFormattedKey(key, pivotData, isCols ? 'colFormats' : 'rowFormats', forceFormatDates));
     }
 
-    highchartsRenderer.getFormattedColKeys = function(pivotData, keys) {
-        return highchartsRenderer.getFormattedKeys(pivotData, true, keys);
+    highchartsRenderer.getFormattedColKeys = function(pivotData, keys, forceFormatDates) {
+        return highchartsRenderer.getFormattedKeys(pivotData, true, keys, forceFormatDates);
     }
 
     highchartsRenderer.getFormattedRowKeys = function(pivotData, keys) {

package/src/index.d.ts

@@ -31,6 +31,13 @@ declare namespace DrRenderFactory {
         isNewAngular: boolean;
     }
 
+    interface EnvironmentOptions {
+        features?: string[];
+        disable_animation?: boolean;
+        fiscal_year_starts_from?: number;
+        fiscal_year_back?: boolean;
+    }
+
     interface Factory {
         init($: any, window: Window, document: Document, Handsontable?: any): void;
         getInitHighchartsRenderer(
@@ -41,12 +48,13 @@ declare namespace DrRenderFactory {
             highchartsRenderer: any,
             lodash: any,
             moment_lib: any,
-            isNewAngular: boolean
+            isNewAngular: boolean,
+            envOptions?: EnvironmentOptions
         ): any;
         getInitPublishedItemsRenderer(publishedItemsRenderer: any, bind?: any): any;
 
         DataFormatter: typeof DataFormatterType;
-        
+
         // Error types and classes
         RendererErrorCodes: typeof ErrorTypes.RendererErrorCodes;
         BaseRendererError: typeof ErrorTypes.BaseRendererError;

package/src/index.js

@@ -17,8 +17,8 @@ dr_render_factory.init = function($, window, document, Handsontable){
     }
 }
 
-dr_render_factory.getInitHighchartsRenderer = function($, document, Highcharts, default_colors, highchartsRenderer, lodash, moment_lib, isNewAngular){
-    return getHighchartsRenderer($, document, Highcharts, default_colors, highchartsRenderer, DataFormatter, lodash, moment_lib, isNewAngular);
+dr_render_factory.getInitHighchartsRenderer = function($, document, Highcharts, default_colors, highchartsRenderer, lodash, moment_lib, isNewAngular, envOptions){
+    return getHighchartsRenderer($, document, Highcharts, default_colors, highchartsRenderer, DataFormatter, lodash, moment_lib, isNewAngular, envOptions);
 }
 
 dr_render_factory.getInitPublishedItemsRenderer = function (publishedItemsRenderer, bind = {}) {

package/tests/highcharts_renderer.test.js

@@ -1624,7 +1624,7 @@ describe('highcharts_renderer', () => {
 
         describe('dynamic system fields', () => {
             it('should count Scenario & Budget Cycle as system field', () => {
-                lodash.set(document, 'ReportHippo.user.features', ['multiple_dimension_tags', 'use_new_scenario_tag']);
+                highchartsRenderer.setFeatures(['multiple_dimension_tags', 'use_new_scenario_tag']);
 
                 const fieldWithScenario = { name: 'Scenario' };
                 expect(highchartsRenderer.isSystemField(fieldWithScenario)).toBe(true);
@@ -5085,7 +5085,7 @@ describe('highcharts_renderer', () => {
             });
 
             it('should return formatted multivalue colKey as array (format_dates_as_other_axis_types is ON)', () => {
-                lodash.set(document, 'ReportHippo.user.features', ['format_dates_as_other_axis_types']);
+                highchartsRenderer.setFeatures(['format_dates_as_other_axis_types']);
                 const initialKey = [1687277052, 1687277052];
                 const pivotData = {
                     colFormats: [{ type: 'Date', format: 'MM/DD/YYYY' }, { type: 'Date', format: 'MMM-yy' }],
@@ -5097,7 +5097,7 @@ describe('highcharts_renderer', () => {
             });
 
             it('should return formatted multivalue rowKey as array (format_dates_as_other_axis_types is ON)', () => {
-                lodash.set(document, 'ReportHippo.user.features', ['format_dates_as_other_axis_types']);
+                highchartsRenderer.setFeatures(['format_dates_as_other_axis_types']);
                 const initialKey = [1687277052, 1687277052];
                 const pivotData = {
                     rowFormats: [{ type: 'Date', format: 'MM/DD/YYYY' }, { type: 'Date', format: 'MMM-yy' }],
@@ -5109,7 +5109,7 @@ describe('highcharts_renderer', () => {
             });
 
             it('should return NOT formatted multivalue colKey as array (format_dates_as_other_axis_types is OFF)', () => {
-                lodash.set(document, 'ReportHippo.user.features', []);
+                highchartsRenderer.setFeatures([]);
                 const initialKeyAlreadyFormatted = ['10/10/2020', '10/10/2021'];
                 const pivotData = {
                     colFormats: [{ type: 'Date', format: 'MM/DD/YYYY' }, { type: 'Date', format: 'MMM-yy' }],
@@ -5121,7 +5121,7 @@ describe('highcharts_renderer', () => {
             });
 
             it('should return NOT formatted multivalue rowKey as array (format_dates_as_other_axis_types is OFF)', () => {
-                lodash.set(document, 'ReportHippo.user.features', []);
+                highchartsRenderer.setFeatures([]);
                 const initialKeyAlreadyFormatted = ['10/10/2020', '10/10/2021'];
                 const pivotData = {
                     rowFormats: [{ type: 'Date', format: 'MM/DD/YYYY' }, { type: 'Date', format: 'MMM-yy' }],
@@ -5266,7 +5266,7 @@ describe('highcharts_renderer', () => {
 
         it('should getFormattedColKeys call getFormattedKeys for colFormats', () => {
             highchartsRenderer.getFormattedColKeys('test1', 'test2');
-            expect(highchartsRenderer.getFormattedKeys).toHaveBeenCalledWith('test1', true, 'test2');
+            expect(highchartsRenderer.getFormattedKeys).toHaveBeenCalledWith('test1', true, 'test2', undefined);
         });
 
         it('should getFormattedRowKeys call getFormattedKeys for rowFormats', () => {