feat: default to actual production mode from Vaadin

Close #79

Author: javier-godoy

pom.xml

@@ -3,7 +3,7 @@
 	<modelVersion>4.0.0</modelVersion>
 	<groupId>com.flowingcode.vaadin.addons</groupId>
 	<artifactId>error-window-vaadin</artifactId>
-	<version>4.0.2-SNAPSHOT</version>
+	<version>4.1.0-SNAPSHOT</version>
 	<name>Error Window Add-on</name>
 	<description>Error handler that displays a dialog with exception information</description>
 	<url>https://www.flowingcode.com/en/open-source/</url>

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

@@ -2,7 +2,7 @@
  * #%L
  * Error Window Add-on
  * %%
- * Copyright (C) 2017 - 2023 Flowing Code
+ * Copyright (C) 2017 - 2024 Flowing Code
  * %%
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -37,6 +37,17 @@ public interface ErrorWindowFactory {
    * @return true if the application is in production mode, false otherwise
    */
   default boolean isProductionMode() {
-    return ("true".equals(System.getProperty("productionMode")));
+    String errorWindowProductionMode = System.getProperty("errorWindowProductionMode");
+    if (errorWindowProductionMode != null) {
+      return Boolean.valueOf(errorWindowProductionMode);
+    }
+
+    String productionMode = System.getProperty("productionMode");
+    if (productionMode != null) {
+      return Boolean.valueOf(productionMode);
+    }
+
+    return VaadinServiceInitListenerImpl.getProductionMode();
   }
+
 }

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

@@ -2,7 +2,7 @@
  * #%L
  * Error Window Add-on
  * %%
- * Copyright (C) 2017 - 2023 Flowing Code
+ * Copyright (C) 2017 - 2024 Flowing Code
  * %%
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -39,6 +39,8 @@ public class VaadinServiceInitListenerImpl implements VaadinServiceInitListener
 
   private static final long serialVersionUID = 1L;
 
+  private static boolean productionMode;
+
   /**
    * Implements the {@code serviceInit} method from {@code VaadinServiceInitListener} interface.
    * This method sets {@code ErrorManager} as the error handler and registers {@link ErrorView} as
@@ -48,6 +50,8 @@ public class VaadinServiceInitListenerImpl implements VaadinServiceInitListener
    */
   @Override
   public void serviceInit(ServiceInitEvent event) {
+    productionMode = event.getSource().getDeploymentConfiguration().isProductionMode();
+
     event
         .getSource()
         .addSessionInitListener(ev -> ev.getSession().setErrorHandler(this::handleError));
@@ -76,4 +80,9 @@ private void handleError(ErrorEvent event) {
       event.getThrowable().printStackTrace();
     }
   }
+
+  static boolean getProductionMode() {
+    return productionMode;
+  }
+
 }

src/test/java/com/flowingcode/vaadin/addons/errorwindow/ErrorwindowDemo.java

@@ -2,7 +2,7 @@
  * #%L
  * Error Window Add-on
  * %%
- * Copyright (C) 2017 - 2023 Flowing Code
+ * Copyright (C) 2017 - 2024 Flowing Code
  * %%
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -30,6 +30,7 @@
 import com.vaadin.flow.component.upload.Upload;
 import com.vaadin.flow.router.PageTitle;
 import com.vaadin.flow.router.Route;
+import java.util.Optional;
 
 @DemoSource
 @PageTitle("Error Window Demo")
@@ -102,9 +103,10 @@ public ErrorwindowDemo() {
             });
 
     Checkbox productionModeCb = new Checkbox("Production Mode");
+    productionModeCb.setValue(getProductionMode());
     productionModeCb.addValueChangeListener(
         e -> {
-          System.setProperty("productionMode", "" + productionModeCb.getValue().toString());
+          setProductionMode(e.getValue());
           Notification.show(
               "Currently production mode is: " + System.getProperty("productionMode"));
         });
@@ -135,4 +137,16 @@ public ErrorwindowDemo() {
     add(upload);
 
   }
+
+  private boolean getProductionMode() {
+    return Optional.ofNullable(System.getProperty("errorWindowProductionMode"))
+        .map(Boolean::valueOf).orElseGet(() -> {
+          setProductionMode(true);
+          return Boolean.TRUE;
+        });
+  }
+
+  private void setProductionMode(boolean mode) {
+    System.setProperty("errorWindowProductionMode", Boolean.toString(mode));
+  }
 }