1 files changed, 98 insertions, 15 deletions

diff --git a/id/server/idserverlib/src/main/java/at/gv/egovernment/moa/id/protocols/stork2/MOASTORKRequest.java b/id/server/idserverlib/src/main/java/at/gv/egovernment/moa/id/protocols/stork2/MOASTORKRequest.java
index e95cfabe2..9ea33c8ef 100644
--- a/id/server/idserverlib/src/main/java/at/gv/egovernment/moa/id/protocols/stork2/MOASTORKRequest.java
+++ b/id/server/idserverlib/src/main/java/at/gv/egovernment/moa/id/protocols/stork2/MOASTORKRequest.java
@@ -9,57 +9,95 @@ import eu.stork.peps.auth.commons.STORKAttrQueryRequest;
 import eu.stork.peps.auth.commons.STORKAuthnRequest;
 
 /**
- * Implements MOA request and stores StorkAuthn/Attr-Request related data
+ * Implements MOA request and stores StorkAuthn/Attr-Request related data.
  *
  * @author bsuzic
  */
-
 public class MOASTORKRequest implements IRequest, Serializable {
+	
+	/** The Constant serialVersionUID. */
 	private static final long serialVersionUID = 4581953368724501376L;
+	
+	/** The request id. */
 	private String requestID;
+    
+    /** The target. */
     private String target = null;
+    
+    /** The module. */
     String module = null;
+    
+    /** The action. */
     String action = null;
+    
+    /** The stork authn request. */
     private STORKAuthnRequest storkAuthnRequest;
+    
+    /** The stork attr query request. */
     private STORKAttrQueryRequest storkAttrQueryRequest;
-    private boolean isAttrRequest = false;
-    private boolean isAuthnRequest = false;
 
+    /**
+     * Sets the sTORK authn request.
+     *
+     * @param request the new sTORK authn request
+     */
     public void setSTORKAuthnRequest(STORKAuthnRequest request) {
         this.storkAuthnRequest = request;
-        if (request != null) {
-            isAuthnRequest = true;
-        }
     }
 
+    /**
+     * Sets the sTORK attr request.
+     *
+     * @param request the new sTORK attr request
+     */
     public void setSTORKAttrRequest(STORKAttrQueryRequest request) {
         this.storkAttrQueryRequest = request;
-        if (request != null) {
-            isAttrRequest = true;
-        }
     }
 
+    /**
+     * Checks if the container holds an AttrQueryRequest
+     *
+     * @return true, if is attr request
+     */
     public boolean isAttrRequest() {
-        return this.isAttrRequest;
+        return null != storkAttrQueryRequest;
     }
 
+    /**
+     * Checks if the container holds an AuthnRequest
+     *
+     * @return true, if is authn request
+     */
     public boolean isAuthnRequest() {
-        return this.isAuthnRequest;
+        return null != storkAuthnRequest;
     }
 
 
+    /**
+     * Gets the stork authn request.
+     *
+     * @return the stork authn request
+     */
     public STORKAuthnRequest getStorkAuthnRequest() {
         return this.storkAuthnRequest;
     }
 
+    /**
+     * Gets the stork attr query request.
+     *
+     * @return the stork attr query request
+     */
     public STORKAttrQueryRequest getStorkAttrQueryRequest() {
         return this.storkAttrQueryRequest;
     }
 
-    public String getOAURL() {     // TODO CHECK IT
-        if (isAuthnRequest)
+    /* (non-Javadoc)
+     * @see at.gv.egovernment.moa.id.moduls.IRequest#getOAURL()
+     */
+    public String getOAURL() {
+        if (isAuthnRequest())
             return storkAuthnRequest.getAssertionConsumerServiceURL();
-        else if (isAttrRequest)
+        else if (isAttrRequest())
             return storkAttrQueryRequest.getAssertionConsumerServiceURL();
         else {
             Logger.error("There is no authentication or attribute request contained in MOASTORKRequest.");
@@ -67,46 +105,81 @@ public class MOASTORKRequest implements IRequest, Serializable {
         }
     }
 
+    /* (non-Javadoc)
+     * @see at.gv.egovernment.moa.id.moduls.IRequest#isPassiv()
+     */
     public boolean isPassiv() {
         return false;
     }
 
+    /* (non-Javadoc)
+     * @see at.gv.egovernment.moa.id.moduls.IRequest#forceAuth()
+     */
     public boolean forceAuth() {
         return false;
     }
 
+    /* (non-Javadoc)
+     * @see at.gv.egovernment.moa.id.moduls.IRequest#isSSOSupported()
+     */
     public boolean isSSOSupported() {
         return false;
     }
 
+    /* (non-Javadoc)
+     * @see at.gv.egovernment.moa.id.moduls.IRequest#requestedModule()
+     */
     public String requestedModule() {
         return this.module;
     }
 
+    /* (non-Javadoc)
+     * @see at.gv.egovernment.moa.id.moduls.IRequest#requestedAction()
+     */
     public String requestedAction() {
         return action;
     }
 
+    /* (non-Javadoc)
+     * @see at.gv.egovernment.moa.id.moduls.IRequest#setModule(java.lang.String)
+     */
     public void setModule(String module) {
         this.module = module;
     }
 
+    /* (non-Javadoc)
+     * @see at.gv.egovernment.moa.id.moduls.IRequest#setAction(java.lang.String)
+     */
     public void setAction(String action) {
         this.action = action;
     }
 
+    /* (non-Javadoc)
+     * @see at.gv.egovernment.moa.id.moduls.IRequest#getTarget()
+     */
     public String getTarget() {
         return this.target;
     }
 
+    /* (non-Javadoc)
+     * @see at.gv.egovernment.moa.id.moduls.IRequest#setRequestID(java.lang.String)
+     */
     public void setRequestID(String id) {
         this.requestID = id;
     }
 
+    /* (non-Javadoc)
+     * @see at.gv.egovernment.moa.id.moduls.IRequest#getRequestID()
+     */
     public String getRequestID() {
         return this.requestID;
     }
     
+    /**
+     * Gets the personal attribute list.
+     *
+     * @return the personal attribute list
+     */
     public IPersonalAttributeList getPersonalAttributeList() {
     	if(isAttrRequest())
     		return this.storkAttrQueryRequest.getPersonalAttributeList();
@@ -114,6 +187,11 @@ public class MOASTORKRequest implements IRequest, Serializable {
     		return this.storkAuthnRequest.getPersonalAttributeList();
     }
 
+	/**
+	 * Gets the sp country.
+	 *
+	 * @return the sp country
+	 */
 	public String getSpCountry() {
     	if(isAttrRequest())
     		return this.storkAttrQueryRequest.getSpCountry();
@@ -121,6 +199,11 @@ public class MOASTORKRequest implements IRequest, Serializable {
     		return this.storkAuthnRequest.getSpCountry();
 	}
 
+	/**
+	 * Gets the assertion consumer service url.
+	 *
+	 * @return the assertion consumer service url
+	 */
 	public String getAssertionConsumerServiceURL() {
     	if(isAttrRequest())
     		return this.storkAttrQueryRequest.getAssertionConsumerServiceURL();