Fixing disabling purpose field behaviour

Author: angelborroy-ks

esign-cert-repo/pom.xml

@@ -4,7 +4,7 @@
     <modelVersion>4.0.0</modelVersion>
     <groupId>es.keensoft.alfresco</groupId>
     <artifactId>esign-cert-repo</artifactId>
-    <version>1.3.0</version>
+    <version>1.3.1</version>
     <name>esign-cert-repo Repository AMP project</name>
     <packaging>amp</packaging>
     <description>Manages the lifecycle of the esign-cert-repo Repository AMP (Alfresco Module Package)</description>

esign-cert-share/pom.xml

@@ -4,7 +4,7 @@
     <modelVersion>4.0.0</modelVersion>
     <groupId>es.keensoft.alfresco</groupId>
     <artifactId>esign-cert-share</artifactId>
-    <version>1.3.0</version>
+    <version>1.3.1</version>
     <name>esign-cert-share AMP project</name>
     <packaging>amp</packaging>
     <description>Manages the lifecycle of the esign-cert-share AMP (Alfresco Module Package)</description>

esign-cert-share/src/main/amp/config/alfresco/web-extension/site-webscripts/es/keensoft/sign/sign-dialog.get.html.ftl

@@ -186,10 +186,9 @@
 
 	      				window.clearInterval(loadingFrameInterval);
 	      				var iframeURL = encodeURI("/share/sign/sign-frame.jsp?mimeType=${mimeType}&paramsCades=${paramsCades}&paramsPades=" + finalSignaturePosition + "&signatureAlg=${signatureAlg}");
-	      				if(signaturePurposeValue)
-	      				{
+	      				<#if signaturePurposeEnabled>
 	      					iframeURL = iframeURL.concat("&signReason=" + unicodeEscape(signaturePurposeValue));
-	      				}
+	      				</#if>
 	      				YAHOO.util.Dom.get("sign-component").innerHTML="<iframe scrolling='no' frameborder='0' allowTransparency='true' id='sign-frame' src='" + iframeURL + "' style='overflow:hidden;width:100%;border:none;height:0px;' />";
 	      				loadingSignComponentInterval = window.setInterval(doSign, 500);
 

esign-cert-share/src/main/amp/web/sign/sign-frame.jsp

@@ -10,14 +10,17 @@
 	    MiniApplet.cargarMiniApplet('<%=request.getScheme() + "://" + request.getServerName() + ":" + request.getServerPort() + request.getContextPath()%>/sign');
 	    
 	    var signatureAlg = '<%=request.getParameter("signatureAlg")%>';
-	    var signaturePurpose = '<%=request.getParameter("signReason")%>';
+	    var signaturePurpose = null;
+        <% if (request.getParameter("signReason") != null) { %>
+	        signaturePurpose = '<%=request.getParameter("signReason")%>';
+        <% } %>
 	    	    
 	    <% if (request.getParameter("mimeType").equals("pdf")) { %>
 	    
 	        var str = '<%=request.getParameter("paramsPades")%>';
 	        str = str.split('\t').join('\n');
 	        
-	        if(signaturePurpose)
+	        if (signaturePurpose)
 	        {
 	        	str = str.concat('\nsignReason=' + unicodeEscape(signaturePurpose));
 	        }