chore(any-chart-web): remove unreachable code

Author: r0b1n

packages/customWidgets/any-chart-web/src/components/PlotlyChart.ts

@@ -123,10 +123,9 @@ class PlotlyChart extends Component<PlotlyChartProps> {
             });
             const layoutOptions = deepMerge.all([layout, getDimensionsFromNode(rootNode)]);
             const plotlyConfig = window.dojo && window.dojo.locale ? { ...config, locale: window.dojo.locale } : config;
-            const logger = window.mx && window.mx.logger ? window.mx.logger : window.logger;
-            if (logger && logger.debug) {
-                logger.debug("newPlot", this.chartNode, chartData as Data[], layoutOptions, plotlyConfig);
-            }
+
+            console.debug("newPlot", this.chartNode, chartData as Data[], layoutOptions, plotlyConfig);
+
             plotly.newPlot(this.chartNode, chartData as Data[], layoutOptions, plotlyConfig).then(myPlot => {
                 if (onClick) {
                     myPlot.on("plotly_click", onClick as any);

packages/customWidgets/any-chart-web/src/components/actions/PlotlyChartActions.ts

@@ -1,19 +1,10 @@
-import {
-    CLEAR_INSTANCE_STATE,
-    ChartData,
-    Plotly,
-    RESET,
-    TOGGLE_PLOTLY_API_LOADING,
-    TOGGLE_PLOTLY_DATA_LOADING,
-    UPDATE_DATA
-} from "../reducers/PlotlyChartReducer";
+import { CLEAR_INSTANCE_STATE, Plotly, TOGGLE_PLOTLY_API_LOADING } from "../reducers/PlotlyChartReducer";
 
-export const resetState = () => ({  type: RESET });
-export const togglePlotlyAPILoading = (widgetID: string, loadingAPI: boolean, plotly?: Plotly) =>
-    ({ type: TOGGLE_PLOTLY_API_LOADING, widgetID, loadingAPI, plotly });
-export const togglePlotlyDataLoading = (widgetID: string, loadingData: boolean) =>
-    ({ type: TOGGLE_PLOTLY_DATA_LOADING, widgetID, loadingData });
-export const updateData = (widgetID: string, data: ChartData) => ({ type: UPDATE_DATA, widgetID, ...data });
+export const togglePlotlyAPILoading = (widgetID: string, loadingAPI: boolean, plotly?: Plotly) => ({
+    type: TOGGLE_PLOTLY_API_LOADING,
+    widgetID,
+    loadingAPI,
+    plotly
+});
 
-export const clearInstanceState = (instanceID: string) =>
-    ({ type: CLEAR_INSTANCE_STATE, instanceID });
+export const clearInstanceState = (instanceID: string) => ({ type: CLEAR_INSTANCE_STATE, instanceID });

packages/customWidgets/any-chart-web/src/package.xml

@@ -1,6 +1,4 @@
-import { Config, Layout, Transform } from "plotly.js";
-import * as deepMerge from "deepmerge";
-import { Data } from "./namespaces";
+import { Config, Layout } from "plotly.js";
 
 export const configs: SharedConfigs = {
     layout: {
@@ -29,100 +27,7 @@ export const configs: SharedConfigs = {
     configuration: { displayModeBar: false, doubleClick: false }
 };
 
-export const fetchThemeConfigs = (type: ChartType): Promise<ChartConfigs> =>
-    new Promise<ChartConfigs>((resolve, reject) => {
-        try {
-            const cacheBurst = window.dojoConfig.cacheBust;
-            window
-                .fetch(`${window.mx.remoteUrl}com.mendix.charts.json?${cacheBurst}`)
-                .then(response => {
-                    if (response.ok) {
-                        return response.json();
-                    }
-
-                    return { layout: {}, configuration: {} };
-                })
-                .then(themeConfigs => {
-                    resolve(processChartConfigs(type, themeConfigs));
-                })
-                .catch(error => {
-                    console.log("An error occurred while fetching theme configs", error); // tslint:disable-line
-                    reject(error);
-                });
-        } catch (e) {
-            console.log("An error occurred while fetching theme configs", e); // tslint:disable-line
-            reject(e);
-        }
-    });
-
-export const processChartConfigs = (type: ChartType, themeConfigs: ThemeConfigs): ChartConfigs => {
-    const sharedLayout = themeConfigs.layout || {};
-    const sharedConfiguration = themeConfigs.configuration || {};
-    const { charts } = themeConfigs;
-    if (charts) {
-        const chartConfigs = (charts as any)[type];
-
-        return {
-            layout: deepMerge.all([sharedLayout, (chartConfigs && chartConfigs.layout) || {}]),
-            configuration: deepMerge.all([sharedConfiguration, (chartConfigs && chartConfigs.configuration) || {}]),
-            data: (chartConfigs && chartConfigs.data) || {}
-        };
-    }
-
-    return { layout: sharedLayout, configuration: sharedConfiguration, data: {} };
-};
-
-export const arrayOverwrite = (_destinationArray: any[], sourceArray: any[]) => sourceArray;
-
-export type ChartType =
-    | "LineChart"
-    | "BubbleChart"
-    | "PieChart"
-    | "HeatMap"
-    | "AnyChart"
-    | "PolarChart"
-    | "BarChart"
-    | "AreaChart"
-    | "TimeSeries"
-    | "ColumnChart";
-
 interface SharedConfigs {
     layout: Partial<Layout>;
     configuration: Partial<Config>;
 }
-
-export type ChartConfigs = SharedConfigs & { data: Partial<{}> };
-export interface ThemeConfigs extends SharedConfigs {
-    charts?: {
-        LineChart?: ChartConfigs;
-        BarChart?: ChartConfigs;
-        ColumnChart?: ChartConfigs;
-        TimeSeries?: ChartConfigs;
-        AreaChart?: ChartConfigs;
-        PieChart?: ChartConfigs;
-        PolarChart?: ChartConfigs;
-        HeatMap?: ChartConfigs;
-        BubbleChart?: ChartConfigs;
-    };
-}
-
-export const getTransforms = (series: Data.SeriesProps, traces: Data.ScatterTrace): Transform[] | undefined => {
-    const { aggregationType } = series;
-    if (aggregationType !== "none" && traces) {
-        return [
-            {
-                type: "aggregate",
-                groups: traces.x,
-                aggregations: [
-                    {
-                        target: "y",
-                        func: aggregationType,
-                        enabled: true
-                    }
-                ]
-            } as Transform
-        ];
-    }
-
-    return undefined;
-};