fix: check if production mode is enabled from ErrorWindow constructors

Make ErrorWindow query the current value of production mode through the ErrorManager, which exposes the ErrorWindowFactory.

Close #18
Co-authored-by: Javier Godoy <[email protected]>

Author: Michael Knigge

src/main/java/com/flowingcode/vaadin/addons/errorwindow/DefaultErrorWindowFactory.java

@@ -23,7 +23,7 @@ public class DefaultErrorWindowFactory implements ErrorWindowFactory {
 
 	@Override
 	public void showError(ErrorDetails details) {
-		ErrorWindow w = new ErrorWindow(details, isProductionMode());
+		ErrorWindow w = new ErrorWindow(details);
 		w.open();
 	}
 

src/main/java/com/flowingcode/vaadin/addons/errorwindow/ErrorManager.java

@@ -39,5 +39,9 @@ public static void showError(Throwable throwable, String cause) {
 	public static void setErrorWindowFactory(ErrorWindowFactory errorWindowFactory) {
 		ErrorManager.errorWindowFactory = errorWindowFactory;
 	}
+	
+	public static ErrorWindowFactory getErrorWindowFactory() {
+		return errorWindowFactory;
+	}
 
 }

src/main/java/com/flowingcode/vaadin/addons/errorwindow/ErrorWindow.java

@@ -63,11 +63,11 @@ public class ErrorWindow extends Dialog {
 	private boolean productionMode;
 
 	public ErrorWindow(final Throwable cause) {
-		this(cause, null, false);
+		this(cause, null, isProductionMode());
 	}
 
 	public ErrorWindow(final Throwable cause, final String errorMessage) {
-		this(cause, errorMessage, false);
+		this(cause, errorMessage, isProductionMode());
 	}
 
 	public ErrorWindow(final Throwable cause, final String errorMessage, boolean productionMode) {
@@ -80,8 +80,16 @@ public ErrorWindow(final Throwable cause, final String errorMessage, boolean pro
 		initWindow();
 	}
 
+	public ErrorWindow(ErrorDetails errorDetails) {
+		this(errorDetails.getThrowable(), errorDetails.getCause());
+	}
+
 	public ErrorWindow(ErrorDetails errorDetails, boolean productionMode) {
-		this(errorDetails.getThrowable(),errorDetails.getCause(),productionMode);
+		this(errorDetails.getThrowable(), errorDetails.getCause(), productionMode);
+	}
+
+	private static boolean isProductionMode() {
+		return ErrorManager.getErrorWindowFactory().isProductionMode();
 	}
 
 	private void initWindow() {
@@ -101,7 +109,7 @@ private VerticalLayout createMainLayout() {
 		mainLayout.setSpacing(false);
 		mainLayout.setPadding(false);
 		mainLayout.setMargin(false);
-	   
+
 		final Html title = new Html(DEFAULT_CAPTION);
 		title.getElement().getStyle().set("width", "100%");
 		mainLayout.add(title);