Index: lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/ResourceConstants.java
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/ResourceConstants.java,v
diff -u -r1.23 -r1.24
--- lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/ResourceConstants.java	15 Jun 2006 00:04:03 -0000	1.23
+++ lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/ResourceConstants.java	3 Jul 2006 00:58:07 -0000	1.24
@@ -81,5 +81,6 @@
 	
 	
 	public static final String PAGE_EDITABLE = "isPageEditable";
+	public static final String MODE_AUTHOR_SESSION = "author_session";
 
 }
Index: lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/web/action/ViewItemAction.java
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/web/action/ViewItemAction.java,v
diff -u -r1.8 -r1.9
--- lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/web/action/ViewItemAction.java	15 Jun 2006 00:04:03 -0000	1.8
+++ lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/web/action/ViewItemAction.java	3 Jul 2006 00:58:07 -0000	1.9
@@ -194,12 +194,12 @@
 	 */
 	private ResourceItem getResourceItem(HttpServletRequest request, String mode) {
 		ResourceItem item = null;		
-		if(ToolAccessMode.AUTHOR.toString().equals(mode)){
+		if(ResourceConstants.MODE_AUTHOR_SESSION.equals(mode)){
 			int itemIdx = NumberUtils.stringToInt(request.getParameter(ResourceConstants.PARAM_ITEM_INDEX),0);
 			//authoring: does not save item yet, so only has ItemList from session and identity by Index
 			List<ResourceItem> resourceList = getResourceItemList(request);
 			item = resourceList.get(itemIdx);
-		}else if(ToolAccessMode.LEARNER.toString().equals(mode) || ToolAccessMode.TEACHER.toString().equals(mode)){
+		}else{
 			Long itemUid = NumberUtils.createLong(request.getParameter(ResourceConstants.PARAM_RESOURCE_ITEM_UID));
 //			get back the resource and item list and display them on page
 			IResourceService service = getResourceService();			
Index: lams_tool_larsrc/web/pages/authoring/basic.jsp
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_larsrc/web/pages/authoring/basic.jsp,v
diff -u -r1.16 -r1.17
--- lams_tool_larsrc/web/pages/authoring/basic.jsp	29 Jun 2006 08:15:11 -0000	1.16
+++ lams_tool_larsrc/web/pages/authoring/basic.jsp	3 Jul 2006 00:58:07 -0000	1.17
@@ -22,7 +22,8 @@
 
 	function previewItem(type,idx){
 		//1:url, 2:file, 3:website,4:learning object
-		var myUrl = "<c:url value='/reviewItem.do?mode=author&itemIndex='/>"+idx;
+		//This mode is special for unsaved author page. It is different with the author mode in preview 
+		var myUrl = "<c:url value='/reviewItem.do?mode=author_session&itemIndex='/>"+idx;
 		launchPopup(myUrl,"Review");
 	}
 	
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.9 -r1.10
--- lams_tool_larsrc/web/pages/itemreview/instructionsnav.jsp	29 Jun 2006 08:15:11 -0000	1.9
+++ lams_tool_larsrc/web/pages/itemreview/instructionsnav.jsp	3 Jul 2006 00:58:07 -0000	1.10
@@ -9,7 +9,8 @@
 		<script language="JavaScript" type="text/JavaScript">
 		<!--
 				function finishIns(){
-					if(${param.mode == "learner"}){
+				//learner and author(preview mode) will mark the finish
+					if(${param.mode == "learner"} || ${param.mode == "author"}){
 					   var reqIDVar = new Date();
 					   //if auto run mode, the opener will be null
 					   if(window.parent.opener != null)