1 files changed, 129 insertions, 108 deletions
diff --git a/id/ConfigWebTool/src/main/java/at/gv/egovernment/moa/id/configuration/data/oa/OARevisionsLogData.java b/id/ConfigWebTool/src/main/java/at/gv/egovernment/moa/id/configuration/data/oa/OARevisionsLogData.java
index 18bebf9d8..76fd31ccd 100644
--- a/id/ConfigWebTool/src/main/java/at/gv/egovernment/moa/id/configuration/data/oa/OARevisionsLogData.java
+++ b/id/ConfigWebTool/src/main/java/at/gv/egovernment/moa/id/configuration/data/oa/OARevisionsLogData.java
@@ -39,113 +39,134 @@ import at.gv.egovernment.moa.util.MiscUtil;
  */
 public class OARevisionsLogData implements IOnlineApplicationData {
 
-	private boolean active = false;
-	private String eventCodes = null;
-	
-	/* (non-Javadoc)
-	 * @see at.gv.egovernment.moa.id.configuration.data.oa.IOnlineApplicationData#getName()
-	 */
-	@Override
-	public String getName() {
-		return "OARevisionsLogging";
-	}
-
-	/* (non-Javadoc)
-	 * @see at.gv.egovernment.moa.id.configuration.data.oa.IOnlineApplicationData#parse(at.gv.egovernment.moa.id.commons.db.dao.config.deprecated.OnlineApplication, at.gv.egovernment.moa.id.configuration.auth.AuthenticatedUser, javax.servlet.http.HttpServletRequest)
-	 */
-	@Override
-	public List<String> parse(OnlineApplication dbOA,
-			AuthenticatedUser authUser, HttpServletRequest request) {
-
-		if (dbOA.getIsRevisionsLogActive() != null)
-			active = dbOA.getIsRevisionsLogActive();
-		
-		if (MiscUtil.isNotEmpty(dbOA.getEventCodes()))
-			eventCodes = dbOA.getEventCodes();
-		
-		return null;
-		
-	}
-
-	/* (non-Javadoc)
-	 * @see at.gv.egovernment.moa.id.configuration.data.oa.IOnlineApplicationData#store(at.gv.egovernment.moa.id.commons.db.dao.config.deprecated.OnlineApplication, at.gv.egovernment.moa.id.configuration.auth.AuthenticatedUser, javax.servlet.http.HttpServletRequest)
-	 */
-	@Override
-	public String store(OnlineApplication dbOA, AuthenticatedUser authUser,
-			HttpServletRequest request) {
-		
-		dbOA.setIsRevisionsLogActive(active);
-		
-		if (MiscUtil.isNotEmpty(eventCodes)) {
-			dbOA.setEventCodes(KeyValueUtils.normalizeCSVValueString(eventCodes));
-			
-		}
-		
-		return null;
-	}
-
-	/* (non-Javadoc)
-	 * @see at.gv.egovernment.moa.id.configuration.data.oa.IOnlineApplicationData#validate(at.gv.egovernment.moa.id.configuration.data.oa.OAGeneralConfig, at.gv.egovernment.moa.id.configuration.auth.AuthenticatedUser, javax.servlet.http.HttpServletRequest)
-	 */
-	@Override
-	public List<String> validate(OAGeneralConfig general,
-			AuthenticatedUser authUser, HttpServletRequest request) {
-		List<String> errors = new ArrayList<String>();
-				
-		if (active && MiscUtil.isEmpty(eventCodes)) {
-			errors.add(LanguageHelper.getErrorString(
-					"error.oa.reversion.log.enabled"));	
-			
-		}
-		
-		if (MiscUtil.isNotEmpty(eventCodes)) {
-			String[] codes = eventCodes.split(",");
-			for (String el: codes) {
-				try {
-					Integer.parseInt(el.trim());
-					
-				} catch (NumberFormatException e) {
-					errors.add(LanguageHelper.getErrorString(
-							"error.oa.reversion.log.eventcodes"));	
-					break;
-					
-				}
-				
-			}
-			
-		}
-		
-		return errors;
-	}
-
-	/**
-	 * @return the active
-	 */
-	public boolean isActive() {
-		return active;
-	}
-
-	/**
-	 * @param active the active to set
-	 */
-	public void setActive(boolean active) {
-		this.active = active;
-	}
-
-	/**
-	 * @return the eventCodes
-	 */
-	public String getEventCodes() {
-		return eventCodes;
-	}
-
-	/**
-	 * @param eventCodes the eventCodes to set
-	 */
-	public void setEventCodes(String eventCodes) {
-		this.eventCodes = eventCodes;
-	}
-	
-	
+  private boolean active = false;
+  private String eventCodes = null;
+
+  /*
+   * (non-Javadoc)
+   *
+   * @see
+   * at.gv.egovernment.moa.id.configuration.data.oa.IOnlineApplicationData#getName
+   * ()
+   */
+  @Override
+  public String getName() {
+    return "OARevisionsLogging";
+  }
+
+  /*
+   * (non-Javadoc)
+   *
+   * @see
+   * at.gv.egovernment.moa.id.configuration.data.oa.IOnlineApplicationData#parse(
+   * at.gv.egovernment.moa.id.commons.db.dao.config.deprecated.OnlineApplication,
+   * at.gv.egovernment.moa.id.configuration.auth.AuthenticatedUser,
+   * javax.servlet.http.HttpServletRequest)
+   */
+  @Override
+  public List<String> parse(OnlineApplication dbOA,
+      AuthenticatedUser authUser, HttpServletRequest request) {
+
+    if (dbOA.getIsRevisionsLogActive() != null) {
+      active = dbOA.getIsRevisionsLogActive();
+    }
+
+    if (MiscUtil.isNotEmpty(dbOA.getEventCodes())) {
+      eventCodes = dbOA.getEventCodes();
+    }
+
+    return null;
+
+  }
+
+  /*
+   * (non-Javadoc)
+   *
+   * @see
+   * at.gv.egovernment.moa.id.configuration.data.oa.IOnlineApplicationData#store(
+   * at.gv.egovernment.moa.id.commons.db.dao.config.deprecated.OnlineApplication,
+   * at.gv.egovernment.moa.id.configuration.auth.AuthenticatedUser,
+   * javax.servlet.http.HttpServletRequest)
+   */
+  @Override
+  public String store(OnlineApplication dbOA, AuthenticatedUser authUser,
+      HttpServletRequest request) {
+
+    dbOA.setIsRevisionsLogActive(active);
+
+    if (MiscUtil.isNotEmpty(eventCodes)) {
+      dbOA.setEventCodes(KeyValueUtils.normalizeCSVValueString(eventCodes));
+
+    }
+
+    return null;
+  }
+
+  /*
+   * (non-Javadoc)
+   *
+   * @see at.gv.egovernment.moa.id.configuration.data.oa.IOnlineApplicationData#
+   * validate(at.gv.egovernment.moa.id.configuration.data.oa.OAGeneralConfig,
+   * at.gv.egovernment.moa.id.configuration.auth.AuthenticatedUser,
+   * javax.servlet.http.HttpServletRequest)
+   */
+  @Override
+  public List<String> validate(OAGeneralConfig general,
+      AuthenticatedUser authUser, HttpServletRequest request) {
+    final List<String> errors = new ArrayList<>();
+
+    if (active && MiscUtil.isEmpty(eventCodes)) {
+      errors.add(LanguageHelper.getErrorString(
+          "error.oa.reversion.log.enabled"));
+
+    }
+
+    if (MiscUtil.isNotEmpty(eventCodes)) {
+      final String[] codes = eventCodes.split(",");
+      for (final String el : codes) {
+        try {
+          Integer.parseInt(el.trim());
+
+        } catch (final NumberFormatException e) {
+          errors.add(LanguageHelper.getErrorString(
+              "error.oa.reversion.log.eventcodes"));
+          break;
+
+        }
+
+      }
+
+    }
+
+    return errors;
+  }
+
+  /**
+   * @return the active
+   */
+  public boolean isActive() {
+    return active;
+  }
+
+  /**
+   * @param active the active to set
+   */
+  public void setActive(boolean active) {
+    this.active = active;
+  }
+
+  /**
+   * @return the eventCodes
+   */
+  public String getEventCodes() {
+    return eventCodes;
+  }
+
+  /**
+   * @param eventCodes the eventCodes to set
+   */
+  public void setEventCodes(String eventCodes) {
+    this.eventCodes = eventCodes;
+  }
 
 }