Index: lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/web/action/LearningAction.java
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/web/action/LearningAction.java,v
diff -u -r1.13 -r1.14
--- lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/web/action/LearningAction.java	30 Jun 2006 02:16:53 -0000	1.13
+++ lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/web/action/LearningAction.java	3 Jul 2006 02:16:12 -0000	1.14
@@ -93,6 +93,7 @@
 	
 	private ActionForward finish(ActionMapping mapping, ActionForm form, HttpServletRequest request, HttpServletResponse response) {
 		//auto run mode, when use finish the only one resource item, mark it as complete then finish this activity as well.
+		String mode = request.getParameter(AttributeNames.ATTR_MODE);
 		String resourceItemUid = request.getParameter(ResourceConstants.PARAM_RESOURCE_ITEM_UID);
 		String runOffline = request.getParameter(ResourceConstants.PARAM_RUN_OFFLINE);
 		if(resourceItemUid != null){
@@ -126,12 +127,15 @@
 		} catch (ResourceApplicationException e) {
 			log.error("Failed get next activity url:" + e.getMessage());
 		}
+		
+		request.setAttribute(AttributeNames.ATTR_MODE,mode);
 		return mapping.findForward("finish");
 	}
 
 	private ActionForward complete(ActionMapping mapping, ActionForm form, HttpServletRequest request, HttpServletResponse response) {
-		
+		String mode = request.getParameter(AttributeNames.ATTR_MODE);
 		doComplete(request);
+		request.setAttribute(AttributeNames.ATTR_MODE,mode);
 		return  mapping.findForward(ResourceConstants.SUCCESS);
 	}
 
@@ -147,6 +151,7 @@
 	private ActionForward saveOrUpdateItem(ActionMapping mapping, ActionForm form, HttpServletRequest request,
 			HttpServletResponse response) {
 		
+		String mode = request.getParameter(AttributeNames.ATTR_MODE);
 		ResourceItemForm itemForm = (ResourceItemForm)form;
 		ActionErrors errors = validateResourceItem(itemForm);
 		
@@ -208,6 +213,7 @@
 		
 		//URL or file upload
 		request.setAttribute("addType",new Short(type));
+		request.setAttribute(AttributeNames.ATTR_MODE,mode);
 		return  mapping.findForward(ResourceConstants.SUCCESS);
 	}
 	/**
Index: lams_tool_larsrc/web/pages/itemreview/instructionsnav.jsp
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_larsrc/web/pages/itemreview/instructionsnav.jsp,v
diff -u -r1.10 -r1.11
--- lams_tool_larsrc/web/pages/itemreview/instructionsnav.jsp	3 Jul 2006 00:58:07 -0000	1.10
+++ lams_tool_larsrc/web/pages/itemreview/instructionsnav.jsp	3 Jul 2006 02:16:12 -0000	1.11
@@ -6,18 +6,23 @@
 		<title>${instructions.title}</title>
 
 		<%@ include file="/common/header.jsp"%>
+		
+		<%-- param has higher level for request attribute --%>
+		<c:if test="${not empty param.mode}">
+			<c:set var="mode" value="${param.mode}"/>
+		</c:if>		
 		<script language="JavaScript" type="text/JavaScript">
 		<!--
 				function finishIns(){
 				//learner and author(preview mode) will mark the finish
-					if(${param.mode == "learner"} || ${param.mode == "author"}){
+					if(${mode == "learner"} || ${mode == "author"}){
 					   var reqIDVar = new Date();
 					   //if auto run mode, the opener will be null
 					   if(window.parent.opener != null) 
-						    window.parent.opener.location.href="<c:url value="/learning/completeItem.do"/>?itemUid=${param.itemUid}&reqID="+reqIDVar.getTime();
+						    window.parent.opener.location.href="<c:url value="/learning/completeItem.do"/>?mode=${mode}&itemUid=${param.itemUid}&reqID="+reqIDVar.getTime();
 					   else{
 					  		//set complete flag and finish this activity as well.
-					        window.parent.location.href='<c:url value="/learning/finish.do?toolSessionID=${param.toolSessionID}&itemUid=${param.itemUid}"/>';
+					        window.parent.location.href='<c:url value="/learning/finish.do?mode=${mode}&toolSessionID=${param.toolSessionID}&itemUid=${param.itemUid}"/>';
 					   }
 					}
 				   if(window.parent.opener != null) {
@@ -26,7 +31,7 @@
 					}
 				}
 				function nextIns(currIns){
-					document.location.href="<c:url value='/nextInstruction.do'/>?insIdx=" + currIns + "&itemUid=${param.itemUid}&itemIndex=${param.itemIndex}&mode=${param.mode}";
+					document.location.href="<c:url value='/nextInstruction.do'/>?mode=${mode}&insIdx=" + currIns + "&itemUid=${param.itemUid}&itemIndex=${param.itemIndex}";
 				}
 		//-->
 		</script>
Index: lams_tool_larsrc/web/pages/learning/addfile.jsp
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_larsrc/web/pages/learning/addfile.jsp,v
diff -u -r1.4 -r1.5
--- lams_tool_larsrc/web/pages/learning/addfile.jsp	28 Jun 2006 07:09:27 -0000	1.4
+++ lams_tool_larsrc/web/pages/learning/addfile.jsp	3 Jul 2006 02:16:12 -0000	1.5
@@ -47,6 +47,7 @@
 					<%@ include file="/common/messages.jsp"%>
 					<html:form action="/learning/saveOrUpdateItem" method="post" styleId="resourceItemForm" enctype="multipart/form-data">
 						<input type="hidden" name="itemType" id="itemType" value="2"/>
+						<input type="hidden" name="mode" value="${param.mode}"/>
 						<table class="innerforms">
 							<tr>
 								<td valign="top">
Index: lams_tool_larsrc/web/pages/learning/addurl.jsp
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_larsrc/web/pages/learning/addurl.jsp,v
diff -u -r1.6 -r1.7
--- lams_tool_larsrc/web/pages/learning/addurl.jsp	30 Jun 2006 04:16:51 -0000	1.6
+++ lams_tool_larsrc/web/pages/learning/addurl.jsp	3 Jul 2006 02:16:12 -0000	1.7
@@ -47,6 +47,7 @@
 					<%@ include file="/common/messages.jsp"%>
 					<html:form action="/learning/saveOrUpdateItem" method="post" styleId="resourceItemForm">
 						<input type="hidden" name="itemType" id="itemType" value="1"/>
+						<input type="hidden" name="mode" value="${param.mode}"/>
 						<table class="innerforms">
 							<tr>
 								<td valign="top">
Index: lams_tool_larsrc/web/pages/learning/learning.jsp
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_larsrc/web/pages/learning/learning.jsp,v
diff -u -r1.21 -r1.22
--- lams_tool_larsrc/web/pages/learning/learning.jsp	30 Jun 2006 04:09:54 -0000	1.21
+++ lams_tool_larsrc/web/pages/learning/learning.jsp	3 Jul 2006 02:16:12 -0000	1.22
@@ -7,35 +7,40 @@
 	<title><fmt:message key="label.learning.title" /></title>
 	<%@ include file="/common/header.jsp"%>
 
+	<%-- param has higher level for request attribute --%>
+	<c:if test="${not empty param.mode}">
+		<c:set var="mode" value="${param.mode}"/>
+	</c:if>
+	
 	<script type="text/javascript">
 	<!--
 		function gotoURL(){
  		    var reqIDVar = new Date();
-			var gurl = "<c:url value="/pages/learning/addurl.jsp"/>?&reqID="+reqIDVar.getTime();
+			var gurl = "<c:url value="/pages/learning/addurl.jsp"/>?mode=${mode}&reqID="+reqIDVar.getTime();
 	      	showMessage(gurl);
 	      	return false;
 		}
 		function gotoFile(){
  		    var reqIDVar = new Date();
- 		    var gurl = "<c:url value="/pages/learning/addfile.jsp"/>?&reqID="+reqIDVar.getTime();
+ 		    var gurl = "<c:url value="/pages/learning/addfile.jsp"/>?mode=${mode}&reqID="+reqIDVar.getTime();
 	      	showMessage(gurl);
 	      	return false;
 		}
 		function checkNew(){
  		    var reqIDVar = new Date();
-			document.location.href = "<c:url value="/learning/start.do"/>?mode=${param.mode}&toolSessionID=${toolSessionID}&reqID="+reqIDVar.getTime();
+			document.location.href = "<c:url value="/learning/start.do"/>?mode=${mode}&toolSessionID=${toolSessionID}&reqID="+reqIDVar.getTime();
  		    return false;
 		}
 		function viewItem(itemUid){
-			var myUrl = "<c:url value="/reviewItem.do"/>?mode=${param.mode}&toolSessionID=${toolSessionID}&itemUid=" + itemUid;
+			var myUrl = "<c:url value="/reviewItem.do"/>?mode=${mode}&toolSessionID=${toolSessionID}&itemUid=" + itemUid;
 			launchPopup(myUrl,"LearnerView");
 		}
 		function completeItem(itemUid){
-			document.location.href = "<c:url value="/learning/completeItem.do"/>?itemUid=" + itemUid;
+			document.location.href = "<c:url value="/learning/completeItem.do"/>?mode=${mode}&itemUid=" + itemUid;
 			return false;
 		}
 		function finishSession(){
-			document.location.href ='<c:url value="/learning/finish.do?toolSessionID=${toolSessionID}"/>';
+			document.location.href ='<c:url value="/learning/finish.do?mode=${mode}&toolSessionID=${toolSessionID}"/>';
 			return false;
 		}
 		function showMessage(url) {
@@ -94,7 +99,7 @@
 				</td>
 				
 				<td>
-					<c:if test="${param.mode != 'teacher'}">
+					<c:if test="${mode != 'teacher'}">
 						<a href="#" class="button" onclick="return completeItem(${item.uid})"><fmt:message key="label.completed" /></a>
 					</c:if>	
 					<a href="javascript:;" class="button" onclick="viewItem(${item.uid})" ><fmt:message key="label.view" /></a>
@@ -105,7 +110,7 @@
 			<td colspan="3" align="right">
 				<a href="#" class="button" onclick="return checkNew()"><fmt:message key="label.check.for.new" /></a>
 				&nbsp;
-				<c:if test="${param.mode != 'teacher'}">
+				<c:if test="${mode != 'teacher'}">
 					<a href="#" class="button" onclick="return finishSession()">
 						<fmt:message key="label.finished" />
 					</a>
@@ -120,7 +125,7 @@
 			</tr>
 		</c:if>
 	</table>
-	<c:if test="${param.mode != 'teacher'}">
+	<c:if test="${mode != 'teacher'}">
 		<c:if test="${resource.allowAddFiles || resource.allowAddUrls}">
 			<table border="0" align="center" width="100%">
 				<tr>
@@ -153,21 +158,21 @@
 			<c:when test="${resource.allowAddFiles && resource.allowAddUrls}">
 				<script type="text/javascript">
 					<!--
-						showMessage("<c:url value='/pages/learning/addurl.jsp'/>");
+						showMessage("<c:url value='/pages/learning/addurl.jsp'/>?mode=${mode}");
 					-->
 				</script>
 			</c:when>
 			<c:when test="${resource.allowAddFiles && !resource.allowAddUrls}">
 				<script type="text/javascript">
 					<!--
-						showMessage("<c:url value='/pages/learning/addfile.jsp'/>");
+						showMessage("<c:url value='/pages/learning/addfile.jsp'/>?mode=${mode}");
 					-->
 				</script>
 			</c:when>
 			<c:when test="${!resource.allowAddFiles && resource.allowAddUrls}">
 				<script type="text/javascript">
 					<!--
-						showMessage("<c:url value='/pages/learning/addurl.jsp'/>");
+						showMessage("<c:url value='/pages/learning/addurl.jsp'/>?mode=${mode}");
 					-->
 				</script>
 			</c:when>
Index: lams_tool_larsrc/web/pages/learning/success.jsp
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_larsrc/web/pages/learning/success.jsp,v
diff -u -r1.4 -r1.5
--- lams_tool_larsrc/web/pages/learning/success.jsp	4 May 2006 01:21:50 -0000	1.4
+++ lams_tool_larsrc/web/pages/learning/success.jsp	3 Jul 2006 02:16:12 -0000	1.5
@@ -8,14 +8,14 @@
 	<body>
 		<script type="text/javascript">
 			var reqIDVar = new Date();
-			window.parent.location.href  = "${tool}/pages/learning/learning.jsp?reqID="+reqIDVar.getTime();
+			window.parent.location.href  = "${tool}/pages/learning/learning.jsp?mode=${mode}&reqID="+reqIDVar.getTime();
 		</script>
 		<div style="align:center">
 			<c:if test="${addType == 1}">
-				Add URL success, <a href="<c:url value='/pages/learning/addurl.jsp'/>" target="newResourceFrame">click here to return</a>.
+				Add URL success, <a href="<c:url value='/pages/learning/addurl.jsp'/>?mode=${mode}" target="newResourceFrame">click here to return</a>.
 			</c:if>
 			<c:if test="${addType == 2}">
-				Add File success, <a href="<c:url value='/pages/learning/addfile.jsp'/>" target="newResourceFrame">click here to return</a>.
+				Add File success, <a href="<c:url value='/pages/learning/addfile.jsp'/>?mode=${mode}" target="newResourceFrame">click here to return</a>.
 			</c:if>
 		</div>
 	</body>