Index: lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/model/Resource.hbm.xml
===================================================================
diff -u -r754e5e666cf510aff7f8f25527987cee00319ee2 -rfaa2c4cb1a645e2c7bdc48e57717dae8decf8783
--- lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/model/Resource.hbm.xml	(.../Resource.hbm.xml)	(revision 754e5e666cf510aff7f8f25527987cee00319ee2)
+++ lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/model/Resource.hbm.xml	(.../Resource.hbm.xml)	(revision faa2c4cb1a645e2c7bdc48e57717dae8decf8783)
@@ -181,11 +181,11 @@
         />
 
         <property
-            name="minViewResourceNumber"
+            name="miniViewResourceNumber"
             type="int"
             update="true"
             insert="true"
-            column="min_view_resource_number"
+            column="mini_view_resource_number"
         />
 
         <property
Index: lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/service/ResourceServiceImpl.java
===================================================================
diff -u -rc8f42df13810316bf19cfe3239ad45fe881b886e -rfaa2c4cb1a645e2c7bdc48e57717dae8decf8783
--- lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/service/ResourceServiceImpl.java	(.../ResourceServiceImpl.java)	(revision c8f42df13810316bf19cfe3239ad45fe881b886e)
+++ lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/service/ResourceServiceImpl.java	(.../ResourceServiceImpl.java)	(revision faa2c4cb1a645e2c7bdc48e57717dae8decf8783)
@@ -296,12 +296,12 @@
 
 	public Resource getResourceBySessionId(Long sessionId){
 		ResourceSession session = resourceSessionDao.getSessionBySessionId(sessionId);
-		//to skip GCLib problem
-		Resource res = session.getResource();
-		log.debug("Resource Uid:"+res.getUid()+"-Resource miniView:"+res.getMiniViewResourceNumber());
+		//to skip CGLib problem
+		Long contentId = session.getResource().getContentId();
+		Resource res = resourceDao.getByContentId(contentId);
+		int miniView = res.getMiniViewResourceNumber();
 		//construct dto fields;
-		res.setMiniViewNumberStr(messageService.getMessage("label.learning.minimum.review"
-				,new Object[new Integer(res.getMiniViewResourceNumber())]));
+		res.setMiniViewNumberStr(messageService.getMessage("label.learning.minimum.review",new Object[]{new Integer(miniView)}));
 		return res;
 	}
 	public ResourceSession getResourceSessionBySessionId(Long sessionId) {
Index: lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/web/action/LearningAction.java
===================================================================
diff -u -r0cd3a215c27e8fd64b488f94fe084b665ab15567 -rfaa2c4cb1a645e2c7bdc48e57717dae8decf8783
--- lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/web/action/LearningAction.java	(.../LearningAction.java)	(revision 0cd3a215c27e8fd64b488f94fe084b665ab15567)
+++ lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/web/action/LearningAction.java	(.../LearningAction.java)	(revision faa2c4cb1a645e2c7bdc48e57717dae8decf8783)
@@ -130,12 +130,22 @@
 	}
 
 	private ActionForward complete(ActionMapping mapping, ActionForm form, HttpServletRequest request, HttpServletResponse response) {
-		Long resourceUid = new Long(request.getParameter(ResourceConstants.PARAM_RESOURCE_ITEM_UID));
+		
+		Long resourceItemUid = new Long(request.getParameter(ResourceConstants.PARAM_RESOURCE_ITEM_UID));
 		IResourceService service = getResourceService();
 		HttpSession ss = SessionManager.getSession();
 		//get back login user DTO
 		UserDTO user = (UserDTO) ss.getAttribute(AttributeNames.USER);
-		service.setItemComplete(resourceUid,new Long(user.getUserID().intValue()));
+		service.setItemComplete(resourceItemUid,new Long(user.getUserID().intValue()));
+		
+		//set resource item complete tag
+		List<ResourceItem> resourceItemList = getResourceItemList(request);
+		for(ResourceItem item:resourceItemList){
+			if(item.getUid().equals(resourceItemUid)){
+				item.setComplete(true);
+				break;
+			}
+		}
 		return  mapping.findForward(ResourceConstants.SUCCESS);
 	}
 	/**
@@ -202,6 +212,12 @@
 		}
 		items.add(item);
 		service.saveOrUpdateResourceSession(resSession);
+		
+		//update session value
+		List<ResourceItem> resourceItemList = getResourceItemList(request);
+		resourceItemList.add(item);
+		
+		//URL or file upload
 		request.setAttribute("addType",new Short(type));
 		return  mapping.findForward(ResourceConstants.SUCCESS);
 	}
@@ -233,6 +249,7 @@
 		Resource resource;
 		try {
 			items = service.getResourceItemsBySessionId(sessionId);
+			ResourceSession session = service.getResourceSessionBySessionId(sessionId);
 			resource = service.getResourceBySessionId(sessionId);
 		} catch (Exception e) {
 			log.error(e);
@@ -247,7 +264,7 @@
 		//set complete flag for display purpose
 		service.retrieveComplete(resourceItemList, resourceUser);
 		
-		request.setAttribute(ResourceConstants.ATTR_RESOURCE,resource);
+		request.getSession().setAttribute(ResourceConstants.ATTR_RESOURCE,resource);
 		return mapping.findForward(ResourceConstants.SUCCESS);
 	}
 	//*************************************************************************************
Index: lams_tool_larsrc/web/pages/learning/learning.jsp
===================================================================
diff -u -rc8f42df13810316bf19cfe3239ad45fe881b886e -rfaa2c4cb1a645e2c7bdc48e57717dae8decf8783
--- lams_tool_larsrc/web/pages/learning/learning.jsp	(.../learning.jsp)	(revision c8f42df13810316bf19cfe3239ad45fe881b886e)
+++ lams_tool_larsrc/web/pages/learning/learning.jsp	(.../learning.jsp)	(revision faa2c4cb1a645e2c7bdc48e57717dae8decf8783)
@@ -8,27 +8,32 @@
 	<!--
 		function gotoURL(){
  		    var reqIDVar = new Date();
-			parent.newResourceFrame.location = "<c:url value="/pages/learning/addurl.jsp"/>?&reqID="+reqIDVar.getTime();
+			parent.frames['newResourceFrame'].location = "<c:url value="/pages/learning/addurl.jsp"/>?&reqID="+reqIDVar.getTime();
 	      	parent.newResourceFrame.focus();
+	      	return false;
 		}
 		function gotoFile(){
  		    var reqIDVar = new Date();
-			parent.newResourceFrame.location = "<c:url value="/pages/learning/addfile.jsp"/>?&reqID="+reqIDVar.getTime();
+			parent.frames['newResourceFrame'].location = "<c:url value="/pages/learning/addfile.jsp"/>?&reqID="+reqIDVar.getTime();
 	      	parent.newResourceFrame.focus();
+	      	return false;
 		}
 		function checkNew(){
  		    var reqIDVar = new Date();
-			parent.learningFrame.location = "<c:url value="/learning/start.do"/>?toolSessionID=${toolSessionID}&reqID="+reqIDVar.getTime();
+			parent.frames['learningFrame'].location= "<c:url value="/learning/start.do"/>?toolSessionID=${toolSessionID}&reqID="+reqIDVar.getTime();
+ 		    return false;
 		}
 		function viewItem(itemUid){
 			var myUrl = "<c:url value="/reviewItem.do"/>?itemUid=" + itemUid;
 			launchPopup(myUrl,"Review");
 		}
 		function completeItem(itemUid){
-			parent.learningFrame.location = "<c:url value="/learning/completeItem.do"/>?itemUid=" + itemUid;
+			parent.frames['learningFrame'].location = "<c:url value="/learning/completeItem.do"/>?itemUid=" + itemUid;
+			return false;
 		}
 		function finishSession(){
-			parent.learningFrame.location='<c:url value="/learning/finish.do?toolSessionID=${toolSessionID}&mode=learner"/>';
+			parent.location ='<c:url value="/learning/finish.do?toolSessionID=${toolSessionID}&mode=learner"/>';
+			return false;
 		}
 	-->        
     </script>
@@ -43,13 +48,11 @@
 				<h1>
 					<fmt:message key="label.learning.heading" />
 				</h1>
+				<h2>
+					${resource.instructions}
+				</h2>
 			</td>
 		</tr>
-		<tr>
-			<td class="formlabel">
-				${resource.instructions}
-			</td>
-		</tr>
 	</table>
 	<%@ include file="/common/messages.jsp"%>
 	<table border="0" align="center" class="forms" width="95%">
@@ -61,7 +64,7 @@
 				<fmt:message key="label.completed" />
 			</th>
 			<th scope="col" style="width:200px">
-				<a href="javascript:;" style="width:120px" class="button" onclick="checkNew()"><fmt:message key="label.check.for.new" /></a>
+				<a href="#" style="width:120px" class="button" onclick="return checkNew()"><fmt:message key="label.check.for.new" /></a>
 			</th>
 		</tr>
 		<c:forEach var="item" items="${resourceList}">
@@ -76,21 +79,21 @@
 				
 				<td>
 					<a href="javascript:;" class="button" onclick="viewItem(${item.uid})" ><fmt:message key="label.view" /></a>
-					<a href="javascript:;" class="button" onclick="completeItem(${item.uid})"><fmt:message key="label.completed" /></a>
+					<a href="#" class="button" onclick="return completeItem(${item.uid})"><fmt:message key="label.completed" /></a>
 				</td>
 			</tr>
 		</c:forEach>
 		<tr>
 			<td colspan="3" align="right">
-				<a href="javascript:;" class="button" onclick="finishSession()">
+				<a href="#" class="button" onclick="return finishSession()">
 					<fmt:message key="label.finished" />
 				</a>
 			</td>
 		</tr>
-		<c:if test="${resource.miniViewNumberStr}">
+		<c:if test="${resource.miniViewResourceNumber > 0}">
 			<tr>
 				<td colspan="4" align="left">
-					<b><fmt:message key="label.learning.minmum.review" /></b>
+					<b>${resource.miniViewNumberStr}</b>
 				</td>
 			</tr>
 		</c:if>
Index: lams_tool_larsrc/web/pages/learning/learningframe.jsp
===================================================================
diff -u -r6b786496dac3e368e8a686373ac4e3cdee391c47 -rfaa2c4cb1a645e2c7bdc48e57717dae8decf8783
--- lams_tool_larsrc/web/pages/learning/learningframe.jsp	(.../learningframe.jsp)	(revision 6b786496dac3e368e8a686373ac4e3cdee391c47)
+++ lams_tool_larsrc/web/pages/learning/learningframe.jsp	(.../learningframe.jsp)	(revision faa2c4cb1a645e2c7bdc48e57717dae8decf8783)
@@ -1,8 +1,8 @@
 <%@ include file="/common/taglibs.jsp" %>
 <html>
 <frameset rows="300,*" frameborder="no">
-	<frame src="<c:url value="/pages/learning/learning.jsp"/>" name="learningFrame" marginheight="0"  scrolling="YES">
-	<frame src="<c:url value="/pages/learning/addurl.jsp"/>" name="newResourceFrame" marginheight="0" scrolling="YES">
+	<frame src="<c:url value="/pages/learning/learning.jsp"/>" name="learningFrame" id="learningFrame" marginheight="0"  scrolling="YES">
+	<frame src="<c:url value="/pages/learning/addurl.jsp"/>" name="newResourceFrame" id="newResourceFrame" marginheight="0" scrolling="YES">
 </frameset>
 
 </html>
\ No newline at end of file
Index: lams_tool_larsrc/web/pages/learning/success.jsp
===================================================================
diff -u -r27653cd039c7a5c282fed7bb692473179a4d89ec -rfaa2c4cb1a645e2c7bdc48e57717dae8decf8783
--- lams_tool_larsrc/web/pages/learning/success.jsp	(.../success.jsp)	(revision 27653cd039c7a5c282fed7bb692473179a4d89ec)
+++ lams_tool_larsrc/web/pages/learning/success.jsp	(.../success.jsp)	(revision faa2c4cb1a645e2c7bdc48e57717dae8decf8783)
@@ -5,7 +5,7 @@
 	</head>
 	<body>
 		<script type="text/javascript">
-			parent.learningFrame.location = "<c:url value="/learning/start.do"/>?toolSessionID=${toolSessionID}";
+			parent.frames['learningFrame'].location  = "<c:url value="/learning/start.do"/>?toolSessionID=${toolSessionID}";
 		</script>
 		<div style="align:center">
 			<c:if test="${addType == 1}">