Skip to content

Commit 69355db

Browse files
committed
Cleanup
1 parent af9cca1 commit 69355db

File tree

5 files changed

+8
-5
lines changed

5 files changed

+8
-5
lines changed

vaadin-chartjs-wrapper/src/main/java/software/xdev/vaadin/chartjs/resources/js/src/ChartClientToServerUpdater.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,5 +24,5 @@ private ChartClientToServerUpdater()
2424
public static final String LOCATION = ChartJSSources.BASE_LOCATION + "chartClientToServerUpdater.js";
2525

2626
public static final String CHECK_IF_EXISTS_ON_CLIENT_AND_UPDATE_SERVER =
27-
"xVaadinChartjsWrapper.checkIfExistsOnClientAndUpdateServer('%s')";
27+
ChartJSSources.FUNCTION_PREFIX + "checkIfExistsOnClientAndUpdateServer('%s')";
2828
}

vaadin-chartjs-wrapper/src/main/java/software/xdev/vaadin/chartjs/resources/js/src/ChartControlFunc.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ private ChartControlFunc()
2323

2424
public static final String LOCATION = ChartJSSources.BASE_LOCATION + "chartControlFunc.js";
2525

26-
public static final String BUILD_CHART = "xVaadinChartjsWrapper.buildChart('%s','%s',%s)";
26+
public static final String BUILD_CHART = ChartJSSources.FUNCTION_PREFIX + "buildChart('%s','%s',%s)";
2727

28-
public static final String DESTROY_CHART = "xVaadinChartjsWrapper.tryDestroyChart('%s')";
28+
public static final String DESTROY_CHART = ChartJSSources.FUNCTION_PREFIX + "tryDestroyChart('%s')";
2929
}

vaadin-chartjs-wrapper/src/main/java/software/xdev/vaadin/chartjs/resources/js/src/ChartJSSources.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -22,4 +22,6 @@ private ChartJSSources()
2222
}
2323

2424
public static final String BASE_LOCATION = "./src/";
25+
26+
public static final String FUNCTION_PREFIX = "window.xVaadinChartjsWrapper.";
2527
}

vaadin-chartjs-wrapper/src/main/java/software/xdev/vaadin/chartjs/resources/js/src/ChartThemeManager.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,5 +23,6 @@ private ChartThemeManager()
2323

2424
public static final String LOCATION = ChartJSSources.BASE_LOCATION + "chartThemeManager.js";
2525

26-
public static final String TRY_UPDATE_CHART_JS_THEMING = "xVaadinChartjsWrapper.tryUpdateChartJsTheming(%s)";
26+
public static final String TRY_UPDATE_CHART_JS_THEMING =
27+
ChartJSSources.FUNCTION_PREFIX + "tryUpdateChartJsTheming(%s)";
2728
}

vaadin-chartjs-wrapper/src/main/resources/META-INF/resources/frontend/src/chartThemeManager.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
if(window.xVaadinChartjsWrapper == undefined) window.xVaadinChartjsWrapper = {};
22

33
window.xVaadinChartjsWrapper.tryUpdateChartJsTheming = function tryUpdateChartJsTheming(force) {
4-
if(!updatingChartJsThemingRequired && !force) {
4+
if(!window.xVaadinChartjsWrapper.updatingChartJsThemingRequired && !force) {
55
return;
66
}
77

0 commit comments

Comments
 (0)