fixed

Issue #492
imixs · Apr 5, 2022 · 7966228 · 7966228
1 parent 7410ade
commit 7966228
Showing 1 changed file with 9 additions and 3 deletions.
diff --git a/...office-workflow-util/src/main/java/org/imixs/workflow/office/config/ConfigController.java b/...office-workflow-util/src/main/java/org/imixs/workflow/office/config/ConfigController.java
@@ -30,11 +30,13 @@
 import java.io.Serializable;
 import java.util.ArrayList;
 import java.util.List;
+import java.util.logging.Logger;
 
 import javax.annotation.PostConstruct;
 import javax.ejb.EJB;
 import javax.enterprise.context.ApplicationScoped;
 import javax.faces.model.SelectItem;
+import javax.persistence.OptimisticLockException;
 
 import org.imixs.workflow.ItemCollection;
 import org.imixs.workflow.exceptions.AccessDeniedException;
@@ -86,6 +88,8 @@ public class ConfigController implements Serializable {
 
 	private static final long serialVersionUID = 1L;
 
+	private static Logger logger = Logger.getLogger(ConfigController.class.getName());
+
 	private String name = "CONFIGURATION";
 
 	private ItemCollection configItemCollection = null;
@@ -195,10 +199,12 @@ public void save() throws AccessDeniedException {
 		// save entity
 		try {
 			configItemCollection = configService.save(configItemCollection);
-			System.out.println("...configuration " + this.getName() + " updated!");
-		} catch (Exception e) {
-			System.out.println("...Failed to save configuration " + this.getName() + " : " + e.getMessage() + " - reloading config!");
+			logger.info("...configuration " + this.getName() + " updated!");
+		} catch (OptimisticLockException e) {
+		    logger.warning("...Failed to save configuration " + this.getName() + " : " + e.getMessage() + " - reloading config!");
 			configItemCollection = configService.loadConfiguration(getName(), true);
+			// force Exception
+			throw e;
 		}
 	}