1 files changed, 110 insertions, 92 deletions
diff --git a/id/ConfigWebTool/src/main/java/at/gv/egovernment/moa/id/configuration/data/oa/PVPGatewayInterfederationConfig.java b/id/ConfigWebTool/src/main/java/at/gv/egovernment/moa/id/configuration/data/oa/PVPGatewayInterfederationConfig.java
index e27c55c90..29598a679 100644
--- a/id/ConfigWebTool/src/main/java/at/gv/egovernment/moa/id/configuration/data/oa/PVPGatewayInterfederationConfig.java
+++ b/id/ConfigWebTool/src/main/java/at/gv/egovernment/moa/id/configuration/data/oa/PVPGatewayInterfederationConfig.java
@@ -27,110 +27,128 @@ import java.util.List;
 
 import javax.servlet.http.HttpServletRequest;
 
-import org.apache.log4j.Logger;
-
 import at.gv.egovernment.moa.id.commons.db.dao.config.deprecated.InterfederationGatewayType;
-import at.gv.egovernment.moa.id.commons.db.dao.config.deprecated.InterfederationIDPType;
 import at.gv.egovernment.moa.id.commons.db.dao.config.deprecated.OnlineApplication;
 import at.gv.egovernment.moa.id.commons.validation.ValidationHelper;
 import at.gv.egovernment.moa.id.configuration.Constants;
 import at.gv.egovernment.moa.id.configuration.auth.AuthenticatedUser;
 import at.gv.egovernment.moa.id.configuration.helper.LanguageHelper;
 import at.gv.egovernment.moa.util.MiscUtil;
+import lombok.extern.slf4j.Slf4j;
 
 /**
  * @author tlenz
  *
  */
+@Slf4j
 public class PVPGatewayInterfederationConfig implements IOnlineApplicationData {
 
-	private static final Logger log = Logger.getLogger(PVPGatewayInterfederationConfig.class);
-	
-	private String entityID = null;
-	
-	/* (non-Javadoc)
-	 * @see at.gv.egovernment.moa.id.configuration.data.oa.IOnlineApplicationData#getName()
-	 */
-	@Override
-	public String getName() {
-		return "PVPGatewayInterfederation";
-	}
-
-	/* (non-Javadoc)
-	 * @see at.gv.egovernment.moa.id.configuration.data.oa.IOnlineApplicationData#parse(at.gv.egovernment.moa.id.commons.db.dao.config.OnlineApplication, at.gv.egovernment.moa.id.configuration.auth.AuthenticatedUser, javax.servlet.http.HttpServletRequest)
-	 */
-	@Override
-	public List<String> parse(OnlineApplication dbOA,
-			AuthenticatedUser authUser, HttpServletRequest request) {
-		
-		InterfederationGatewayType gateway = dbOA.getInterfederationGateway();
-		if (gateway != null) {
-			this.entityID = gateway.getForwardIDPIdentifier();
-
-		}
-		
-		return null;
-	}
-
-	/* (non-Javadoc)
-	 * @see at.gv.egovernment.moa.id.configuration.data.oa.IOnlineApplicationData#store(at.gv.egovernment.moa.id.commons.db.dao.config.OnlineApplication, at.gv.egovernment.moa.id.configuration.auth.AuthenticatedUser, javax.servlet.http.HttpServletRequest)
-	 */
-	@Override
-	public String store(OnlineApplication dbOA, AuthenticatedUser authUser,
-			HttpServletRequest request) {
-
-		if (authUser.isAdmin()) {			
-			dbOA.setIsInterfederationGateway(true);
-			
-			InterfederationGatewayType gateway = dbOA.getInterfederationGateway();
-			if (gateway == null) {
-				gateway = new InterfederationGatewayType();
-				dbOA.setInterfederationGateway(gateway);
-			}
-			
-			gateway.setForwardIDPIdentifier(entityID);
-		}
-		
-		dbOA.setType(Constants.MOA_CONFIG_BUSINESSSERVICE);		
-		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 (MiscUtil.isNotEmpty(entityID)) {
-			if (!ValidationHelper.validateURL(entityID)) {
-				log.info("PVP gateway EntityID is not valid");
-				errors.add(LanguageHelper.getErrorString("validation.interfederation.gateway.entityID.valid", request));
-				
-			}
-			
-		} else
-			errors.add(LanguageHelper.getErrorString("validation.interfederation.gateway.entityID.empty", request));
-				
-		return errors;
-	}
-
-	/**
-	 * @return the entityID
-	 */
-	public String getEntityID() {
-		return entityID;
-	}
-
-	/**
-	 * @param entityID the entityID to set
-	 */
-	public void setEntityID(String entityID) {
-		this.entityID = entityID;
-	}
-
-
-	
+  private String entityID = null;
+
+  /*
+   * (non-Javadoc)
+   *
+   * @see
+   * at.gv.egovernment.moa.id.configuration.data.oa.IOnlineApplicationData#getName
+   * ()
+   */
+  @Override
+  public String getName() {
+    return "PVPGatewayInterfederation";
+  }
+
+  /*
+   * (non-Javadoc)
+   *
+   * @see
+   * at.gv.egovernment.moa.id.configuration.data.oa.IOnlineApplicationData#parse(
+   * at.gv.egovernment.moa.id.commons.db.dao.config.OnlineApplication,
+   * at.gv.egovernment.moa.id.configuration.auth.AuthenticatedUser,
+   * javax.servlet.http.HttpServletRequest)
+   */
+  @Override
+  public List<String> parse(OnlineApplication dbOA,
+      AuthenticatedUser authUser, HttpServletRequest request) {
+
+    final InterfederationGatewayType gateway = dbOA.getInterfederationGateway();
+    if (gateway != null) {
+      this.entityID = gateway.getForwardIDPIdentifier();
+
+    }
+
+    return null;
+  }
+
+  /*
+   * (non-Javadoc)
+   *
+   * @see
+   * at.gv.egovernment.moa.id.configuration.data.oa.IOnlineApplicationData#store(
+   * at.gv.egovernment.moa.id.commons.db.dao.config.OnlineApplication,
+   * at.gv.egovernment.moa.id.configuration.auth.AuthenticatedUser,
+   * javax.servlet.http.HttpServletRequest)
+   */
+  @Override
+  public String store(OnlineApplication dbOA, AuthenticatedUser authUser,
+      HttpServletRequest request) {
+
+    if (authUser.isAdmin()) {
+      dbOA.setIsInterfederationGateway(true);
+
+      InterfederationGatewayType gateway = dbOA.getInterfederationGateway();
+      if (gateway == null) {
+        gateway = new InterfederationGatewayType();
+        dbOA.setInterfederationGateway(gateway);
+      }
+
+      gateway.setForwardIDPIdentifier(entityID);
+    }
+
+    dbOA.setType(Constants.MOA_CONFIG_BUSINESSSERVICE);
+    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 (MiscUtil.isNotEmpty(entityID)) {
+      if (!ValidationHelper.validateURL(entityID)) {
+        log.info("PVP gateway EntityID is not valid");
+        errors.add(LanguageHelper.getErrorString("validation.interfederation.gateway.entityID.valid",
+            request));
+
+      }
+
+    } else {
+      errors.add(LanguageHelper.getErrorString("validation.interfederation.gateway.entityID.empty", request));
+    }
+
+    return errors;
+  }
+
+  /**
+   * @return the entityID
+   */
+  public String getEntityID() {
+    return entityID;
+  }
+
+  /**
+   * @param entityID the entityID to set
+   */
+  public void setEntityID(String entityID) {
+    this.entityID = entityID;
+  }
+
 }