Index: lams_tool_larsrc/conf/xdoclet/struts-actions.xml
===================================================================
diff -u -rc25a2c783e51d05045d1fb61981c169357cd3e98 -r12fef57b1c4aea30f19676803c58600da0a5a167
--- lams_tool_larsrc/conf/xdoclet/struts-actions.xml	(.../struts-actions.xml)	(revision c25a2c783e51d05045d1fb61981c169357cd3e98)
+++ lams_tool_larsrc/conf/xdoclet/struts-actions.xml	(.../struts-actions.xml)	(revision 12fef57b1c4aea30f19676803c58600da0a5a167)
@@ -167,6 +167,25 @@
 			<forward name="finish" path="/pages/learning/finish.jsp" />
 		</action>
 		
+		
+		<action path="/learning/addfile"
+			type="org.lamsfoundation.lams.tool.rsrc.web.action.LearningAction"
+			parameter="addfile"
+			name="resourceItemForm" 
+			scope="request" 
+			validate="false">
+			<forward name="success" path="/pages/learning/addfile.jsp" />
+		</action>
+		
+		<action path="/learning/addurl"
+			type="org.lamsfoundation.lams.tool.rsrc.web.action.LearningAction"
+			parameter="addurl"
+			name="resourceItemForm" 
+			scope="request" 
+			validate="false">
+			<forward name="success" path="/pages/learning/addurl.jsp" />
+		</action>
+		
 		<action path="/learning/saveOrUpdateItem"
 			type="org.lamsfoundation.lams.tool.rsrc.web.action.LearningAction"
 			parameter="saveOrUpdateItem"
Index: lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/web/action/LearningAction.java
===================================================================
diff -u -r92e8b8d1454cfbd9ae8a20934a677340dac3b507 -r12fef57b1c4aea30f19676803c58600da0a5a167
--- lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/web/action/LearningAction.java	(.../LearningAction.java)	(revision 92e8b8d1454cfbd9ae8a20934a677340dac3b507)
+++ lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/web/action/LearningAction.java	(.../LearningAction.java)	(revision 12fef57b1c4aea30f19676803c58600da0a5a167)
@@ -56,6 +56,7 @@
 import org.lamsfoundation.lams.tool.rsrc.service.UploadResourceFileException;
 import org.lamsfoundation.lams.tool.rsrc.web.form.ResourceItemForm;
 import org.lamsfoundation.lams.usermanagement.dto.UserDTO;
+import org.lamsfoundation.lams.util.WebUtil;
 import org.lamsfoundation.lams.web.session.SessionManager;
 import org.lamsfoundation.lams.web.util.AttributeNames;
 import org.lamsfoundation.lams.web.util.SessionMap;
@@ -86,12 +87,32 @@
 		if(param.equals("finish")){
 			return finish(mapping, form, request, response);
 		}
+		if (param.equals("addfile")) {
+			return addItem(mapping, form, request, response);
+		}
+		if (param.equals("addurl")) {
+			return addItem(mapping, form, request, response);
+		}
         if (param.equals("saveOrUpdateItem")) {
         	return saveOrUpdateItem(mapping, form, request, response);
         }
 		return  mapping.findForward(ResourceConstants.ERROR);
 	}
 	/**
+	 * Initial page for add resource item (single file or URL).
+	 * @param mapping
+	 * @param form
+	 * @param request
+	 * @param response
+	 * @return
+	 */
+	private ActionForward addItem(ActionMapping mapping, ActionForm form, HttpServletRequest request, HttpServletResponse response) {
+		ResourceItemForm itemForm = (ResourceItemForm) form;
+		itemForm.setMode(WebUtil.readStrParam(request, AttributeNames.ATTR_MODE));
+		itemForm.setSessionMapID(WebUtil.readStrParam(request, ResourceConstants.ATTR_SESSION_MAP_ID));
+		return mapping.findForward(ResourceConstants.SUCCESS);
+	}
+	/**
 	 * Read resource data from database and put them into HttpSession. It will redirect to init.do directly after this
 	 * method run successfully. 
 	 *  
Index: lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/web/form/ResourceItemForm.java
===================================================================
diff -u -rc25a2c783e51d05045d1fb61981c169357cd3e98 -r12fef57b1c4aea30f19676803c58600da0a5a167
--- lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/web/form/ResourceItemForm.java	(.../ResourceItemForm.java)	(revision c25a2c783e51d05045d1fb61981c169357cd3e98)
+++ lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/web/form/ResourceItemForm.java	(.../ResourceItemForm.java)	(revision 12fef57b1c4aea30f19676803c58600da0a5a167)
@@ -36,6 +36,9 @@
 	private String itemIndex;
 	private String sessionMapID;
 	
+	//tool access mode;
+	private String mode;
+	
 	private String title;
 	private short itemType;
 	private String description;
@@ -120,4 +123,10 @@
 	public void setSessionMapID(String sessionMapID) {
 		this.sessionMapID = sessionMapID;
 	}
+	public String getMode() {
+		return mode;
+	}
+	public void setMode(String mode) {
+		this.mode = mode;
+	}
 }
Index: lams_tool_larsrc/web/pages/itemreview/mainframe.jsp
===================================================================
diff -u -rc25a2c783e51d05045d1fb61981c169357cd3e98 -r12fef57b1c4aea30f19676803c58600da0a5a167
--- lams_tool_larsrc/web/pages/itemreview/mainframe.jsp	(.../mainframe.jsp)	(revision c25a2c783e51d05045d1fb61981c169357cd3e98)
+++ lams_tool_larsrc/web/pages/itemreview/mainframe.jsp	(.../mainframe.jsp)	(revision 12fef57b1c4aea30f19676803c58600da0a5a167)
@@ -22,5 +22,5 @@
 <frameset rows="110,*" frameborder="no">
 	<frame src="<c:url value="/pages/itemreview/initnav.jsp"/>?mode=${mode}&itemIndex=${itemIndex}&itemUid=${itemUid}&toolSessionID=${toolSessionID}&sessionMapID=${sessionMapID}" 
 	name=headerFrame" marginheight="0" scrolling="NO">
-	<frame src="<c:url value='${resourceItemReviewUrl}'/>" name="resourceFrame" marginheight="0" scrolling="YES">
+	<frame src="<c:url value='${resourceItemReviewUrl}'/>?sessionMapID=${sessionMapID}" name="resourceFrame" marginheight="0" scrolling="YES">
 </frameset>
Index: lams_tool_larsrc/web/pages/learning/addfile.jsp
===================================================================
diff -u -r92e8b8d1454cfbd9ae8a20934a677340dac3b507 -r12fef57b1c4aea30f19676803c58600da0a5a167
--- lams_tool_larsrc/web/pages/learning/addfile.jsp	(.../addfile.jsp)	(revision 92e8b8d1454cfbd9ae8a20934a677340dac3b507)
+++ lams_tool_larsrc/web/pages/learning/addfile.jsp	(.../addfile.jsp)	(revision 12fef57b1c4aea30f19676803c58600da0a5a167)
@@ -5,9 +5,9 @@
 	</head>
 	<body>
 		<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}" />
-			<input type="hidden" name="sessionMapID" value="${param.sessionMapID}" />
+			<html:hidden property="itemType" styleId="itemType" value="2" />
+			<html:hidden property="mode"/>
+			<html:hidden property="sessionMapID"/>
 			<table width="100%" border="0" align="left" cellpadding="0" cellspacing="0">
 				<tr>
 					<td colspan="2">
Index: lams_tool_larsrc/web/pages/learning/addurl.jsp
===================================================================
diff -u -r92e8b8d1454cfbd9ae8a20934a677340dac3b507 -r12fef57b1c4aea30f19676803c58600da0a5a167
--- lams_tool_larsrc/web/pages/learning/addurl.jsp	(.../addurl.jsp)	(revision 92e8b8d1454cfbd9ae8a20934a677340dac3b507)
+++ lams_tool_larsrc/web/pages/learning/addurl.jsp	(.../addurl.jsp)	(revision 12fef57b1c4aea30f19676803c58600da0a5a167)
@@ -7,9 +7,9 @@
 	<body>
 		<!-- Basic Info Form-->
 		<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}" />
-			<input type="hidden" name="sessionMapID" value="${param.sessionMapID}" />
+			<html:hidden property="itemType" styleId="itemType" value="1" />
+			<html:hidden property="mode"/>
+			<html:hidden property="sessionMapID"/>
 			<table width="100%" border="0" align="left" cellpadding="0" cellspacing="0">
 				<tr>
 					<td  colspan="2">
Index: lams_tool_larsrc/web/pages/learning/learning.jsp
===================================================================
diff -u -r92e8b8d1454cfbd9ae8a20934a677340dac3b507 -r12fef57b1c4aea30f19676803c58600da0a5a167
--- lams_tool_larsrc/web/pages/learning/learning.jsp	(.../learning.jsp)	(revision 92e8b8d1454cfbd9ae8a20934a677340dac3b507)
+++ lams_tool_larsrc/web/pages/learning/learning.jsp	(.../learning.jsp)	(revision 12fef57b1c4aea30f19676803c58600da0a5a167)
@@ -24,13 +24,13 @@
 	<!--
 		function gotoURL(){
  		    var reqIDVar = new Date();
-			var gurl = "<c:url value="/pages/learning/addurl.jsp"/>?sessionMapID=${sessionMapID}&mode=${mode}&reqID="+reqIDVar.getTime();
+			var gurl = "<c:url value="/learning/addurl.do"/>?sessionMapID=${sessionMapID}&mode=${mode}&reqID="+reqIDVar.getTime();
 	      	showMessage(gurl);
 	      	return false;
 		}
 		function gotoFile(){
  		    var reqIDVar = new Date();
- 		    var gurl = "<c:url value="/pages/learning/addfile.jsp"/>?sessionMapID=${sessionMapID}&mode=${mode}&reqID="+reqIDVar.getTime();
+ 		    var gurl = "<c:url value="/learning/addfile.do"/>?sessionMapID=${sessionMapID}&mode=${mode}&reqID="+reqIDVar.getTime();
 	      	showMessage(gurl);
 	      	return false;
 		}
@@ -71,9 +71,9 @@
 		<!--closes header-->
 
 		<div id="content-learner">
-			<p>
+			<h2>
 				${resource.instructions}
-			</p>
+			</h2>
 
 			<%@ include file="/common/messages.jsp"%>
 			<table cellpadding="0" cellspacing="0" class="alternative-color">
@@ -175,21 +175,21 @@
 					<c:when test="${resource.allowAddFiles && resource.allowAddUrls}">
 						<script type="text/javascript">
 					<!--
-						showMessage("<c:url value='/pages/learning/addurl.jsp'/>?sessionMapID=${sessionMapID}&mode=${mode}");
+						showMessage("<c:url value='/learning/addurl.do'/>?sessionMapID=${sessionMapID}&mode=${mode}");
 					-->
 				</script>
 					</c:when>
 					<c:when test="${resource.allowAddFiles && !resource.allowAddUrls}">
 						<script type="text/javascript">
 					<!--
-						showMessage("<c:url value='/pages/learning/addfile.jsp'/>?sessionMapID=${sessionMapID}&mode=${mode}");
+						showMessage("<c:url value='/learning/addfile.do'/>?sessionMapID=${sessionMapID}&mode=${mode}");
 					-->
 				</script>
 					</c:when>
 					<c:when test="${!resource.allowAddFiles && resource.allowAddUrls}">
 						<script type="text/javascript">
 					<!--
-						showMessage("<c:url value='/pages/learning/addurl.jsp'/>?sessionMapID=${sessionMapID}&mode=${mode}");
+						showMessage("<c:url value='/learning/addurl.do'/>?sessionMapID=${sessionMapID}&mode=${mode}");
 					-->
 				</script>
 					</c:when>
Index: lams_tool_larsrc/web/pages/learning/success.jsp
===================================================================
diff -u -r92e8b8d1454cfbd9ae8a20934a677340dac3b507 -r12fef57b1c4aea30f19676803c58600da0a5a167
--- lams_tool_larsrc/web/pages/learning/success.jsp	(.../success.jsp)	(revision 92e8b8d1454cfbd9ae8a20934a677340dac3b507)
+++ lams_tool_larsrc/web/pages/learning/success.jsp	(.../success.jsp)	(revision 12fef57b1c4aea30f19676803c58600da0a5a167)
@@ -12,10 +12,10 @@
 		</script>
 		<div style="align:center">
 			<c:if test="${addType == 1}">
-				Add URL success, <a href="<c:url value='/pages/learning/addurl.jsp'/>?sessionMapID=${sessionMapID}&mode=${mode}" target="newResourceFrame">click here to return</a>.
+				Add URL success, <a href="<c:url value='/learning/addurl.do'/>?sessionMapID=${sessionMapID}&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'/>?sessionMapID=${sessionMapID}&mode=${mode}" target="newResourceFrame">click here to return</a>.
+				Add File success, <a href="<c:url value='/learning/addfile.do'/>?sessionMapID=${sessionMapID}&mode=${mode}" target="newResourceFrame">click here to return</a>.
 			</c:if>
 		</div>
 	</body>
Index: lams_tool_larsrc/web/pages/learningobj/mainframe.jsp
===================================================================
diff -u -rc25a2c783e51d05045d1fb61981c169357cd3e98 -r12fef57b1c4aea30f19676803c58600da0a5a167
--- lams_tool_larsrc/web/pages/learningobj/mainframe.jsp	(.../mainframe.jsp)	(revision c25a2c783e51d05045d1fb61981c169357cd3e98)
+++ lams_tool_larsrc/web/pages/learningobj/mainframe.jsp	(.../mainframe.jsp)	(revision 12fef57b1c4aea30f19676803c58600da0a5a167)
@@ -21,7 +21,7 @@
 <%@ include file="/common/taglibs.jsp" %>
 <html>
 <frameset cols="25%,75%" rows="*">
-	<frame src="navigatortree.jsp?sessionMapID=${sessionMapID}" name="navigatorFrame" marginheight="0"  scrolling="YES">
+	<frame src="navigatortree.jsp?sessionMapID=${param.sessionMapID}" name="navigatorFrame" marginheight="0"  scrolling="YES">
 	<frame src="defaultcontent.jsp" name="contentFrame" marginheight="0" scrolling="YES">
 </frameset>
 
Index: lams_tool_larsrc/web/pages/learningobj/navigatortree.jsp
===================================================================
diff -u -rc25a2c783e51d05045d1fb61981c169357cd3e98 -r12fef57b1c4aea30f19676803c58600da0a5a167
--- lams_tool_larsrc/web/pages/learningobj/navigatortree.jsp	(.../navigatortree.jsp)	(revision c25a2c783e51d05045d1fb61981c169357cd3e98)
+++ lams_tool_larsrc/web/pages/learningobj/navigatortree.jsp	(.../navigatortree.jsp)	(revision 12fef57b1c4aea30f19676803c58600da0a5a167)
@@ -28,9 +28,10 @@
 	<html:base />
 	<%@ include file="/common/header.jsp"%>
 	
-	<c:set var="sessionMap" value="${sessionScope[param.sessionMapID]}"/>
-	<c:set var="crPackage" value="${sessionMap.crPackage}"/>
-	
+	<c:set var="sessionMapID" value="${param.sessionMapID}"/>
+	<c:set var="sessionMap" value="${sessionScope[sessionMapID]}"/>
+	<c:set var="cpPackage" value="${sessionMap.cpPackage}"/>
+	 
 	<style type="text/css">
 	<!--
 	table {