Index: lams_admin/web/WEB-INF/tags/AuthoringButton.tag
===================================================================
RCS file: /usr/local/cvsroot/lams_admin/web/WEB-INF/tags/AuthoringButton.tag,v
diff -u -r1.12 -r1.13
--- lams_admin/web/WEB-INF/tags/AuthoringButton.tag	11 May 2012 09:30:01 -0000	1.12
+++ lams_admin/web/WEB-INF/tags/AuthoringButton.tag	3 Jul 2012 12:56:40 -0000	1.13
@@ -99,10 +99,10 @@
 			var clearSessionUrl = "<c:url value='${clearSessionActionUrl}?action=" + nextAction + "&mode=${accessMode}&defineLater=${defineLater}&customiseSessionID=${customiseSessionID}&signature=${toolSignature}&toolContentID=${toolContentID}'/>";
 			doAjaxCall(clearSessionUrl);
 		} else {
-			if (window.parent.opener == null){
-				doAjaxCall(notifyCloseURL);
-			} else if ('${param.noopener}' == 'true' || notifyCloseURL.indexOf('noopener=true') >= 0){
+			if ('${param.noopener}' == 'true' || notifyCloseURL.indexOf('noopener=true') >= 0) {
 				window.location.href = notifyCloseURL;
+			} else if (window.parent.opener == null){
+				doAjaxCall(notifyCloseURL);
 			} else {
 				window.parent.opener.location.href = notifyCloseURL;
 			}
@@ -114,7 +114,7 @@
     	if(userAgent.indexOf('MSIE') != -1)
         	window.opener = "authoring"
     	window.close();
-    }				
+    }  				
 </script>	
 <p id="saveCancelButtons" >
 		<html:link href="javascript:;" property="cancel" onclick="javascript:doCancel()" styleClass="button right-buttons space-left">
Index: lams_central/web/WEB-INF/tags/AuthoringButton.tag
===================================================================
RCS file: /usr/local/cvsroot/lams_central/web/WEB-INF/tags/AuthoringButton.tag,v
diff -u -r1.19 -r1.20
--- lams_central/web/WEB-INF/tags/AuthoringButton.tag	11 May 2012 09:29:48 -0000	1.19
+++ lams_central/web/WEB-INF/tags/AuthoringButton.tag	3 Jul 2012 12:57:32 -0000	1.20
@@ -99,10 +99,10 @@
 			var clearSessionUrl = "<c:url value='${clearSessionActionUrl}?action=" + nextAction + "&mode=${accessMode}&defineLater=${defineLater}&customiseSessionID=${customiseSessionID}&signature=${toolSignature}&toolContentID=${toolContentID}'/>";
 			doAjaxCall(clearSessionUrl);
 		} else {
-			if (window.parent.opener == null){
-				doAjaxCall(notifyCloseURL);
-			} else if ('${param.noopener}' == 'true' || notifyCloseURL.indexOf('noopener=true') >= 0){
+			if ('${param.noopener}' == 'true' || notifyCloseURL.indexOf('noopener=true') >= 0) {
 				window.location.href = notifyCloseURL;
+			} else if (window.parent.opener == null){
+				doAjaxCall(notifyCloseURL);
 			} else {
 				window.parent.opener.location.href = notifyCloseURL;
 			}
Index: lams_gradebook/web/WEB-INF/tags/AuthoringButton.tag
===================================================================
RCS file: /usr/local/cvsroot/lams_gradebook/web/WEB-INF/tags/AuthoringButton.tag,v
diff -u -r1.6 -r1.7
--- lams_gradebook/web/WEB-INF/tags/AuthoringButton.tag	11 May 2012 09:29:40 -0000	1.6
+++ lams_gradebook/web/WEB-INF/tags/AuthoringButton.tag	3 Jul 2012 12:56:19 -0000	1.7
@@ -99,10 +99,10 @@
 			var clearSessionUrl = "<c:url value='${clearSessionActionUrl}?action=" + nextAction + "&mode=${accessMode}&defineLater=${defineLater}&customiseSessionID=${customiseSessionID}&signature=${toolSignature}&toolContentID=${toolContentID}'/>";
 			doAjaxCall(clearSessionUrl);
 		} else {
-			if (window.parent.opener == null){
-				doAjaxCall(notifyCloseURL);
-			} else if ('${param.noopener}' == 'true' || notifyCloseURL.indexOf('noopener=true') >= 0){
+			if ('${param.noopener}' == 'true' || notifyCloseURL.indexOf('noopener=true') >= 0) {
 				window.location.href = notifyCloseURL;
+			} else if (window.parent.opener == null){
+				doAjaxCall(notifyCloseURL);
 			} else {
 				window.parent.opener.location.href = notifyCloseURL;
 			}
@@ -114,7 +114,7 @@
     	if(userAgent.indexOf('MSIE') != -1)
         	window.opener = "authoring"
     	window.close();
-    } 				
+    }  				
 </script>	
 <p id="saveCancelButtons" >
 		<html:link href="javascript:;" property="cancel" onclick="javascript:doCancel()" styleClass="button right-buttons space-left">
Index: lams_learning/web/WEB-INF/tags/AuthoringButton.tag
===================================================================
RCS file: /usr/local/cvsroot/lams_learning/web/WEB-INF/tags/AuthoringButton.tag,v
diff -u -r1.16 -r1.17
Binary files differ
Index: lams_monitoring/web/WEB-INF/tags/AuthoringButton.tag
===================================================================
RCS file: /usr/local/cvsroot/lams_monitoring/web/WEB-INF/tags/AuthoringButton.tag,v
diff -u -r1.12 -r1.13
--- lams_monitoring/web/WEB-INF/tags/AuthoringButton.tag	11 May 2012 09:29:25 -0000	1.12
+++ lams_monitoring/web/WEB-INF/tags/AuthoringButton.tag	3 Jul 2012 12:56:47 -0000	1.13
@@ -72,7 +72,7 @@
 <script type="text/javascript">
 	//we set LAMS_AUTHORING_SUCCESS_FLAG to true in AuthoringAction.update() method 
 	if(<c:choose><c:when test="${LAMS_AUTHORING_SUCCESS_FLAG == true}">true</c:when><c:otherwise>false</c:otherwise></c:choose>){
-
+		
 		//if defineLater is true close current window
 		if (("${defineLater}" == "true") || ("${defineLater}" == "yes")) {
 			closeWindow("defineLater");
@@ -99,10 +99,10 @@
 			var clearSessionUrl = "<c:url value='${clearSessionActionUrl}?action=" + nextAction + "&mode=${accessMode}&defineLater=${defineLater}&customiseSessionID=${customiseSessionID}&signature=${toolSignature}&toolContentID=${toolContentID}'/>";
 			doAjaxCall(clearSessionUrl);
 		} else {
-			if (window.parent.opener == null){
-				doAjaxCall(notifyCloseURL);
-			} else if ('${param.noopener}' == 'true' || notifyCloseURL.indexOf('noopener=true') >= 0){
+			if ('${param.noopener}' == 'true' || notifyCloseURL.indexOf('noopener=true') >= 0) {
 				window.location.href = notifyCloseURL;
+			} else if (window.parent.opener == null){
+				doAjaxCall(notifyCloseURL);
 			} else {
 				window.parent.opener.location.href = notifyCloseURL;
 			}
@@ -114,7 +114,7 @@
     	if(userAgent.indexOf('MSIE') != -1)
         	window.opener = "authoring"
     	window.close();
-    } 				
+    }  				
 </script>	
 <p id="saveCancelButtons" >
 		<html:link href="javascript:;" property="cancel" onclick="javascript:doCancel()" styleClass="button right-buttons space-left">
Index: lams_tool_assessment/web/WEB-INF/tags/AuthoringButton.tag
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_assessment/web/WEB-INF/tags/AuthoringButton.tag,v
diff -u -r1.10 -r1.11
--- lams_tool_assessment/web/WEB-INF/tags/AuthoringButton.tag	11 May 2012 09:29:19 -0000	1.10
+++ lams_tool_assessment/web/WEB-INF/tags/AuthoringButton.tag	3 Jul 2012 12:56:59 -0000	1.11
@@ -66,15 +66,13 @@
 	<c:set var="notifyCloseURL" value="${param.notifyCloseURL}" scope="session"/>
 </c:if>
 
-<c:set var="unableToSaveMsgKey" value="authoring.msg.one.question.to.be.saved" scope="request"/>
-
 <script type="text/javascript" src="<lams:LAMSURL />includes/javascript/common.js"></script>
 
 <!-- begin tab content -->
 <script type="text/javascript">
 	//we set LAMS_AUTHORING_SUCCESS_FLAG to true in AuthoringAction.update() method 
 	if(<c:choose><c:when test="${LAMS_AUTHORING_SUCCESS_FLAG == true}">true</c:when><c:otherwise>false</c:otherwise></c:choose>){
-
+		
 		//if defineLater is true close current window
 		if (("${defineLater}" == "true") || ("${defineLater}" == "yes")) {
 			closeWindow("defineLater");
@@ -85,18 +83,7 @@
 		}
 	}
     function doSubmit_Form_Only() {
-    	var amountOfQuestions = $('#questionTable tr').size();
-    	var amountOfReferences = $('#referencesTable tr').size();
-
-		if( amountOfQuestions <= 1 ) {
-			$("#overallFeedbackList").val($('#advancedInputArea').contents().find('#overallFeedbackForm').serialize(true));
-	    	document.getElementById("${formID}").submit();
-		} else if (amountOfReferences > amountOfQuestions) {
-			alert("<fmt:message key='label.authoring.basic.warning.too.many.questions'/>");
-		} else {
-	    	$("#overallFeedbackList").val($('#advancedInputArea').contents().find('#overallFeedbackForm').serialize(true));
-	    	document.getElementById("${formID}").submit();
-		}
+    	document.getElementById("${formID}").submit();
     }
     function doCancel() {
     	if(confirm("<fmt:message key='${cancelConfirmMsgKey}'/>")){
@@ -112,10 +99,10 @@
 			var clearSessionUrl = "<c:url value='${clearSessionActionUrl}?action=" + nextAction + "&mode=${accessMode}&defineLater=${defineLater}&customiseSessionID=${customiseSessionID}&signature=${toolSignature}&toolContentID=${toolContentID}'/>";
 			doAjaxCall(clearSessionUrl);
 		} else {
-			if (window.parent.opener == null){
-				doAjaxCall(notifyCloseURL);
-			} else if ('${param.noopener}' == 'true' || notifyCloseURL.indexOf('noopener=true') >= 0){
+			if ('${param.noopener}' == 'true' || notifyCloseURL.indexOf('noopener=true') >= 0) {
 				window.location.href = notifyCloseURL;
+			} else if (window.parent.opener == null){
+				doAjaxCall(notifyCloseURL);
 			} else {
 				window.parent.opener.location.href = notifyCloseURL;
 			}
@@ -127,7 +114,7 @@
     	if(userAgent.indexOf('MSIE') != -1)
         	window.opener = "authoring"
     	window.close();
-    } 				
+    }  				
 </script>	
 <p id="saveCancelButtons" >
 		<html:link href="javascript:;" property="cancel" onclick="javascript:doCancel()" styleClass="button right-buttons space-left">
Index: lams_tool_bbb/web/WEB-INF/tags/AuthoringButton.tag
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_bbb/web/WEB-INF/tags/AuthoringButton.tag,v
diff -u -r1.5 -r1.6
--- lams_tool_bbb/web/WEB-INF/tags/AuthoringButton.tag	11 May 2012 09:29:24 -0000	1.5
+++ lams_tool_bbb/web/WEB-INF/tags/AuthoringButton.tag	3 Jul 2012 12:57:14 -0000	1.6
@@ -72,7 +72,7 @@
 <script type="text/javascript">
 	//we set LAMS_AUTHORING_SUCCESS_FLAG to true in AuthoringAction.update() method 
 	if(<c:choose><c:when test="${LAMS_AUTHORING_SUCCESS_FLAG == true}">true</c:when><c:otherwise>false</c:otherwise></c:choose>){
-
+		
 		//if defineLater is true close current window
 		if (("${defineLater}" == "true") || ("${defineLater}" == "yes")) {
 			closeWindow("defineLater");
@@ -99,10 +99,10 @@
 			var clearSessionUrl = "<c:url value='${clearSessionActionUrl}?action=" + nextAction + "&mode=${accessMode}&defineLater=${defineLater}&customiseSessionID=${customiseSessionID}&signature=${toolSignature}&toolContentID=${toolContentID}'/>";
 			doAjaxCall(clearSessionUrl);
 		} else {
-			if (window.parent.opener == null){
-				doAjaxCall(notifyCloseURL);
-			} else if ('${param.noopener}' == 'true' || notifyCloseURL.indexOf('noopener=true') >= 0){
+			if ('${param.noopener}' == 'true' || notifyCloseURL.indexOf('noopener=true') >= 0) {
 				window.location.href = notifyCloseURL;
+			} else if (window.parent.opener == null){
+				doAjaxCall(notifyCloseURL);
 			} else {
 				window.parent.opener.location.href = notifyCloseURL;
 			}
Index: lams_tool_chat/web/WEB-INF/tags/AuthoringButton.tag
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_chat/web/WEB-INF/tags/AuthoringButton.tag,v
diff -u -r1.16 -r1.17
--- lams_tool_chat/web/WEB-INF/tags/AuthoringButton.tag	11 May 2012 09:30:11 -0000	1.16
+++ lams_tool_chat/web/WEB-INF/tags/AuthoringButton.tag	3 Jul 2012 12:58:03 -0000	1.17
@@ -72,7 +72,7 @@
 <script type="text/javascript">
 	//we set LAMS_AUTHORING_SUCCESS_FLAG to true in AuthoringAction.update() method 
 	if(<c:choose><c:when test="${LAMS_AUTHORING_SUCCESS_FLAG == true}">true</c:when><c:otherwise>false</c:otherwise></c:choose>){
-
+		
 		//if defineLater is true close current window
 		if (("${defineLater}" == "true") || ("${defineLater}" == "yes")) {
 			closeWindow("defineLater");
@@ -99,10 +99,10 @@
 			var clearSessionUrl = "<c:url value='${clearSessionActionUrl}?action=" + nextAction + "&mode=${accessMode}&defineLater=${defineLater}&customiseSessionID=${customiseSessionID}&signature=${toolSignature}&toolContentID=${toolContentID}'/>";
 			doAjaxCall(clearSessionUrl);
 		} else {
-			if (window.parent.opener == null){
-				doAjaxCall(notifyCloseURL);
-			} else if ('${param.noopener}' == 'true' || notifyCloseURL.indexOf('noopener=true') >= 0){
+			if ('${param.noopener}' == 'true' || notifyCloseURL.indexOf('noopener=true') >= 0) {
 				window.location.href = notifyCloseURL;
+			} else if (window.parent.opener == null){
+				doAjaxCall(notifyCloseURL);
 			} else {
 				window.parent.opener.location.href = notifyCloseURL;
 			}
Index: lams_tool_daco/web/WEB-INF/tags/AuthoringButton.tag
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_daco/web/WEB-INF/tags/AuthoringButton.tag,v
diff -u -r1.8 -r1.9
--- lams_tool_daco/web/WEB-INF/tags/AuthoringButton.tag	11 May 2012 09:29:22 -0000	1.8
+++ lams_tool_daco/web/WEB-INF/tags/AuthoringButton.tag	3 Jul 2012 12:57:25 -0000	1.9
@@ -72,7 +72,7 @@
 <script type="text/javascript">
 	//we set LAMS_AUTHORING_SUCCESS_FLAG to true in AuthoringAction.update() method 
 	if(<c:choose><c:when test="${LAMS_AUTHORING_SUCCESS_FLAG == true}">true</c:when><c:otherwise>false</c:otherwise></c:choose>){
-
+		
 		//if defineLater is true close current window
 		if (("${defineLater}" == "true") || ("${defineLater}" == "yes")) {
 			closeWindow("defineLater");
@@ -87,9 +87,9 @@
     }
     function doCancel() {
     	if(confirm("<fmt:message key='${cancelConfirmMsgKey}'/>")){
-	    	closeWindow("cancel");
-	    }
-	}
+    		closeWindow("cancel");
+    	}
+    }
     function closeWindow(nextAction) {
 		var notifyCloseURL = "${notifyCloseURL}";
 		if (notifyCloseURL == ""){
@@ -99,10 +99,10 @@
 			var clearSessionUrl = "<c:url value='${clearSessionActionUrl}?action=" + nextAction + "&mode=${accessMode}&defineLater=${defineLater}&customiseSessionID=${customiseSessionID}&signature=${toolSignature}&toolContentID=${toolContentID}'/>";
 			doAjaxCall(clearSessionUrl);
 		} else {
-			if (window.parent.opener == null){
-				doAjaxCall(notifyCloseURL);
-			} else if ('${param.noopener}' == 'true' || notifyCloseURL.indexOf('noopener=true') >= 0){
+			if ('${param.noopener}' == 'true' || notifyCloseURL.indexOf('noopener=true') >= 0) {
 				window.location.href = notifyCloseURL;
+			} else if (window.parent.opener == null){
+				doAjaxCall(notifyCloseURL);
 			} else {
 				window.parent.opener.location.href = notifyCloseURL;
 			}
@@ -114,7 +114,7 @@
     	if(userAgent.indexOf('MSIE') != -1)
         	window.opener = "authoring"
     	window.close();
-    } 				
+    }  				
 </script>	
 <p id="saveCancelButtons" >
 		<html:link href="javascript:;" property="cancel" onclick="javascript:doCancel()" styleClass="button right-buttons space-left">
Index: lams_tool_dimdim/web/WEB-INF/tags/AuthoringButton.tag
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_dimdim/web/WEB-INF/tags/Attic/AuthoringButton.tag,v
diff -u -r1.7 -r1.8
--- lams_tool_dimdim/web/WEB-INF/tags/AuthoringButton.tag	11 May 2012 09:29:59 -0000	1.7
+++ lams_tool_dimdim/web/WEB-INF/tags/AuthoringButton.tag	3 Jul 2012 12:58:00 -0000	1.8
@@ -72,7 +72,7 @@
 <script type="text/javascript">
 	//we set LAMS_AUTHORING_SUCCESS_FLAG to true in AuthoringAction.update() method 
 	if(<c:choose><c:when test="${LAMS_AUTHORING_SUCCESS_FLAG == true}">true</c:when><c:otherwise>false</c:otherwise></c:choose>){
-
+		
 		//if defineLater is true close current window
 		if (("${defineLater}" == "true") || ("${defineLater}" == "yes")) {
 			closeWindow("defineLater");
@@ -99,10 +99,10 @@
 			var clearSessionUrl = "<c:url value='${clearSessionActionUrl}?action=" + nextAction + "&mode=${accessMode}&defineLater=${defineLater}&customiseSessionID=${customiseSessionID}&signature=${toolSignature}&toolContentID=${toolContentID}'/>";
 			doAjaxCall(clearSessionUrl);
 		} else {
-			if (window.parent.opener == null){
-				doAjaxCall(notifyCloseURL);
-			} else if ('${param.noopener}' == 'true' || notifyCloseURL.indexOf('noopener=true') >= 0){
+			if ('${param.noopener}' == 'true' || notifyCloseURL.indexOf('noopener=true') >= 0) {
 				window.location.href = notifyCloseURL;
+			} else if (window.parent.opener == null){
+				doAjaxCall(notifyCloseURL);
 			} else {
 				window.parent.opener.location.href = notifyCloseURL;
 			}
@@ -114,7 +114,7 @@
     	if(userAgent.indexOf('MSIE') != -1)
         	window.opener = "authoring"
     	window.close();
-    } 				
+    }  				
 </script>	
 <p id="saveCancelButtons" >
 		<html:link href="javascript:;" property="cancel" onclick="javascript:doCancel()" styleClass="button right-buttons space-left">
Index: lams_tool_eadventure/web/WEB-INF/tags/AuthoringButton.tag
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_eadventure/web/WEB-INF/tags/AuthoringButton.tag,v
diff -u -r1.7 -r1.8
--- lams_tool_eadventure/web/WEB-INF/tags/AuthoringButton.tag	11 May 2012 09:30:08 -0000	1.7
+++ lams_tool_eadventure/web/WEB-INF/tags/AuthoringButton.tag	3 Jul 2012 12:57:54 -0000	1.8
@@ -72,7 +72,7 @@
 <script type="text/javascript">
 	//we set LAMS_AUTHORING_SUCCESS_FLAG to true in AuthoringAction.update() method 
 	if(<c:choose><c:when test="${LAMS_AUTHORING_SUCCESS_FLAG == true}">true</c:when><c:otherwise>false</c:otherwise></c:choose>){
-
+		
 		//if defineLater is true close current window
 		if (("${defineLater}" == "true") || ("${defineLater}" == "yes")) {
 			closeWindow("defineLater");
@@ -99,10 +99,10 @@
 			var clearSessionUrl = "<c:url value='${clearSessionActionUrl}?action=" + nextAction + "&mode=${accessMode}&defineLater=${defineLater}&customiseSessionID=${customiseSessionID}&signature=${toolSignature}&toolContentID=${toolContentID}'/>";
 			doAjaxCall(clearSessionUrl);
 		} else {
-			if (window.parent.opener == null){
-				doAjaxCall(notifyCloseURL);
-			} else if ('${param.noopener}' == 'true' || notifyCloseURL.indexOf('noopener=true') >= 0){
+			if ('${param.noopener}' == 'true' || notifyCloseURL.indexOf('noopener=true') >= 0) {
 				window.location.href = notifyCloseURL;
+			} else if (window.parent.opener == null){
+				doAjaxCall(notifyCloseURL);
 			} else {
 				window.parent.opener.location.href = notifyCloseURL;
 			}
Index: lams_tool_forum/web/WEB-INF/tags/AuthoringButton.tag
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_forum/web/WEB-INF/tags/AuthoringButton.tag,v
diff -u -r1.18 -r1.19
--- lams_tool_forum/web/WEB-INF/tags/AuthoringButton.tag	11 May 2012 09:29:39 -0000	1.18
+++ lams_tool_forum/web/WEB-INF/tags/AuthoringButton.tag	3 Jul 2012 12:56:33 -0000	1.19
@@ -99,10 +99,10 @@
 			var clearSessionUrl = "<c:url value='${clearSessionActionUrl}?action=" + nextAction + "&mode=${accessMode}&defineLater=${defineLater}&customiseSessionID=${customiseSessionID}&signature=${toolSignature}&toolContentID=${toolContentID}'/>";
 			doAjaxCall(clearSessionUrl);
 		} else {
-			if (window.parent.opener == null){
-				doAjaxCall(notifyCloseURL);
-			} else if ('${param.noopener}' == 'true' || notifyCloseURL.indexOf('noopener=true') >= 0){
+			if ('${param.noopener}' == 'true' || notifyCloseURL.indexOf('noopener=true') >= 0) {
 				window.location.href = notifyCloseURL;
+			} else if (window.parent.opener == null){
+				doAjaxCall(notifyCloseURL);
 			} else {
 				window.parent.opener.location.href = notifyCloseURL;
 			}
Index: lams_tool_gmap/web/WEB-INF/tags/AuthoringButton.tag
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_gmap/web/WEB-INF/tags/AuthoringButton.tag,v
diff -u -r1.10 -r1.11
--- lams_tool_gmap/web/WEB-INF/tags/AuthoringButton.tag	11 May 2012 09:29:47 -0000	1.10
+++ lams_tool_gmap/web/WEB-INF/tags/AuthoringButton.tag	3 Jul 2012 12:57:48 -0000	1.11
@@ -72,7 +72,7 @@
 <script type="text/javascript">
 	//we set LAMS_AUTHORING_SUCCESS_FLAG to true in AuthoringAction.update() method 
 	if(<c:choose><c:when test="${LAMS_AUTHORING_SUCCESS_FLAG == true}">true</c:when><c:otherwise>false</c:otherwise></c:choose>){
-
+		
 		//if defineLater is true close current window
 		if (("${defineLater}" == "true") || ("${defineLater}" == "yes")) {
 			closeWindow("defineLater");
@@ -89,7 +89,7 @@
     	if(confirm("<fmt:message key='${cancelConfirmMsgKey}'/>")){
     		closeWindow("cancel");
     	}
-    }  	
+    }
     function closeWindow(nextAction) {
 		var notifyCloseURL = "${notifyCloseURL}";
 		if (notifyCloseURL == ""){
@@ -99,10 +99,10 @@
 			var clearSessionUrl = "<c:url value='${clearSessionActionUrl}?action=" + nextAction + "&mode=${accessMode}&defineLater=${defineLater}&customiseSessionID=${customiseSessionID}&signature=${toolSignature}&toolContentID=${toolContentID}'/>";
 			doAjaxCall(clearSessionUrl);
 		} else {
-			if (window.parent.opener == null){
-				doAjaxCall(notifyCloseURL);
-			} else if ('${param.noopener}' == 'true' || notifyCloseURL.indexOf('noopener=true') >= 0){
+			if ('${param.noopener}' == 'true' || notifyCloseURL.indexOf('noopener=true') >= 0) {
 				window.location.href = notifyCloseURL;
+			} else if (window.parent.opener == null){
+				doAjaxCall(notifyCloseURL);
 			} else {
 				window.parent.opener.location.href = notifyCloseURL;
 			}
@@ -114,7 +114,7 @@
     	if(userAgent.indexOf('MSIE') != -1)
         	window.opener = "authoring"
     	window.close();
-    }
+    }  				
 </script>	
 <p id="saveCancelButtons" >
 		<html:link href="javascript:;" property="cancel" onclick="javascript:doCancel()" styleClass="button right-buttons space-left">
Index: lams_tool_images/web/WEB-INF/tags/AuthoringButton.tag
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_images/web/WEB-INF/tags/AuthoringButton.tag,v
diff -u -r1.7 -r1.8
--- lams_tool_images/web/WEB-INF/tags/AuthoringButton.tag	11 May 2012 09:29:17 -0000	1.7
+++ lams_tool_images/web/WEB-INF/tags/AuthoringButton.tag	3 Jul 2012 12:57:05 -0000	1.8
@@ -72,7 +72,7 @@
 <script type="text/javascript">
 	//we set LAMS_AUTHORING_SUCCESS_FLAG to true in AuthoringAction.update() method 
 	if(<c:choose><c:when test="${LAMS_AUTHORING_SUCCESS_FLAG == true}">true</c:when><c:otherwise>false</c:otherwise></c:choose>){
-
+		
 		//if defineLater is true close current window
 		if (("${defineLater}" == "true") || ("${defineLater}" == "yes")) {
 			closeWindow("defineLater");
@@ -99,10 +99,10 @@
 			var clearSessionUrl = "<c:url value='${clearSessionActionUrl}?action=" + nextAction + "&mode=${accessMode}&defineLater=${defineLater}&customiseSessionID=${customiseSessionID}&signature=${toolSignature}&toolContentID=${toolContentID}'/>";
 			doAjaxCall(clearSessionUrl);
 		} else {
-			if (window.parent.opener == null){
-				doAjaxCall(notifyCloseURL);
-			} else if ('${param.noopener}' == 'true' || notifyCloseURL.indexOf('noopener=true') >= 0){
+			if ('${param.noopener}' == 'true' || notifyCloseURL.indexOf('noopener=true') >= 0) {
 				window.location.href = notifyCloseURL;
+			} else if (window.parent.opener == null){
+				doAjaxCall(notifyCloseURL);
 			} else {
 				window.parent.opener.location.href = notifyCloseURL;
 			}
@@ -114,7 +114,7 @@
     	if(userAgent.indexOf('MSIE') != -1)
         	window.opener = "authoring"
     	window.close();
-    }
+    }  				
 </script>	
 <p id="saveCancelButtons" >
 		<html:link href="javascript:;" property="cancel" onclick="javascript:doCancel()" styleClass="button right-buttons space-left">
Index: lams_tool_imscc/web/WEB-INF/tags/AuthoringButton.tag
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_imscc/web/WEB-INF/tags/AuthoringButton.tag,v
diff -u -r1.5 -r1.6
--- lams_tool_imscc/web/WEB-INF/tags/AuthoringButton.tag	11 May 2012 09:29:29 -0000	1.5
+++ lams_tool_imscc/web/WEB-INF/tags/AuthoringButton.tag	3 Jul 2012 12:58:07 -0000	1.6
@@ -72,7 +72,7 @@
 <script type="text/javascript">
 	//we set LAMS_AUTHORING_SUCCESS_FLAG to true in AuthoringAction.update() method 
 	if(<c:choose><c:when test="${LAMS_AUTHORING_SUCCESS_FLAG == true}">true</c:when><c:otherwise>false</c:otherwise></c:choose>){
-
+		
 		//if defineLater is true close current window
 		if (("${defineLater}" == "true") || ("${defineLater}" == "yes")) {
 			closeWindow("defineLater");
@@ -99,10 +99,10 @@
 			var clearSessionUrl = "<c:url value='${clearSessionActionUrl}?action=" + nextAction + "&mode=${accessMode}&defineLater=${defineLater}&customiseSessionID=${customiseSessionID}&signature=${toolSignature}&toolContentID=${toolContentID}'/>";
 			doAjaxCall(clearSessionUrl);
 		} else {
-			if (window.parent.opener == null){
-				doAjaxCall(notifyCloseURL);
-			} else if ('${param.noopener}' == 'true' || notifyCloseURL.indexOf('noopener=true') >= 0){
+			if ('${param.noopener}' == 'true' || notifyCloseURL.indexOf('noopener=true') >= 0) {
 				window.location.href = notifyCloseURL;
+			} else if (window.parent.opener == null){
+				doAjaxCall(notifyCloseURL);
 			} else {
 				window.parent.opener.location.href = notifyCloseURL;
 			}
@@ -114,7 +114,7 @@
     	if(userAgent.indexOf('MSIE') != -1)
         	window.opener = "authoring"
     	window.close();
-    } 				
+    }  				
 </script>	
 <p id="saveCancelButtons" >
 		<html:link href="javascript:;" property="cancel" onclick="javascript:doCancel()" styleClass="button right-buttons space-left">
Index: lams_tool_lamc/web/WEB-INF/tags/AuthoringButton.tag
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_lamc/web/WEB-INF/tags/AuthoringButton.tag,v
diff -u -r1.15 -r1.16
--- lams_tool_lamc/web/WEB-INF/tags/AuthoringButton.tag	11 May 2012 09:29:16 -0000	1.15
+++ lams_tool_lamc/web/WEB-INF/tags/AuthoringButton.tag	3 Jul 2012 12:57:41 -0000	1.16
@@ -72,7 +72,7 @@
 <script type="text/javascript">
 	//we set LAMS_AUTHORING_SUCCESS_FLAG to true in AuthoringAction.update() method 
 	if(<c:choose><c:when test="${LAMS_AUTHORING_SUCCESS_FLAG == true}">true</c:when><c:otherwise>false</c:otherwise></c:choose>){
-
+		
 		//if defineLater is true close current window
 		if (("${defineLater}" == "true") || ("${defineLater}" == "yes")) {
 			closeWindow("defineLater");
@@ -99,10 +99,10 @@
 			var clearSessionUrl = "<c:url value='${clearSessionActionUrl}?action=" + nextAction + "&mode=${accessMode}&defineLater=${defineLater}&customiseSessionID=${customiseSessionID}&signature=${toolSignature}&toolContentID=${toolContentID}'/>";
 			doAjaxCall(clearSessionUrl);
 		} else {
-			if (window.parent.opener == null){
-				doAjaxCall(notifyCloseURL);
-			} else if ('${param.noopener}' == 'true' || notifyCloseURL.indexOf('noopener=true') >= 0){
+			if ('${param.noopener}' == 'true' || notifyCloseURL.indexOf('noopener=true') >= 0) {
 				window.location.href = notifyCloseURL;
+			} else if (window.parent.opener == null){
+				doAjaxCall(notifyCloseURL);
 			} else {
 				window.parent.opener.location.href = notifyCloseURL;
 			}
Index: lams_tool_laqa/web/WEB-INF/tags/AuthoringButton.tag
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_laqa/web/WEB-INF/tags/AuthoringButton.tag,v
diff -u -r1.16 -r1.17
--- lams_tool_laqa/web/WEB-INF/tags/AuthoringButton.tag	11 May 2012 09:29:45 -0000	1.16
+++ lams_tool_laqa/web/WEB-INF/tags/AuthoringButton.tag	3 Jul 2012 12:57:28 -0000	1.17
@@ -72,7 +72,7 @@
 <script type="text/javascript">
 	//we set LAMS_AUTHORING_SUCCESS_FLAG to true in AuthoringAction.update() method 
 	if(<c:choose><c:when test="${LAMS_AUTHORING_SUCCESS_FLAG == true}">true</c:when><c:otherwise>false</c:otherwise></c:choose>){
-
+		
 		//if defineLater is true close current window
 		if (("${defineLater}" == "true") || ("${defineLater}" == "yes")) {
 			closeWindow("defineLater");
@@ -99,10 +99,10 @@
 			var clearSessionUrl = "<c:url value='${clearSessionActionUrl}?action=" + nextAction + "&mode=${accessMode}&defineLater=${defineLater}&customiseSessionID=${customiseSessionID}&signature=${toolSignature}&toolContentID=${toolContentID}'/>";
 			doAjaxCall(clearSessionUrl);
 		} else {
-			if (window.parent.opener == null){
-				doAjaxCall(notifyCloseURL);
-			} else if ('${param.noopener}' == 'true' || notifyCloseURL.indexOf('noopener=true') >= 0){
+			if ('${param.noopener}' == 'true' || notifyCloseURL.indexOf('noopener=true') >= 0) {
 				window.location.href = notifyCloseURL;
+			} else if (window.parent.opener == null){
+				doAjaxCall(notifyCloseURL);
 			} else {
 				window.parent.opener.location.href = notifyCloseURL;
 			}
@@ -114,7 +114,7 @@
     	if(userAgent.indexOf('MSIE') != -1)
         	window.opener = "authoring"
     	window.close();
-    } 				
+    }  				
 </script>	
 <p id="saveCancelButtons" >
 		<html:link href="javascript:;" property="cancel" onclick="javascript:doCancel()" styleClass="button right-buttons space-left">
Index: lams_tool_larsrc/web/WEB-INF/tags/AuthoringButton.tag
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_larsrc/web/WEB-INF/tags/AuthoringButton.tag,v
diff -u -r1.19 -r1.20
--- lams_tool_larsrc/web/WEB-INF/tags/AuthoringButton.tag	11 May 2012 09:29:43 -0000	1.19
+++ lams_tool_larsrc/web/WEB-INF/tags/AuthoringButton.tag	3 Jul 2012 12:57:35 -0000	1.20
@@ -72,7 +72,7 @@
 <script type="text/javascript">
 	//we set LAMS_AUTHORING_SUCCESS_FLAG to true in AuthoringAction.update() method 
 	if(<c:choose><c:when test="${LAMS_AUTHORING_SUCCESS_FLAG == true}">true</c:when><c:otherwise>false</c:otherwise></c:choose>){
-
+		
 		//if defineLater is true close current window
 		if (("${defineLater}" == "true") || ("${defineLater}" == "yes")) {
 			closeWindow("defineLater");
@@ -99,10 +99,10 @@
 			var clearSessionUrl = "<c:url value='${clearSessionActionUrl}?action=" + nextAction + "&mode=${accessMode}&defineLater=${defineLater}&customiseSessionID=${customiseSessionID}&signature=${toolSignature}&toolContentID=${toolContentID}'/>";
 			doAjaxCall(clearSessionUrl);
 		} else {
-			if (window.parent.opener == null){
-				doAjaxCall(notifyCloseURL);
-			} else if ('${param.noopener}' == 'true' || notifyCloseURL.indexOf('noopener=true') >= 0){
+			if ('${param.noopener}' == 'true' || notifyCloseURL.indexOf('noopener=true') >= 0) {
 				window.location.href = notifyCloseURL;
+			} else if (window.parent.opener == null){
+				doAjaxCall(notifyCloseURL);
 			} else {
 				window.parent.opener.location.href = notifyCloseURL;
 			}
Index: lams_tool_mindmap/web/WEB-INF/tags/AuthoringButton.tag
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_mindmap/web/WEB-INF/tags/AuthoringButton.tag,v
diff -u -r1.6 -r1.7
--- lams_tool_mindmap/web/WEB-INF/tags/AuthoringButton.tag	11 May 2012 09:29:56 -0000	1.6
+++ lams_tool_mindmap/web/WEB-INF/tags/AuthoringButton.tag	3 Jul 2012 12:57:18 -0000	1.7
@@ -72,7 +72,7 @@
 <script type="text/javascript">
 	//we set LAMS_AUTHORING_SUCCESS_FLAG to true in AuthoringAction.update() method 
 	if(<c:choose><c:when test="${LAMS_AUTHORING_SUCCESS_FLAG == true}">true</c:when><c:otherwise>false</c:otherwise></c:choose>){
-
+		
 		//if defineLater is true close current window
 		if (("${defineLater}" == "true") || ("${defineLater}" == "yes")) {
 			closeWindow("defineLater");
@@ -99,10 +99,10 @@
 			var clearSessionUrl = "<c:url value='${clearSessionActionUrl}?action=" + nextAction + "&mode=${accessMode}&defineLater=${defineLater}&customiseSessionID=${customiseSessionID}&signature=${toolSignature}&toolContentID=${toolContentID}'/>";
 			doAjaxCall(clearSessionUrl);
 		} else {
-			if (window.parent.opener == null){
-				doAjaxCall(notifyCloseURL);
-			} else if ('${param.noopener}' == 'true' || notifyCloseURL.indexOf('noopener=true') >= 0){
+			if ('${param.noopener}' == 'true' || notifyCloseURL.indexOf('noopener=true') >= 0) {
 				window.location.href = notifyCloseURL;
+			} else if (window.parent.opener == null){
+				doAjaxCall(notifyCloseURL);
 			} else {
 				window.parent.opener.location.href = notifyCloseURL;
 			}
@@ -114,7 +114,7 @@
     	if(userAgent.indexOf('MSIE') != -1)
         	window.opener = "authoring"
     	window.close();
-    } 				
+    }  				
 </script>	
 <p id="saveCancelButtons" >
 		<html:link href="javascript:;" property="cancel" onclick="javascript:doCancel()" styleClass="button right-buttons space-left">
Index: lams_tool_nb/web/WEB-INF/tags/AuthoringButton.tag
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_nb/web/WEB-INF/tags/AuthoringButton.tag,v
diff -u -r1.16 -r1.17
--- lams_tool_nb/web/WEB-INF/tags/AuthoringButton.tag	11 May 2012 09:30:03 -0000	1.16
+++ lams_tool_nb/web/WEB-INF/tags/AuthoringButton.tag	3 Jul 2012 12:57:38 -0000	1.17
@@ -72,7 +72,7 @@
 <script type="text/javascript">
 	//we set LAMS_AUTHORING_SUCCESS_FLAG to true in AuthoringAction.update() method 
 	if(<c:choose><c:when test="${LAMS_AUTHORING_SUCCESS_FLAG == true}">true</c:when><c:otherwise>false</c:otherwise></c:choose>){
-
+		
 		//if defineLater is true close current window
 		if (("${defineLater}" == "true") || ("${defineLater}" == "yes")) {
 			closeWindow("defineLater");
@@ -89,7 +89,7 @@
     	if(confirm("<fmt:message key='${cancelConfirmMsgKey}'/>")){
     		closeWindow("cancel");
     	}
-    } 
+    }
     function closeWindow(nextAction) {
 		var notifyCloseURL = "${notifyCloseURL}";
 		if (notifyCloseURL == ""){
@@ -99,10 +99,10 @@
 			var clearSessionUrl = "<c:url value='${clearSessionActionUrl}?action=" + nextAction + "&mode=${accessMode}&defineLater=${defineLater}&customiseSessionID=${customiseSessionID}&signature=${toolSignature}&toolContentID=${toolContentID}'/>";
 			doAjaxCall(clearSessionUrl);
 		} else {
-			if (window.parent.opener == null){
-				doAjaxCall(notifyCloseURL);
-			} else if ('${param.noopener}' == 'true' || notifyCloseURL.indexOf('noopener=true') >= 0){
+			if ('${param.noopener}' == 'true' || notifyCloseURL.indexOf('noopener=true') >= 0) {
 				window.location.href = notifyCloseURL;
+			} else if (window.parent.opener == null){
+				doAjaxCall(notifyCloseURL);
 			} else {
 				window.parent.opener.location.href = notifyCloseURL;
 			}
@@ -114,7 +114,7 @@
     	if(userAgent.indexOf('MSIE') != -1)
         	window.opener = "authoring"
     	window.close();
-    }
+    }  				
 </script>	
 <p id="saveCancelButtons" >
 		<html:link href="javascript:;" property="cancel" onclick="javascript:doCancel()" styleClass="button right-buttons space-left">
Index: lams_tool_notebook/web/WEB-INF/tags/AuthoringButton.tag
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_notebook/web/WEB-INF/tags/AuthoringButton.tag,v
diff -u -r1.14 -r1.15
Binary files differ
Index: lams_tool_pixlr/web/WEB-INF/tags/AuthoringButton.tag
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_pixlr/web/WEB-INF/tags/AuthoringButton.tag,v
diff -u -r1.7 -r1.8
--- lams_tool_pixlr/web/WEB-INF/tags/AuthoringButton.tag	11 May 2012 09:29:37 -0000	1.7
+++ lams_tool_pixlr/web/WEB-INF/tags/AuthoringButton.tag	3 Jul 2012 12:56:50 -0000	1.8
@@ -72,7 +72,7 @@
 <script type="text/javascript">
 	//we set LAMS_AUTHORING_SUCCESS_FLAG to true in AuthoringAction.update() method 
 	if(<c:choose><c:when test="${LAMS_AUTHORING_SUCCESS_FLAG == true}">true</c:when><c:otherwise>false</c:otherwise></c:choose>){
-
+		
 		//if defineLater is true close current window
 		if (("${defineLater}" == "true") || ("${defineLater}" == "yes")) {
 			closeWindow("defineLater");
@@ -99,10 +99,10 @@
 			var clearSessionUrl = "<c:url value='${clearSessionActionUrl}?action=" + nextAction + "&mode=${accessMode}&defineLater=${defineLater}&customiseSessionID=${customiseSessionID}&signature=${toolSignature}&toolContentID=${toolContentID}'/>";
 			doAjaxCall(clearSessionUrl);
 		} else {
-			if (window.parent.opener == null){
-				doAjaxCall(notifyCloseURL);
-			} else if ('${param.noopener}' == 'true' || notifyCloseURL.indexOf('noopener=true') >= 0){
+			if ('${param.noopener}' == 'true' || notifyCloseURL.indexOf('noopener=true') >= 0) {
 				window.location.href = notifyCloseURL;
+			} else if (window.parent.opener == null){
+				doAjaxCall(notifyCloseURL);
 			} else {
 				window.parent.opener.location.href = notifyCloseURL;
 			}
@@ -114,7 +114,7 @@
     	if(userAgent.indexOf('MSIE') != -1)
         	window.opener = "authoring"
     	window.close();
-    } 				
+    }  				
 </script>	
 <p id="saveCancelButtons" >
 		<html:link href="javascript:;" property="cancel" onclick="javascript:doCancel()" styleClass="button right-buttons space-left">
Index: lams_tool_sbmt/web/WEB-INF/tags/AuthoringButton.tag
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_sbmt/web/WEB-INF/tags/AuthoringButton.tag,v
diff -u -r1.17 -r1.18
--- lams_tool_sbmt/web/WEB-INF/tags/AuthoringButton.tag	11 May 2012 09:29:51 -0000	1.17
+++ lams_tool_sbmt/web/WEB-INF/tags/AuthoringButton.tag	3 Jul 2012 12:56:36 -0000	1.18
@@ -72,7 +72,7 @@
 <script type="text/javascript">
 	//we set LAMS_AUTHORING_SUCCESS_FLAG to true in AuthoringAction.update() method 
 	if(<c:choose><c:when test="${LAMS_AUTHORING_SUCCESS_FLAG == true}">true</c:when><c:otherwise>false</c:otherwise></c:choose>){
-
+		
 		//if defineLater is true close current window
 		if (("${defineLater}" == "true") || ("${defineLater}" == "yes")) {
 			closeWindow("defineLater");
@@ -89,7 +89,7 @@
     	if(confirm("<fmt:message key='${cancelConfirmMsgKey}'/>")){
     		closeWindow("cancel");
     	}
-    }  
+    }
     function closeWindow(nextAction) {
 		var notifyCloseURL = "${notifyCloseURL}";
 		if (notifyCloseURL == ""){
@@ -99,10 +99,10 @@
 			var clearSessionUrl = "<c:url value='${clearSessionActionUrl}?action=" + nextAction + "&mode=${accessMode}&defineLater=${defineLater}&customiseSessionID=${customiseSessionID}&signature=${toolSignature}&toolContentID=${toolContentID}'/>";
 			doAjaxCall(clearSessionUrl);
 		} else {
-			if (window.parent.opener == null){
-				doAjaxCall(notifyCloseURL);
-			} else if ('${param.noopener}' == 'true' || notifyCloseURL.indexOf('noopener=true') >= 0){
+			if ('${param.noopener}' == 'true' || notifyCloseURL.indexOf('noopener=true') >= 0) {
 				window.location.href = notifyCloseURL;
+			} else if (window.parent.opener == null){
+				doAjaxCall(notifyCloseURL);
 			} else {
 				window.parent.opener.location.href = notifyCloseURL;
 			}
@@ -114,7 +114,7 @@
     	if(userAgent.indexOf('MSIE') != -1)
         	window.opener = "authoring"
     	window.close();
-    }
+    }  				
 </script>	
 <p id="saveCancelButtons" >
 		<html:link href="javascript:;" property="cancel" onclick="javascript:doCancel()" styleClass="button right-buttons space-left">
Index: lams_tool_scribe/web/WEB-INF/tags/AuthoringButton.tag
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_scribe/web/WEB-INF/tags/AuthoringButton.tag,v
diff -u -r1.14 -r1.15
--- lams_tool_scribe/web/WEB-INF/tags/AuthoringButton.tag	11 May 2012 09:30:12 -0000	1.14
+++ lams_tool_scribe/web/WEB-INF/tags/AuthoringButton.tag	3 Jul 2012 12:57:51 -0000	1.15
@@ -72,7 +72,7 @@
 <script type="text/javascript">
 	//we set LAMS_AUTHORING_SUCCESS_FLAG to true in AuthoringAction.update() method 
 	if(<c:choose><c:when test="${LAMS_AUTHORING_SUCCESS_FLAG == true}">true</c:when><c:otherwise>false</c:otherwise></c:choose>){
-
+		
 		//if defineLater is true close current window
 		if (("${defineLater}" == "true") || ("${defineLater}" == "yes")) {
 			closeWindow("defineLater");
@@ -99,10 +99,10 @@
 			var clearSessionUrl = "<c:url value='${clearSessionActionUrl}?action=" + nextAction + "&mode=${accessMode}&defineLater=${defineLater}&customiseSessionID=${customiseSessionID}&signature=${toolSignature}&toolContentID=${toolContentID}'/>";
 			doAjaxCall(clearSessionUrl);
 		} else {
-			if (window.parent.opener == null){
-				doAjaxCall(notifyCloseURL);
-			} else if ('${param.noopener}' == 'true' || notifyCloseURL.indexOf('noopener=true') >= 0){
+			if ('${param.noopener}' == 'true' || notifyCloseURL.indexOf('noopener=true') >= 0) {
 				window.location.href = notifyCloseURL;
+			} else if (window.parent.opener == null){
+				doAjaxCall(notifyCloseURL);
 			} else {
 				window.parent.opener.location.href = notifyCloseURL;
 			}
@@ -114,7 +114,7 @@
     	if(userAgent.indexOf('MSIE') != -1)
         	window.opener = "authoring"
     	window.close();
-    } 				
+    }  				
 </script>	
 <p id="saveCancelButtons" >
 		<html:link href="javascript:;" property="cancel" onclick="javascript:doCancel()" styleClass="button right-buttons space-left">
Index: lams_tool_spreadsheet/web/WEB-INF/tags/AuthoringButton.tag
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_spreadsheet/web/WEB-INF/tags/AuthoringButton.tag,v
diff -u -r1.7 -r1.8
--- lams_tool_spreadsheet/web/WEB-INF/tags/AuthoringButton.tag	11 May 2012 09:30:04 -0000	1.7
+++ lams_tool_spreadsheet/web/WEB-INF/tags/AuthoringButton.tag	3 Jul 2012 12:56:56 -0000	1.8
@@ -72,7 +72,7 @@
 <script type="text/javascript">
 	//we set LAMS_AUTHORING_SUCCESS_FLAG to true in AuthoringAction.update() method 
 	if(<c:choose><c:when test="${LAMS_AUTHORING_SUCCESS_FLAG == true}">true</c:when><c:otherwise>false</c:otherwise></c:choose>){
-
+		
 		//if defineLater is true close current window
 		if (("${defineLater}" == "true") || ("${defineLater}" == "yes")) {
 			closeWindow("defineLater");
@@ -99,10 +99,10 @@
 			var clearSessionUrl = "<c:url value='${clearSessionActionUrl}?action=" + nextAction + "&mode=${accessMode}&defineLater=${defineLater}&customiseSessionID=${customiseSessionID}&signature=${toolSignature}&toolContentID=${toolContentID}'/>";
 			doAjaxCall(clearSessionUrl);
 		} else {
-			if (window.parent.opener == null){
-				doAjaxCall(notifyCloseURL);
-			} else if ('${param.noopener}' == 'true' || notifyCloseURL.indexOf('noopener=true') >= 0){
+			if ('${param.noopener}' == 'true' || notifyCloseURL.indexOf('noopener=true') >= 0) {
 				window.location.href = notifyCloseURL;
+			} else if (window.parent.opener == null){
+				doAjaxCall(notifyCloseURL);
 			} else {
 				window.parent.opener.location.href = notifyCloseURL;
 			}
@@ -114,7 +114,7 @@
     	if(userAgent.indexOf('MSIE') != -1)
         	window.opener = "authoring"
     	window.close();
-    }
+    }  				
 </script>	
 <p id="saveCancelButtons" >
 		<html:link href="javascript:;" property="cancel" onclick="javascript:doCancel()" styleClass="button right-buttons space-left">
Index: lams_tool_survey/web/WEB-INF/tags/AuthoringButton.tag
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_survey/web/WEB-INF/tags/AuthoringButton.tag,v
diff -u -r1.13 -r1.14
--- lams_tool_survey/web/WEB-INF/tags/AuthoringButton.tag	11 May 2012 09:29:42 -0000	1.13
+++ lams_tool_survey/web/WEB-INF/tags/AuthoringButton.tag	3 Jul 2012 12:57:57 -0000	1.14
@@ -72,7 +72,7 @@
 <script type="text/javascript">
 	//we set LAMS_AUTHORING_SUCCESS_FLAG to true in AuthoringAction.update() method 
 	if(<c:choose><c:when test="${LAMS_AUTHORING_SUCCESS_FLAG == true}">true</c:when><c:otherwise>false</c:otherwise></c:choose>){
-
+		
 		//if defineLater is true close current window
 		if (("${defineLater}" == "true") || ("${defineLater}" == "yes")) {
 			closeWindow("defineLater");
@@ -99,10 +99,10 @@
 			var clearSessionUrl = "<c:url value='${clearSessionActionUrl}?action=" + nextAction + "&mode=${accessMode}&defineLater=${defineLater}&customiseSessionID=${customiseSessionID}&signature=${toolSignature}&toolContentID=${toolContentID}'/>";
 			doAjaxCall(clearSessionUrl);
 		} else {
-			if (window.parent.opener == null){
-				doAjaxCall(notifyCloseURL);
-			} else if ('${param.noopener}' == 'true' || notifyCloseURL.indexOf('noopener=true') >= 0){
+			if ('${param.noopener}' == 'true' || notifyCloseURL.indexOf('noopener=true') >= 0) {
 				window.location.href = notifyCloseURL;
+			} else if (window.parent.opener == null){
+				doAjaxCall(notifyCloseURL);
 			} else {
 				window.parent.opener.location.href = notifyCloseURL;
 			}
Index: lams_tool_task/web/WEB-INF/tags/AuthoringButton.tag
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_task/web/WEB-INF/tags/AuthoringButton.tag,v
diff -u -r1.9 -r1.10
--- lams_tool_task/web/WEB-INF/tags/AuthoringButton.tag	11 May 2012 09:29:58 -0000	1.9
+++ lams_tool_task/web/WEB-INF/tags/AuthoringButton.tag	3 Jul 2012 12:56:30 -0000	1.10
@@ -72,7 +72,7 @@
 <script type="text/javascript">
 	//we set LAMS_AUTHORING_SUCCESS_FLAG to true in AuthoringAction.update() method 
 	if(<c:choose><c:when test="${LAMS_AUTHORING_SUCCESS_FLAG == true}">true</c:when><c:otherwise>false</c:otherwise></c:choose>){
-
+		
 		//if defineLater is true close current window
 		if (("${defineLater}" == "true") || ("${defineLater}" == "yes")) {
 			closeWindow("defineLater");
@@ -89,7 +89,7 @@
     	if(confirm("<fmt:message key='${cancelConfirmMsgKey}'/>")){
     		closeWindow("cancel");
     	}
-    } 
+    }
     function closeWindow(nextAction) {
 		var notifyCloseURL = "${notifyCloseURL}";
 		if (notifyCloseURL == ""){
@@ -99,10 +99,10 @@
 			var clearSessionUrl = "<c:url value='${clearSessionActionUrl}?action=" + nextAction + "&mode=${accessMode}&defineLater=${defineLater}&customiseSessionID=${customiseSessionID}&signature=${toolSignature}&toolContentID=${toolContentID}'/>";
 			doAjaxCall(clearSessionUrl);
 		} else {
-			if (window.parent.opener == null){
-				doAjaxCall(notifyCloseURL);
-			} else if ('${param.noopener}' == 'true' || notifyCloseURL.indexOf('noopener=true') >= 0){
+			if ('${param.noopener}' == 'true' || notifyCloseURL.indexOf('noopener=true') >= 0) {
 				window.location.href = notifyCloseURL;
+			} else if (window.parent.opener == null){
+				doAjaxCall(notifyCloseURL);
 			} else {
 				window.parent.opener.location.href = notifyCloseURL;
 			}
@@ -114,7 +114,7 @@
     	if(userAgent.indexOf('MSIE') != -1)
         	window.opener = "authoring"
     	window.close();
-    }
+    }  				
 </script>	
 <p id="saveCancelButtons" >
 		<html:link href="javascript:;" property="cancel" onclick="javascript:doCancel()" styleClass="button right-buttons space-left">
Index: lams_tool_videorecorder/web/WEB-INF/tags/AuthoringButton.tag
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_videorecorder/web/WEB-INF/tags/AuthoringButton.tag,v
diff -u -r1.7 -r1.8
--- lams_tool_videorecorder/web/WEB-INF/tags/AuthoringButton.tag	11 May 2012 09:29:53 -0000	1.7
+++ lams_tool_videorecorder/web/WEB-INF/tags/AuthoringButton.tag	3 Jul 2012 12:56:25 -0000	1.8
@@ -72,7 +72,7 @@
 <script type="text/javascript">
 	//we set LAMS_AUTHORING_SUCCESS_FLAG to true in AuthoringAction.update() method 
 	if(<c:choose><c:when test="${LAMS_AUTHORING_SUCCESS_FLAG == true}">true</c:when><c:otherwise>false</c:otherwise></c:choose>){
-
+		
 		//if defineLater is true close current window
 		if (("${defineLater}" == "true") || ("${defineLater}" == "yes")) {
 			closeWindow("defineLater");
@@ -99,10 +99,10 @@
 			var clearSessionUrl = "<c:url value='${clearSessionActionUrl}?action=" + nextAction + "&mode=${accessMode}&defineLater=${defineLater}&customiseSessionID=${customiseSessionID}&signature=${toolSignature}&toolContentID=${toolContentID}'/>";
 			doAjaxCall(clearSessionUrl);
 		} else {
-			if (window.parent.opener == null){
-				doAjaxCall(notifyCloseURL);
-			} else if ('${param.noopener}' == 'true' || notifyCloseURL.indexOf('noopener=true') >= 0){
+			if ('${param.noopener}' == 'true' || notifyCloseURL.indexOf('noopener=true') >= 0) {
 				window.location.href = notifyCloseURL;
+			} else if (window.parent.opener == null){
+				doAjaxCall(notifyCloseURL);
 			} else {
 				window.parent.opener.location.href = notifyCloseURL;
 			}
@@ -114,7 +114,7 @@
     	if(userAgent.indexOf('MSIE') != -1)
         	window.opener = "authoring"
     	window.close();
-    } 				
+    }  				
 </script>	
 <p id="saveCancelButtons" >
 		<html:link href="javascript:;" property="cancel" onclick="javascript:doCancel()" styleClass="button right-buttons space-left">
Index: lams_tool_vote/web/WEB-INF/tags/AuthoringButton.tag
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_vote/web/WEB-INF/tags/AuthoringButton.tag,v
diff -u -r1.16 -r1.17
--- lams_tool_vote/web/WEB-INF/tags/AuthoringButton.tag	11 May 2012 09:30:06 -0000	1.16
+++ lams_tool_vote/web/WEB-INF/tags/AuthoringButton.tag	3 Jul 2012 12:56:53 -0000	1.17
@@ -72,7 +72,7 @@
 <script type="text/javascript">
 	//we set LAMS_AUTHORING_SUCCESS_FLAG to true in AuthoringAction.update() method 
 	if(<c:choose><c:when test="${LAMS_AUTHORING_SUCCESS_FLAG == true}">true</c:when><c:otherwise>false</c:otherwise></c:choose>){
-
+		
 		//if defineLater is true close current window
 		if (("${defineLater}" == "true") || ("${defineLater}" == "yes")) {
 			closeWindow("defineLater");
@@ -99,10 +99,10 @@
 			var clearSessionUrl = "<c:url value='${clearSessionActionUrl}?action=" + nextAction + "&mode=${accessMode}&defineLater=${defineLater}&customiseSessionID=${customiseSessionID}&signature=${toolSignature}&toolContentID=${toolContentID}'/>";
 			doAjaxCall(clearSessionUrl);
 		} else {
-			if (window.parent.opener == null){
-				doAjaxCall(notifyCloseURL);
-			} else if ('${param.noopener}' == 'true' || notifyCloseURL.indexOf('noopener=true') >= 0){
+			if ('${param.noopener}' == 'true' || notifyCloseURL.indexOf('noopener=true') >= 0) {
 				window.location.href = notifyCloseURL;
+			} else if (window.parent.opener == null){
+				doAjaxCall(notifyCloseURL);
 			} else {
 				window.parent.opener.location.href = notifyCloseURL;
 			}
@@ -114,7 +114,7 @@
     	if(userAgent.indexOf('MSIE') != -1)
         	window.opener = "authoring"
     	window.close();
-    } 				
+    }  				
 </script>	
 <p id="saveCancelButtons" >
 		<html:link href="javascript:;" property="cancel" onclick="javascript:doCancel()" styleClass="button right-buttons space-left">
Index: lams_tool_wiki/web/WEB-INF/tags/AuthoringButton.tag
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_wiki/web/WEB-INF/tags/AuthoringButton.tag,v
diff -u -r1.7 -r1.8
--- lams_tool_wiki/web/WEB-INF/tags/AuthoringButton.tag	11 May 2012 09:30:09 -0000	1.7
+++ lams_tool_wiki/web/WEB-INF/tags/AuthoringButton.tag	3 Jul 2012 12:57:02 -0000	1.8
@@ -72,7 +72,7 @@
 <script type="text/javascript">
 	//we set LAMS_AUTHORING_SUCCESS_FLAG to true in AuthoringAction.update() method 
 	if(<c:choose><c:when test="${LAMS_AUTHORING_SUCCESS_FLAG == true}">true</c:when><c:otherwise>false</c:otherwise></c:choose>){
-
+		
 		//if defineLater is true close current window
 		if (("${defineLater}" == "true") || ("${defineLater}" == "yes")) {
 			closeWindow("defineLater");
@@ -89,7 +89,7 @@
     	if(confirm("<fmt:message key='${cancelConfirmMsgKey}'/>")){
     		closeWindow("cancel");
     	}
-    } 
+    }
     function closeWindow(nextAction) {
 		var notifyCloseURL = "${notifyCloseURL}";
 		if (notifyCloseURL == ""){
@@ -99,10 +99,10 @@
 			var clearSessionUrl = "<c:url value='${clearSessionActionUrl}?action=" + nextAction + "&mode=${accessMode}&defineLater=${defineLater}&customiseSessionID=${customiseSessionID}&signature=${toolSignature}&toolContentID=${toolContentID}'/>";
 			doAjaxCall(clearSessionUrl);
 		} else {
-			if (window.parent.opener == null){
-				doAjaxCall(notifyCloseURL);
-			} else if ('${param.noopener}' == 'true' || notifyCloseURL.indexOf('noopener=true') >= 0){
+			if ('${param.noopener}' == 'true' || notifyCloseURL.indexOf('noopener=true') >= 0) {
 				window.location.href = notifyCloseURL;
+			} else if (window.parent.opener == null){
+				doAjaxCall(notifyCloseURL);
 			} else {
 				window.parent.opener.location.href = notifyCloseURL;
 			}
@@ -114,7 +114,7 @@
     	if(userAgent.indexOf('MSIE') != -1)
         	window.opener = "authoring"
     	window.close();
-    }
+    }  				
 </script>	
 <p id="saveCancelButtons" >
 		<html:link href="javascript:;" property="cancel" onclick="javascript:doCancel()" styleClass="button right-buttons space-left">
Index: lams_tool_wookie/web/WEB-INF/tags/AuthoringButton.tag
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_wookie/web/WEB-INF/tags/AuthoringButton.tag,v
diff -u -r1.5 -r1.6
--- lams_tool_wookie/web/WEB-INF/tags/AuthoringButton.tag	11 May 2012 09:29:55 -0000	1.5
+++ lams_tool_wookie/web/WEB-INF/tags/AuthoringButton.tag	3 Jul 2012 12:57:22 -0000	1.6
@@ -72,7 +72,7 @@
 <script type="text/javascript">
 	//we set LAMS_AUTHORING_SUCCESS_FLAG to true in AuthoringAction.update() method 
 	if(<c:choose><c:when test="${LAMS_AUTHORING_SUCCESS_FLAG == true}">true</c:when><c:otherwise>false</c:otherwise></c:choose>){
-
+		
 		//if defineLater is true close current window
 		if (("${defineLater}" == "true") || ("${defineLater}" == "yes")) {
 			closeWindow("defineLater");
@@ -99,10 +99,10 @@
 			var clearSessionUrl = "<c:url value='${clearSessionActionUrl}?action=" + nextAction + "&mode=${accessMode}&defineLater=${defineLater}&customiseSessionID=${customiseSessionID}&signature=${toolSignature}&toolContentID=${toolContentID}'/>";
 			doAjaxCall(clearSessionUrl);
 		} else {
-			if (window.parent.opener == null){
-				doAjaxCall(notifyCloseURL);
-			} else if ('${param.noopener}' == 'true' || notifyCloseURL.indexOf('noopener=true') >= 0){
+			if ('${param.noopener}' == 'true' || notifyCloseURL.indexOf('noopener=true') >= 0) {
 				window.location.href = notifyCloseURL;
+			} else if (window.parent.opener == null){
+				doAjaxCall(notifyCloseURL);
 			} else {
 				window.parent.opener.location.href = notifyCloseURL;
 			}
@@ -114,7 +114,7 @@
     	if(userAgent.indexOf('MSIE') != -1)
         	window.opener = "authoring"
     	window.close();
-    }			
+    }  				
 </script>	
 <p id="saveCancelButtons" >
 		<html:link href="javascript:;" property="cancel" onclick="javascript:doCancel()" styleClass="button right-buttons space-left">
Index: lams_www/web/WEB-INF/tags/AuthoringButton.tag
===================================================================
RCS file: /usr/local/cvsroot/lams_www/web/WEB-INF/tags/AuthoringButton.tag,v
diff -u -r1.11 -r1.12
--- lams_www/web/WEB-INF/tags/AuthoringButton.tag	11 May 2012 09:29:27 -0000	1.11
+++ lams_www/web/WEB-INF/tags/AuthoringButton.tag	3 Jul 2012 12:57:11 -0000	1.12
@@ -62,9 +62,12 @@
 <c:if test="${empty accessMode}">
 	<c:set var="accessMode" value="author" scope="request"/>
 </c:if>
+<c:if test="${not empty param.notifyCloseURL}">
+	<c:set var="notifyCloseURL" value="${param.notifyCloseURL}" scope="session"/>
+</c:if>
 
 <script type="text/javascript" src="<lams:LAMSURL />includes/javascript/common.js"></script>
-	
+
 <!-- begin tab content -->
 <script type="text/javascript">
 	//we set LAMS_AUTHORING_SUCCESS_FLAG to true in AuthoringAction.update() method 
@@ -85,7 +88,7 @@
     function doCancel() {
     	if(confirm("<fmt:message key='${cancelConfirmMsgKey}'/>")){
     		closeWindow("cancel");
-		}
+    	}
     }
     function closeWindow(nextAction) {
 		var notifyCloseURL = "${notifyCloseURL}";
@@ -96,10 +99,10 @@
 			var clearSessionUrl = "<c:url value='${clearSessionActionUrl}?action=" + nextAction + "&mode=${accessMode}&defineLater=${defineLater}&customiseSessionID=${customiseSessionID}&signature=${toolSignature}&toolContentID=${toolContentID}'/>";
 			doAjaxCall(clearSessionUrl);
 		} else {
-			if (window.parent.opener == null){
-				doAjaxCall(notifyCloseURL);
-			} else if ('${param.noopener}' == 'true' || notifyCloseURL.indexOf('noopener=true') >= 0){
+			if ('${param.noopener}' == 'true' || notifyCloseURL.indexOf('noopener=true') >= 0) {
 				window.location.href = notifyCloseURL;
+			} else if (window.parent.opener == null){
+				doAjaxCall(notifyCloseURL);
 			} else {
 				window.parent.opener.location.href = notifyCloseURL;
 			}
@@ -111,7 +114,7 @@
     	if(userAgent.indexOf('MSIE') != -1)
         	window.opener = "authoring"
     	window.close();
-    }
+    }  				
 </script>	
 <p id="saveCancelButtons" >
 		<html:link href="javascript:;" property="cancel" onclick="javascript:doCancel()" styleClass="button right-buttons space-left">