Index: lams_tool_larsrc/conf/language/ApplicationResources.properties
===================================================================
diff -u -r71cccb6c7558909803c2886472385427cdc4a281 -red04e2e75728d635ec5c4530bcebe69ab32d9cdb
--- lams_tool_larsrc/conf/language/ApplicationResources.properties	(.../ApplicationResources.properties)	(revision 71cccb6c7558909803c2886472385427cdc4a281)
+++ lams_tool_larsrc/conf/language/ApplicationResources.properties	(.../ApplicationResources.properties)	(revision ed04e2e75728d635ec5c4530bcebe69ab32d9cdb)
@@ -150,4 +150,12 @@
 
 
 #======= End labels: Exported 141 labels for en AU =====
-label.monitoring.heading.access=Learners list 
\ No newline at end of file
+label.monitoring.heading.access=Learners list 
+label.authoring.advanced.reflectOnActivity=Reflect on share resources
+error.reflection.emtpy=Please input reflection
+title.reflection=Reflection
+label.continue=Continue
+monitoring.user.fullname=Name
+monitoring.user.reflection=Refection
+page.title.monitoring.view.reflection=View Reflection
+button.close=Close
\ No newline at end of file
Index: lams_tool_larsrc/conf/xdoclet/struts-actions.xml
===================================================================
diff -u -r671e0f31315ba99ec13f82789869ce2a01e0ea04 -red04e2e75728d635ec5c4530bcebe69ab32d9cdb
--- lams_tool_larsrc/conf/xdoclet/struts-actions.xml	(.../struts-actions.xml)	(revision 671e0f31315ba99ec13f82789869ce2a01e0ea04)
+++ lams_tool_larsrc/conf/xdoclet/struts-actions.xml	(.../struts-actions.xml)	(revision ed04e2e75728d635ec5c4530bcebe69ab32d9cdb)
@@ -164,7 +164,7 @@
 			type="org.lamsfoundation.lams.tool.rsrc.web.action.LearningAction"
 			input="/pages/learning/learning.jsp"
 			parameter="finish" >
-			<forward name="finish" path="/pages/learning/finish.jsp" />
+			<forward name="success" path="/pages/learning/finish.jsp" />
 		</action>
 		
 		
@@ -197,6 +197,27 @@
 			<forward name="success" path="/pages/learning/success.jsp" />
 		</action>
 		
+		<!--  ========= Reflection =============== -->
+		<action path="/learning/newReflection"
+			type="org.lamsfoundation.lams.tool.rsrc.web.action.LearningAction"
+			name="reflectionForm" 
+			validate="false" 
+			parameter="newReflection" 
+			input="/pages/learning/learning.jsp" 
+			scope="request">
+			<forward name="success" path="/pages/learning/notebook.jsp" />
+		</action>
+		
+		<action path="/learning/submitReflection"
+			type="org.lamsfoundation.lams.tool.rsrc.web.action.LearningAction"
+			name="reflectionForm" 
+			validate="true" 
+			input="/pages/learning/notebook.jsp"
+			parameter="submitReflection" 
+			scope="request">
+			<forward name="success" path="/pages/learning/finish.jsp" />
+
+		</action>
 		<!-- ========== BEGIN: Monitoring Definitions ============================== -->
 		<action path="/monitoring/summary"
 			type="org.lamsfoundation.lams.tool.rsrc.web.action.MonitoringAction"
@@ -218,6 +239,12 @@
 			parameter="showitem" >
 			<forward name="success" path="/pages/monitoring/monitoring.jsp" />
 		</action>
+		
+		<action path="/monitoring/viewReflection"
+			type="org.lamsfoundation.lams.tool.rsrc.web.action.MonitoringAction"
+			parameter="viewReflection">
+			<forward name="success" path="/pages/monitoring/notebook.jsp" />
+		</action>
 		<!-- ========== BEGIN: Common ============================== -->
 		<action path="/clearsession"
 			type="org.lamsfoundation.lams.tool.rsrc.web.action.ClearSessionAction">
Index: lams_tool_larsrc/db/sql/create_lams_tool_rsrc.sql
===================================================================
diff -u -rc25a2c783e51d05045d1fb61981c169357cd3e98 -red04e2e75728d635ec5c4530bcebe69ab32d9cdb
--- lams_tool_larsrc/db/sql/create_lams_tool_rsrc.sql	(.../create_lams_tool_rsrc.sql)	(revision c25a2c783e51d05045d1fb61981c169357cd3e98)
+++ lams_tool_larsrc/db/sql/create_lams_tool_rsrc.sql	(.../create_lams_tool_rsrc.sql)	(revision ed04e2e75728d635ec5c4530bcebe69ab32d9cdb)
@@ -41,6 +41,8 @@
    allow_add_urls tinyint,
    mini_view_resource_number integer,
    allow_auto_run tinyint,
+   reflect_instructions varchar(255), 
+   reflect_on_activity smallint, 
    primary key (uid)
 );
 create table tl_larsrc11_resource_item (
@@ -109,6 +111,6 @@
 
 
 
-INSERT INTO `tl_larsrc11_resource` (`uid`, `create_date`, `update_date`, `create_by`, `title`, `run_offline`, `lock_on_finished`, `instructions`, `online_instructions`, `offline_instructions`, `content_in_use`, `define_later`, `content_id`, `allow_add_files`, `allow_add_urls`, `mini_view_resource_number`, `allow_auto_run`) VALUES
-  (2,NULL,NULL,NULL,'Shared Resources','0','0','Instruction','Online instruction','Offline instruction',0,0,${default_content_id},1,1,0,1);
+INSERT INTO `tl_larsrc11_resource` (`uid`, `create_date`, `update_date`, `create_by`, `title`, `run_offline`, `lock_on_finished`, `instructions`, `online_instructions`, `offline_instructions`, `content_in_use`, `define_later`, `content_id`, `allow_add_files`, `allow_add_urls`, `mini_view_resource_number`, `allow_auto_run`,`reflect_on_activity`) VALUES
+  (2,NULL,NULL,NULL,'Shared Resources','0','0','Instruction','Online instruction','Offline instruction',0,0,${default_content_id},1,1,0,1,0);
 SET FOREIGN_KEY_CHECKS=1;
\ No newline at end of file
Index: lams_tool_larsrc/db/sql/table-schema.sql
===================================================================
diff -u -rc25a2c783e51d05045d1fb61981c169357cd3e98 -red04e2e75728d635ec5c4530bcebe69ab32d9cdb
--- lams_tool_larsrc/db/sql/table-schema.sql	(.../table-schema.sql)	(revision c25a2c783e51d05045d1fb61981c169357cd3e98)
+++ lams_tool_larsrc/db/sql/table-schema.sql	(.../table-schema.sql)	(revision ed04e2e75728d635ec5c4530bcebe69ab32d9cdb)
@@ -7,8 +7,8 @@
 alter table tl_larsrc11_resource_item drop foreign key FKF52D1F93EC0D3147;
 alter table tl_larsrc11_resource_item drop foreign key FKF52D1F93758092FB;
 alter table tl_larsrc11_session drop foreign key FK24AA78C530E79035;
+alter table tl_larsrc11_user drop foreign key FK30113BFC30E79035;
 alter table tl_larsrc11_user drop foreign key FK30113BFCEC0D3147;
-alter table tl_larsrc11_user drop foreign key FK30113BFC309ED320;
 drop table if exists tl_larsrc11_attachment;
 drop table if exists tl_larsrc11_item_instruction;
 drop table if exists tl_larsrc11_item_log;
@@ -19,10 +19,10 @@
 create table tl_larsrc11_attachment (uid bigint not null auto_increment, file_version_id bigint, file_type varchar(255), file_name varchar(255), file_uuid bigint, create_date datetime, resource_uid bigint, primary key (uid));
 create table tl_larsrc11_item_instruction (uid bigint not null auto_increment, description varchar(255), sequence_id integer, item_uid bigint, primary key (uid));
 create table tl_larsrc11_item_log (uid bigint not null auto_increment, access_date datetime, resource_item_uid bigint, user_uid bigint, complete bit, session_id bigint, primary key (uid));
-create table tl_larsrc11_resource (uid bigint not null auto_increment, create_date datetime, update_date datetime, create_by bigint, title varchar(255), run_offline bit, lock_on_finished bit, instructions text, online_instructions text, offline_instructions text, content_in_use bit, define_later bit, content_id bigint unique, allow_add_files bit, allow_add_urls bit, mini_view_resource_number integer, allow_auto_run bit, primary key (uid));
+create table tl_larsrc11_resource (uid bigint not null auto_increment, create_date datetime, update_date datetime, create_by bigint, title varchar(255), run_offline bit, lock_on_finished bit, instructions text, online_instructions text, offline_instructions text, content_in_use bit, define_later bit, content_id bigint unique, allow_add_files bit, allow_add_urls bit, mini_view_resource_number integer, allow_auto_run bit, reflect_instructions varchar(255), reflect_on_activity bit, primary key (uid));
 create table tl_larsrc11_resource_item (uid bigint not null auto_increment, file_uuid bigint, file_version_id bigint, description varchar(255), ims_schema varchar(255), init_item varchar(255), organization_xml text, title varchar(255), url text, create_by bigint, create_date datetime, create_by_author bit, is_hide bit, item_type smallint, file_type varchar(255), file_name varchar(255), open_url_new_window bit, resource_uid bigint, session_uid bigint, primary key (uid));
 create table tl_larsrc11_session (uid bigint not null auto_increment, session_end_date datetime, session_start_date datetime, status integer, resource_uid bigint, session_id bigint, session_name varchar(250), primary key (uid));
-create table tl_larsrc11_user (uid bigint not null auto_increment, user_id bigint, last_name varchar(255), first_name varchar(255), login_name varchar(255), session_uid bigint, content_uid bigint, session_finished bit, primary key (uid));
+create table tl_larsrc11_user (uid bigint not null auto_increment, user_id bigint, last_name varchar(255), first_name varchar(255), login_name varchar(255), session_uid bigint, resource_uid bigint, session_finished bit, primary key (uid));
 alter table tl_larsrc11_attachment add index FK1E7009430E79035 (resource_uid), add constraint FK1E7009430E79035 foreign key (resource_uid) references tl_larsrc11_resource (uid);
 alter table tl_larsrc11_item_instruction add index FKA5665013980570ED (item_uid), add constraint FKA5665013980570ED foreign key (item_uid) references tl_larsrc11_resource_item (uid);
 alter table tl_larsrc11_item_log add index FK63195BC938BF8DFE (resource_item_uid), add constraint FK63195BC938BF8DFE foreign key (resource_item_uid) references tl_larsrc11_resource_item (uid);
@@ -32,5 +32,5 @@
 alter table tl_larsrc11_resource_item add index FKF52D1F93EC0D3147 (session_uid), add constraint FKF52D1F93EC0D3147 foreign key (session_uid) references tl_larsrc11_session (uid);
 alter table tl_larsrc11_resource_item add index FKF52D1F93758092FB (create_by), add constraint FKF52D1F93758092FB foreign key (create_by) references tl_larsrc11_user (uid);
 alter table tl_larsrc11_session add index FK24AA78C530E79035 (resource_uid), add constraint FK24AA78C530E79035 foreign key (resource_uid) references tl_larsrc11_resource (uid);
+alter table tl_larsrc11_user add index FK30113BFC30E79035 (resource_uid), add constraint FK30113BFC30E79035 foreign key (resource_uid) references tl_larsrc11_resource (uid);
 alter table tl_larsrc11_user add index FK30113BFCEC0D3147 (session_uid), add constraint FK30113BFCEC0D3147 foreign key (session_uid) references tl_larsrc11_session (uid);
-alter table tl_larsrc11_user add index FK30113BFC309ED320 (content_uid), add constraint FK30113BFC309ED320 foreign key (content_uid) references tl_larsrc11_resource (uid);
Index: lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/ResourceConstants.java
===================================================================
diff -u -re76c3706b6db1d6af4fc1c15a0afabbf2ad04d4d -red04e2e75728d635ec5c4530bcebe69ab32d9cdb
--- lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/ResourceConstants.java	(.../ResourceConstants.java)	(revision e76c3706b6db1d6af4fc1c15a0afabbf2ad04d4d)
+++ lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/ResourceConstants.java	(.../ResourceConstants.java)	(revision ed04e2e75728d635ec5c4530bcebe69ab32d9cdb)
@@ -72,10 +72,10 @@
 	public static final String ATTR_RESOURCE_INSTRUCTION = "instructions";
 	public static final String ATTR_FINISH_LOCK = "finishedLock";
 	public static final String ATTR_SESSION_MAP_ID = "sessionMapID";
-	public static final Object ATTR_RESOURCE_FORM = "resourceForm";
+	public static final String ATTR_RESOURCE_FORM = "resourceForm";
 	public static final String ATTR_ADD_RESOURCE_TYPE = "addType";
 	public static final String ATTR_FILE_TYPE_FLAG = "fileTypeFlag";
-	public static final Object ATTR_TITLE = "title";
+	public static final String ATTR_TITLE = "title";
 	
 	//error message keys
 	public static final String ERROR_MSG_TITLE_BLANK = "error.resource.item.title.blank";
@@ -88,5 +88,9 @@
 	
 	public static final String PAGE_EDITABLE = "isPageEditable";
 	public static final String MODE_AUTHOR_SESSION = "author_session";
+	public static final String ATTR_REFLECTION_ON = "reflectOn";
+	public static final String ATTR_REFLECTION_INSTRUCTION = "reflectInstructions";
+	public static final String ATTR_REFLECT_LIST = "reflectList";
+	public static final String ATTR_USER_UID = "userUid";
 
 }
Index: lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/dao/ResourceUserDAO.java
===================================================================
diff -u -rc25a2c783e51d05045d1fb61981c169357cd3e98 -red04e2e75728d635ec5c4530bcebe69ab32d9cdb
--- lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/dao/ResourceUserDAO.java	(.../ResourceUserDAO.java)	(revision c25a2c783e51d05045d1fb61981c169357cd3e98)
+++ lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/dao/ResourceUserDAO.java	(.../ResourceUserDAO.java)	(revision ed04e2e75728d635ec5c4530bcebe69ab32d9cdb)
@@ -23,6 +23,8 @@
 /* $$Id$$ */
 package org.lamsfoundation.lams.tool.rsrc.dao;
 
+import java.util.List;
+
 import org.lamsfoundation.lams.tool.rsrc.model.ResourceUser;
 
 public interface ResourceUserDAO extends DAO {
@@ -31,4 +33,5 @@
 
 	ResourceUser getUserByUserIDAndContentID(Long userId, Long contentId);
 
+	List<ResourceUser> getBySessionID(Long sessionId);
 }
Index: lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/dao/hibernate/ResourceUserDAOHibernate.java
===================================================================
diff -u -r94866457dd89a14fdab60779c3cd30c4b078b659 -red04e2e75728d635ec5c4530bcebe69ab32d9cdb
--- lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/dao/hibernate/ResourceUserDAOHibernate.java	(.../ResourceUserDAOHibernate.java)	(revision 94866457dd89a14fdab60779c3cd30c4b078b659)
+++ lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/dao/hibernate/ResourceUserDAOHibernate.java	(.../ResourceUserDAOHibernate.java)	(revision ed04e2e75728d635ec5c4530bcebe69ab32d9cdb)
@@ -33,6 +33,7 @@
 	
 	private static final String FIND_BY_USER_ID_CONTENT_ID = "from " + ResourceUser.class.getName() + " as u where u.userId =? and u.resource.contentId=?";
 	private static final String FIND_BY_USER_ID_SESSION_ID = "from " + ResourceUser.class.getName() + " as u where u.userId =? and u.session.sessionId=?";
+	private static final String FIND_BY_SESSION_ID = "from " + ResourceUser.class.getName() + " as u where u.session.sessionId=?";
 
 	public ResourceUser getUserByUserIDAndSessionID(Long userID, Long sessionId) {
 		List list = this.getHibernateTemplate().find(FIND_BY_USER_ID_SESSION_ID,new Object[]{userID,sessionId});
@@ -48,5 +49,9 @@
 		return (ResourceUser) list.get(0);
 	}
 
+	public List<ResourceUser> getBySessionID(Long sessionId) {
+		return this.getHibernateTemplate().find(FIND_BY_SESSION_ID,sessionId);
+	}
 
+
 }
Index: lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/dto/ReflectDTO.java
===================================================================
diff -u
--- lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/dto/ReflectDTO.java	(revision 0)
+++ lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/dto/ReflectDTO.java	(revision ed04e2e75728d635ec5c4530bcebe69ab32d9cdb)
@@ -0,0 +1,67 @@
+package org.lamsfoundation.lams.tool.rsrc.dto;
+
+import org.lamsfoundation.lams.tool.rsrc.model.ResourceUser;
+
+/**
+ * 
+ * @author Dapeng Ni
+ *
+ */
+public class ReflectDTO {
+	private Long userUid;
+	private String fullName;
+	private String loginName;
+	private boolean hasRefection;
+	private String reflectInstrctions;
+	private boolean finishReflection;
+	private String reflect;
+	
+	public ReflectDTO(ResourceUser user) {
+		this.setLoginName(user.getLoginName());
+		this.setFullName(user.getFirstName()+" "+user.getLastName());
+		this.setUserUid(user.getUid());
+	}
+	
+	public boolean isFinishReflection() {
+		return finishReflection;
+	}
+	public void setFinishReflection(boolean finishReflection) {
+		this.finishReflection = finishReflection;
+	}
+	public String getFullName() {
+		return fullName;
+	}
+	public void setFullName(String fullName) {
+		this.fullName = fullName;
+	}
+	public boolean isHasRefection() {
+		return hasRefection;
+	}
+	public void setHasRefection(boolean hasRefection) {
+		this.hasRefection = hasRefection;
+	}
+	public String getLoginName() {
+		return loginName;
+	}
+	public void setLoginName(String loginName) {
+		this.loginName = loginName;
+	}
+	public String getReflect() {
+		return reflect;
+	}
+	public void setReflect(String reflect) {
+		this.reflect = reflect;
+	}
+	public String getReflectInstrctions() {
+		return reflectInstrctions;
+	}
+	public void setReflectInstrctions(String reflectInstrctions) {
+		this.reflectInstrctions = reflectInstrctions;
+	}
+	public Long getUserUid() {
+		return userUid;
+	}
+	public void setUserUid(Long userUid) {
+		this.userUid = userUid;
+	}
+}
Index: lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/model/Resource.hbm.xml
===================================================================
diff -u -rd4d46f925789ad7702b486b240867bdaa05e93f9 -red04e2e75728d635ec5c4530bcebe69ab32d9cdb
--- lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/model/Resource.hbm.xml	(.../Resource.hbm.xml)	(revision d4d46f925789ad7702b486b240867bdaa05e93f9)
+++ lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/model/Resource.hbm.xml	(.../Resource.hbm.xml)	(revision ed04e2e75728d635ec5c4530bcebe69ab32d9cdb)
@@ -196,6 +196,22 @@
             column="allow_auto_run"
         />
 
+        <property
+            name="reflectInstructions"
+            type="java.lang.String"
+            update="true"
+            insert="true"
+            column="reflect_instructions"
+        />
+
+        <property
+            name="reflectOnActivity"
+            type="boolean"
+            update="true"
+            insert="true"
+            column="reflect_on_activity"
+        />
+
         <!--
             To add non XDoclet property mappings, create a file named
                 hibernate-properties-Resource.xml
Index: lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/model/Resource.java
===================================================================
diff -u -r92e8b8d1454cfbd9ae8a20934a677340dac3b507 -red04e2e75728d635ec5c4530bcebe69ab32d9cdb
--- lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/model/Resource.java	(.../Resource.java)	(revision 92e8b8d1454cfbd9ae8a20934a677340dac3b507)
+++ lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/model/Resource.java	(.../Resource.java)	(revision ed04e2e75728d635ec5c4530bcebe69ab32d9cdb)
@@ -80,6 +80,9 @@
 	//resource Items
 	private Set resourceItems;
 	
+	private boolean reflectOnActivity;
+	private String reflectInstructions;
+	
 	//*************** NON Persist Fields ********************
 	private IToolContentHandler toolContentHandler;
 
@@ -537,5 +540,24 @@
 		this.toolContentHandler = toolContentHandler;
 	}
 
-
+	/**
+	 * @hibernate.property column="reflect_instructions"
+	 * @return
+	 */	
+	public String getReflectInstructions() {
+		return reflectInstructions;
+	}
+	public void setReflectInstructions(String reflectInstructions) {
+		this.reflectInstructions = reflectInstructions;
+	}
+	/**
+	 * @hibernate.property column="reflect_on_activity"
+	 * @return
+	 */		
+	public boolean isReflectOnActivity() {
+		return reflectOnActivity;
+	}
+	public void setReflectOnActivity(boolean reflectOnActivity) {
+		this.reflectOnActivity = reflectOnActivity;
+	}
 }
Index: lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/rsrcApplicationContext.xml
===================================================================
diff -u -r24ca84b9e61b98c068eaa369cb194a35e5511162 -red04e2e75728d635ec5c4530bcebe69ab32d9cdb
--- lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/rsrcApplicationContext.xml	(.../rsrcApplicationContext.xml)	(revision 24ca84b9e61b98c068eaa369cb194a35e5511162)
+++ lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/rsrcApplicationContext.xml	(.../rsrcApplicationContext.xml)	(revision ed04e2e75728d635ec5c4530bcebe69ab32d9cdb)
@@ -111,6 +111,7 @@
    			<ref bean="exportToolContentService"/>
    		</property>
    		<property name="messageService"><ref bean="messageService"/></property>
+		<property name="coreNotebookService"><ref bean="coreNotebookService"/></property>   		
     </bean>
 
     <bean id="resourceService" class="org.springframework.transaction.interceptor.TransactionProxyFactoryBean">
Index: lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/service/IResourceService.java
===================================================================
diff -u -rc25a2c783e51d05045d1fb61981c169357cd3e98 -red04e2e75728d635ec5c4530bcebe69ab32d9cdb
--- lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/service/IResourceService.java	(.../IResourceService.java)	(revision c25a2c783e51d05045d1fb61981c169357cd3e98)
+++ lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/service/IResourceService.java	(.../IResourceService.java)	(revision ed04e2e75728d635ec5c4530bcebe69ab32d9cdb)
@@ -24,9 +24,13 @@
 package org.lamsfoundation.lams.tool.rsrc.service;
 
 import java.util.List;
+import java.util.Map;
+import java.util.Set;
 
 import org.apache.struts.upload.FormFile;
 import org.lamsfoundation.lams.contentrepository.IVersionedNode;
+import org.lamsfoundation.lams.notebook.model.NotebookEntry;
+import org.lamsfoundation.lams.tool.rsrc.dto.ReflectDTO;
 import org.lamsfoundation.lams.tool.rsrc.dto.Summary;
 import org.lamsfoundation.lams.tool.rsrc.model.Resource;
 import org.lamsfoundation.lams.tool.rsrc.model.ResourceAttachment;
@@ -205,5 +209,37 @@
 	public List<Summary> exportBySessionId(Long sessionId, boolean skipHide);
 	public List<List<Summary>> exportByContentId(Long contentId);
 
+	/**
+	 * Create refection entry into notebook tool.
+	 * @param sessionId
+	 * @param notebook_tool
+	 * @param tool_signature
+	 * @param userId
+	 * @param entryText
+	 */
+	public Long createNotebookEntry(Long sessionId, Integer notebookToolType, String toolSignature, Integer userId, String entryText);
+	/**
+	 * Get reflection entry from notebook tool.
+	 * @param sessionId
+	 * @param idType
+	 * @param signature
+	 * @param userID
+	 * @return
+	 */
+	public NotebookEntry getEntry(Long sessionId, Integer idType, String signature, Integer userID);
+
+	/**
+	 * Get Reflect DTO list grouped by sessionID.
+	 * @param contentId
+	 * @return
+	 */
+	Map<Long, Set<ReflectDTO>> getReflectList(Long contentId);
+
+	/**
+	 * Get user by UID
+	 * @param uid
+	 * @return
+	 */
+	ResourceUser getUser(Long uid);
 }
 
Index: lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/service/ResourceServiceImpl.java
===================================================================
diff -u -r24ca84b9e61b98c068eaa369cb194a35e5511162 -red04e2e75728d635ec5c4530bcebe69ab32d9cdb
--- lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/service/ResourceServiceImpl.java	(.../ResourceServiceImpl.java)	(revision 24ca84b9e61b98c068eaa369cb194a35e5511162)
+++ lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/service/ResourceServiceImpl.java	(.../ResourceServiceImpl.java)	(revision ed04e2e75728d635ec5c4530bcebe69ab32d9cdb)
@@ -29,13 +29,16 @@
 import java.io.InputStream;
 import java.sql.Timestamp;
 import java.util.ArrayList;
+import java.util.Comparator;
 import java.util.Date;
+import java.util.HashMap;
 import java.util.HashSet;
 import java.util.Hashtable;
 import java.util.Iterator;
 import java.util.List;
 import java.util.Map;
 import java.util.Set;
+import java.util.TreeSet;
 import java.util.Vector;
 
 import org.apache.commons.lang.StringUtils;
@@ -57,6 +60,8 @@
 import org.lamsfoundation.lams.learningdesign.service.ExportToolContentException;
 import org.lamsfoundation.lams.learningdesign.service.IExportToolContentService;
 import org.lamsfoundation.lams.learningdesign.service.ImportToolContentException;
+import org.lamsfoundation.lams.notebook.model.NotebookEntry;
+import org.lamsfoundation.lams.notebook.service.ICoreNotebookService;
 import org.lamsfoundation.lams.tool.ToolContentImport102Manager;
 import org.lamsfoundation.lams.tool.ToolContentManager;
 import org.lamsfoundation.lams.tool.ToolSessionExportOutputData;
@@ -71,6 +76,7 @@
 import org.lamsfoundation.lams.tool.rsrc.dao.ResourceItemVisitDAO;
 import org.lamsfoundation.lams.tool.rsrc.dao.ResourceSessionDAO;
 import org.lamsfoundation.lams.tool.rsrc.dao.ResourceUserDAO;
+import org.lamsfoundation.lams.tool.rsrc.dto.ReflectDTO;
 import org.lamsfoundation.lams.tool.rsrc.dto.Summary;
 import org.lamsfoundation.lams.tool.rsrc.ims.IContentPackageConverter;
 import org.lamsfoundation.lams.tool.rsrc.ims.IMSManifestException;
@@ -121,7 +127,19 @@
 	private IAuditService auditService;
 	private IUserManagementService userManagementService; 
 	private IExportToolContentService exportContentService;
+	private ICoreNotebookService coreNotebookService;
 	
+	
+	private class ReflectDTOComparator implements Comparator<ReflectDTO>{
+		public int compare(ReflectDTO o1, ReflectDTO o2) {
+			if(o1 != null && o2 != null){
+				return o1.getFullName().compareTo(o2.getFullName());
+			}else if(o1 != null)
+				return 1;
+			else
+				return -1;
+		}
+	}
 	public IVersionedNode getFileNode(Long itemUid, String relPathString) throws ResourceApplicationException {
 		ResourceItem item = (ResourceItem) resourceItemDao.getObject(ResourceItem.class,itemUid);
 		if ( item == null )
@@ -530,7 +548,26 @@
 		return groupList;
 
 	}
+	public Map<Long, Set<ReflectDTO>> getReflectList(Long contentId){
+		Map<Long, Set<ReflectDTO>> map = new HashMap<Long, Set<ReflectDTO>>();
 
+		List<ResourceSession> sessionList = resourceSessionDao.getByContentId(contentId);
+		for(ResourceSession session:sessionList){
+			Long sessionId = session.getSessionId();
+			boolean hasRefection = session.getResource().isReflectOnActivity();
+			Set<ReflectDTO> list = new TreeSet<ReflectDTO>(this.new ReflectDTOComparator());
+			//get all users in this session
+			List<ResourceUser> users = resourceUserDao.getBySessionID(sessionId);
+			for(ResourceUser user : users){
+				ReflectDTO ref = new ReflectDTO(user);
+				ref.setHasRefection(hasRefection);
+				list.add(ref);
+			}
+			map.put(sessionId, list);
+		}
+		
+		return map;
+	}
 	public List<ResourceUser> getUserListBySessionItem(Long sessionId, Long itemUid) {
 		List<ResourceItemVisitLog> logList = resourceItemVisitDao.getResourceItemLogBySession(sessionId,itemUid);
 		List<ResourceUser> userList = new ArrayList(logList.size());
@@ -556,7 +593,21 @@
 			resourceItemDao.saveObject(item);
 		}
 	}
-
+	public Long createNotebookEntry(Long sessionId, Integer notebookToolType, String toolSignature, Integer userId, String entryText) {
+		return coreNotebookService.createNotebookEntry(sessionId, notebookToolType, toolSignature, userId, "", entryText);
+	}
+	public NotebookEntry getEntry(Long sessionId, Integer idType, String signature, Integer userID){
+		List<NotebookEntry> list = coreNotebookService.getEntry(sessionId, idType, signature, userID);
+		if (list == null || list.isEmpty()) {
+			return null;
+		} else {
+			return list.get(0);
+		}
+	}
+	
+	public ResourceUser getUser(Long uid){
+		return (ResourceUser) resourceUserDao.getObject(ResourceUser.class, uid);
+	}
 	//*****************************************************************************
 	// private methods
 	//*****************************************************************************
@@ -1082,5 +1133,15 @@
 	}
 
 
+	public ICoreNotebookService getCoreNotebookService() {
+		return coreNotebookService;
+	}
 
+
+	public void setCoreNotebookService(ICoreNotebookService coreNotebookService) {
+		this.coreNotebookService = coreNotebookService;
+	}
+
+
+
 }
\ No newline at end of file
Index: lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/web/action/AuthoringAction.java
===================================================================
diff -u -r671e0f31315ba99ec13f82789869ce2a01e0ea04 -red04e2e75728d635ec5c4530bcebe69ab32d9cdb
--- lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/web/action/AuthoringAction.java	(.../AuthoringAction.java)	(revision 671e0f31315ba99ec13f82789869ce2a01e0ea04)
+++ lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/web/action/AuthoringAction.java	(.../AuthoringAction.java)	(revision ed04e2e75728d635ec5c4530bcebe69ab32d9cdb)
@@ -577,9 +577,9 @@
 			service.deleteFromRepository(delAtt.getFileUuid(),delAtt.getFileVersionId());
 		}
 		
-		//if miniview number is bigger than available items, then force it to 1
+		//if miniview number is bigger than available items, then set it topics size
 		if(resourcePO.getMiniViewResourceNumber() > topics.size())
-			resourcePO.setMiniViewResourceNumber((topics.size()==0?0:1));
+			resourcePO.setMiniViewResourceNumber((topics.size()));
 		//**********************************************
 		//finally persist resourcePO again
 		service.saveOrUpdateResource(resourcePO);
Index: lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/web/action/LearningAction.java
===================================================================
diff -u -re76c3706b6db1d6af4fc1c15a0afabbf2ad04d4d -red04e2e75728d635ec5c4530bcebe69ab32d9cdb
--- lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/web/action/LearningAction.java	(.../LearningAction.java)	(revision e76c3706b6db1d6af4fc1c15a0afabbf2ad04d4d)
+++ lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/web/action/LearningAction.java	(.../LearningAction.java)	(revision ed04e2e75728d635ec5c4530bcebe69ab32d9cdb)
@@ -46,6 +46,8 @@
 import org.apache.struts.action.ActionMapping;
 import org.apache.struts.action.ActionMessage;
 import org.apache.struts.action.ActionMessages;
+import org.lamsfoundation.lams.notebook.service.CoreNotebookConstants;
+import org.lamsfoundation.lams.tool.ToolAccessMode;
 import org.lamsfoundation.lams.tool.rsrc.ResourceConstants;
 import org.lamsfoundation.lams.tool.rsrc.model.Resource;
 import org.lamsfoundation.lams.tool.rsrc.model.ResourceItem;
@@ -54,6 +56,7 @@
 import org.lamsfoundation.lams.tool.rsrc.service.IResourceService;
 import org.lamsfoundation.lams.tool.rsrc.service.ResourceApplicationException;
 import org.lamsfoundation.lams.tool.rsrc.service.UploadResourceFileException;
+import org.lamsfoundation.lams.tool.rsrc.web.form.ReflectionForm;
 import org.lamsfoundation.lams.tool.rsrc.web.form.ResourceItemForm;
 import org.lamsfoundation.lams.usermanagement.dto.UserDTO;
 import org.lamsfoundation.lams.util.WebUtil;
@@ -96,6 +99,15 @@
         if (param.equals("saveOrUpdateItem")) {
         	return saveOrUpdateItem(mapping, form, request, response);
         }
+        
+		//================ Reflection =======================
+		if (param.equals("newReflection")) {
+			return newReflection(mapping, form, request, response);
+		}
+		if (param.equals("submitReflection")) {
+			return submitReflection(mapping, form, request, response);
+		}
+		
 		return  mapping.findForward(ResourceConstants.ERROR);
 	}
 	/**
@@ -126,10 +138,9 @@
 		request.getSession().setAttribute(sessionMap.getSessionID(), sessionMap);
 		
 		//save toolContentID into HTTPSession
-		String mode = request.getParameter(AttributeNames.ATTR_MODE);
+		ToolAccessMode mode = WebUtil.readToolAccessModeParam(request,AttributeNames.PARAM_MODE, true);
 		
 		Long sessionId =  new Long(request.getParameter(ResourceConstants.PARAM_TOOL_SESSION_ID));
-		sessionMap.put(AttributeNames.PARAM_TOOL_SESSION_ID,sessionId);
 		
 		request.setAttribute(ResourceConstants.ATTR_SESSION_MAP_ID, sessionMap.getSessionID());
 		request.setAttribute(AttributeNames.ATTR_MODE,mode);
@@ -143,16 +154,48 @@
 		Resource resource;
 		items = service.getResourceItemsBySessionId(sessionId);
 		resource = service.getResourceBySessionId(sessionId);
+
+		//check whehter finish lock is on/off
+		boolean lock = resource.getLockWhenFinished() && resourceUser.isSessionFinished();
 		
 		
+		//check whether there is only one resource item and run auto flag is true or not.
+		boolean runAuto = false;
+		int itemsNumber = 0;
+		if(resource.getResourceItems() != null){
+			itemsNumber = resource.getResourceItems().size();
+			if(resource.isRunAuto() && itemsNumber == 1){
+				ResourceItem item = (ResourceItem) resource.getResourceItems().iterator().next();
+				//only visible item can be run auto.
+				if(!item.isHide()){
+					runAuto = true;
+					request.setAttribute(ResourceConstants.ATTR_RESOURCE_ITEM_UID,item.getUid());
+				}
+			}
+		}
+		
+		//basic information
+		sessionMap.put(ResourceConstants.ATTR_TITLE,resource.getTitle());
+		sessionMap.put(ResourceConstants.ATTR_RESOURCE_INSTRUCTION,resource.getInstructions());
+		sessionMap.put(ResourceConstants.ATTR_FINISH_LOCK,lock);
+		
+		sessionMap.put(AttributeNames.PARAM_TOOL_SESSION_ID,sessionId);
+		sessionMap.put(AttributeNames.ATTR_MODE,mode);
+		//reflection information
+		sessionMap.put(ResourceConstants.ATTR_REFLECTION_ON,resource.isReflectOnActivity());
+		sessionMap.put(ResourceConstants.ATTR_REFLECTION_INSTRUCTION,resource.getReflectInstructions());
+		sessionMap.put(ResourceConstants.ATTR_RUN_AUTO,new Boolean(runAuto));
+		
 		//add define later support
 		if(resource.isDefineLater()){
 			return mapping.findForward("defineLater");
 		}
 		//add run offline support
 		if(resource.getRunOffline()){
+			sessionMap.put(ResourceConstants.PARAM_RUN_OFFLINE, true);
 			return mapping.findForward("runOffline");
-		}
+		}else
+			sessionMap.put(ResourceConstants.PARAM_RUN_OFFLINE, false);
 				
 		//init resource item list
 		List<ResourceItem> resourceItemList = getResourceItemList(sessionMap);
@@ -174,26 +217,6 @@
 		
 		sessionMap.put(ResourceConstants.ATTR_RESOURCE,resource);
 		
-		//check whether there is only one resource item and run auto flag is true or not.
-		boolean runAuto = false;
-		int itemsNumber = 0;
-		if(resource.getResourceItems() != null){
-			itemsNumber = resource.getResourceItems().size();
-			if(resource.isRunAuto() && itemsNumber == 1){
-				ResourceItem item = (ResourceItem) resource.getResourceItems().iterator().next();
-				//only visible item can be run auto.
-				if(!item.isHide()){
-					runAuto = true;
-					request.setAttribute(ResourceConstants.ATTR_RESOURCE_ITEM_UID,item.getUid());
-				}
-			}
-		}
-		request.setAttribute(ResourceConstants.ATTR_RUN_AUTO,new Boolean(runAuto));
-		
-		//check whehter finish lock is on/off
-		boolean lock = resource.getLockWhenFinished() && resourceUser.isSessionFinished();
-		sessionMap.put(ResourceConstants.ATTR_FINISH_LOCK,lock);
-		
 		//set contentInUse flag to true!
 		resource.setContentInUse(true);
 		resource.setDefineLater(false);
@@ -229,49 +252,43 @@
 	 * @return
 	 */
 	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);
 		
 		//get back SessionMap
 		String sessionMapID = request.getParameter(ResourceConstants.ATTR_SESSION_MAP_ID);
 		SessionMap sessionMap = (SessionMap) request.getSession().getAttribute(sessionMapID);
 		
-		Long sessionId =  (Long) sessionMap.get(ResourceConstants.ATTR_TOOL_SESSION_ID);
+		//get mode and ToolSessionID from sessionMAP
+		ToolAccessMode mode = (ToolAccessMode) sessionMap.get(AttributeNames.ATTR_MODE);
+		Long sessionId = (Long) sessionMap.get(AttributeNames.PARAM_TOOL_SESSION_ID);
 		
+		//auto run mode, when use finish the only one resource item, mark it as complete then finish this activity as well.
+		String resourceItemUid = request.getParameter(ResourceConstants.PARAM_RESOURCE_ITEM_UID);
 		if(resourceItemUid != null){
 			doComplete(request);
+			//NOTE:So far this flag is useless(31/08/2006).
+			//set flag, then finish page can know redir target is parent(AUTO_RUN) or self(normal)
 			request.setAttribute(ResourceConstants.ATTR_RUN_AUTO,true);
 		}else
 			request.setAttribute(ResourceConstants.ATTR_RUN_AUTO,false);
 		
-		HttpSession ss = SessionManager.getSession();
-		UserDTO user = (UserDTO) ss.getAttribute(AttributeNames.USER);
-		Long userID = new Long(user.getUserID().longValue());
+		if(!validateBeforeFinish(request, sessionMapID))
+			return mapping.getInputForward();
 		
 		IResourceService service = getResourceService();
-		int miniViewFlag = service.checkMiniView(sessionId,userID);
-		//if current user view less than reqired view count number, then just return error message.
-		//if it is runOffline content, then need not check minimum view count
-		if(miniViewFlag > 0 && !StringUtils.equalsIgnoreCase(runOffline,"true")){
-			ActionErrors errors = new ActionErrors();
-			errors.add(ActionMessages.GLOBAL_MESSAGE,new ActionMessage("lable.learning.minimum.view.number.less",miniViewFlag));
-			this.addErrors(request,errors);
-			return mapping.getInputForward();
-			
-		}
 		// get sessionId from HttpServletRequest
 		String nextActivityUrl = null ;
 		try {
+			HttpSession ss = SessionManager.getSession();
+			UserDTO user = (UserDTO) ss.getAttribute(AttributeNames.USER);
+			Long userID = new Long(user.getUserID().longValue());
+			
 			nextActivityUrl = service.finishToolSession(sessionId,userID);
 			request.setAttribute(ResourceConstants.ATTR_NEXT_ACTIVITY_URL,nextActivityUrl);
 		} catch (ResourceApplicationException e) {
 			log.error("Failed get next activity url:" + e.getMessage());
 		}
 		
-		request.setAttribute(AttributeNames.ATTR_MODE,mode);
-		return mapping.findForward("finish");
+		return  mapping.findForward(ResourceConstants.SUCCESS);
 	}
 
 	
@@ -349,10 +366,84 @@
 		request.setAttribute(AttributeNames.ATTR_MODE,mode);
 		return  mapping.findForward(ResourceConstants.SUCCESS);
 	}
+	/**
+	 * Display empty reflection form.
+	 * @param mapping
+	 * @param form
+	 * @param request
+	 * @param response
+	 * @return
+	 */
+	private ActionForward newReflection(ActionMapping mapping, ActionForm form, HttpServletRequest request,
+			HttpServletResponse response) {
+		
+		//get session value
+		String sessionMapID = WebUtil.readStrParam(request, ResourceConstants.ATTR_SESSION_MAP_ID);
+		if(!validateBeforeFinish(request, sessionMapID))
+			return mapping.getInputForward();
+
+		ReflectionForm refForm = (ReflectionForm) form;
+		HttpSession ss = SessionManager.getSession();
+		UserDTO user = (UserDTO) ss.getAttribute(AttributeNames.USER);
+		
+		refForm.setUserID(user.getUserID());
+		refForm.setSessionMapID(sessionMapID);
+		
+		return mapping.findForward(ResourceConstants.SUCCESS);
+	}
+	/**
+	 * Submit reflection form input database.
+	 * @param mapping
+	 * @param form
+	 * @param request
+	 * @param response
+	 * @return
+	 */
+	private ActionForward submitReflection(ActionMapping mapping, ActionForm form, HttpServletRequest request,
+			HttpServletResponse response) {
+		ReflectionForm refForm = (ReflectionForm) form;
+		Integer userId = refForm.getUserID();
+		
+		String sessionMapID = WebUtil.readStrParam(request, ResourceConstants.ATTR_SESSION_MAP_ID);
+		SessionMap sessionMap = (SessionMap) request.getSession().getAttribute(sessionMapID);
+		Long sessionId = (Long) sessionMap.get(AttributeNames.PARAM_TOOL_SESSION_ID);
+		
+		IResourceService service = getResourceService();
+		service.createNotebookEntry(sessionId, 
+				CoreNotebookConstants.NOTEBOOK_TOOL,
+				ResourceConstants.TOOL_SIGNATURE, 
+				userId,
+				refForm.getEntryText());
+
+		return finish(mapping, form, request, response);
+	}
 	
+
 	//*************************************************************************************
 	// Private method 
 	//*************************************************************************************
+	private boolean validateBeforeFinish(HttpServletRequest request, String sessionMapID) {
+		SessionMap sessionMap = (SessionMap) request.getSession().getAttribute(sessionMapID);
+		Long sessionId = (Long) sessionMap.get(AttributeNames.PARAM_TOOL_SESSION_ID);
+		
+		HttpSession ss = SessionManager.getSession();
+		UserDTO user = (UserDTO) ss.getAttribute(AttributeNames.USER);
+		Long userID = new Long(user.getUserID().longValue());
+		
+		IResourceService service = getResourceService();
+		int miniViewFlag = service.checkMiniView(sessionId,userID);
+		//if current user view less than reqired view count number, then just return error message.
+		//if it is runOffline content, then need not check minimum view count
+		Boolean runOffline = (Boolean) sessionMap.get(ResourceConstants.PARAM_RUN_OFFLINE);
+		if(miniViewFlag > 0 && !runOffline){
+			ActionErrors errors = new ActionErrors();
+			errors.add(ActionMessages.GLOBAL_MESSAGE,new ActionMessage("lable.learning.minimum.view.number.less",miniViewFlag));
+			this.addErrors(request,errors);
+			return false;
+		}
+		
+		return true;
+	}
 	private IResourceService getResourceService() {
 	      WebApplicationContext wac = WebApplicationContextUtils.getRequiredWebApplicationContext(getServlet().getServletContext());
 	      return (IResourceService) wac.getBean(ResourceConstants.RESOURCE_SERVICE);
Index: lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/web/action/MonitoringAction.java
===================================================================
diff -u -r71cccb6c7558909803c2886472385427cdc4a281 -red04e2e75728d635ec5c4530bcebe69ab32d9cdb
--- lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/web/action/MonitoringAction.java	(.../MonitoringAction.java)	(revision 71cccb6c7558909803c2886472385427cdc4a281)
+++ lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/web/action/MonitoringAction.java	(.../MonitoringAction.java)	(revision ed04e2e75728d635ec5c4530bcebe69ab32d9cdb)
@@ -26,6 +26,8 @@
 
 import java.io.IOException;
 import java.util.List;
+import java.util.Map;
+import java.util.Set;
 
 import javax.servlet.ServletException;
 import javax.servlet.http.HttpServletRequest;
@@ -36,21 +38,22 @@
 import org.apache.struts.action.ActionForm;
 import org.apache.struts.action.ActionForward;
 import org.apache.struts.action.ActionMapping;
-import org.lamsfoundation.lams.tool.ToolAccessMode;
+import org.lamsfoundation.lams.notebook.model.NotebookEntry;
+import org.lamsfoundation.lams.notebook.service.CoreNotebookConstants;
 import org.lamsfoundation.lams.tool.rsrc.ResourceConstants;
+import org.lamsfoundation.lams.tool.rsrc.dto.ReflectDTO;
 import org.lamsfoundation.lams.tool.rsrc.dto.Summary;
 import org.lamsfoundation.lams.tool.rsrc.model.Resource;
+import org.lamsfoundation.lams.tool.rsrc.model.ResourceSession;
+import org.lamsfoundation.lams.tool.rsrc.model.ResourceUser;
 import org.lamsfoundation.lams.tool.rsrc.service.IResourceService;
-import org.lamsfoundation.lams.tool.rsrc.service.ResourceApplicationException;
 import org.lamsfoundation.lams.tool.rsrc.util.ResourceWebUtils;
 import org.lamsfoundation.lams.util.WebUtil;
 import org.lamsfoundation.lams.web.util.AttributeNames;
 import org.lamsfoundation.lams.web.util.SessionMap;
 import org.springframework.web.context.WebApplicationContext;
 import org.springframework.web.context.support.WebApplicationContextUtils;
 
-import sun.util.logging.resources.logging;
-
 public class MonitoringAction extends Action {
 	public static Logger log = Logger.getLogger(MonitoringAction.class);
 	
@@ -72,8 +75,11 @@
 		if (param.equals("hideitem")) {
 			return hideitem(mapping, form, request, response);
 		}
+		if (param.equals("viewReflection")) {
+			return viewReflection(mapping, form, request, response);
+		}
+		
 
-
 		return mapping.findForward(ResourceConstants.ERROR);
 	}
 
@@ -142,12 +148,15 @@
 		
 		Resource resource = service.getResourceByContentId(contentId);
 		resource.toDTO();
-
+		
+		Map<Long,Set<ReflectDTO> >relectList = service.getReflectList(contentId);
+		
 		//cache into sessionMap
 		sessionMap.put(ResourceConstants.ATTR_SUMMARY_LIST, groupList);
 		sessionMap.put(ResourceConstants.PAGE_EDITABLE, new Boolean(ResourceWebUtils.isResourceEditable(resource)));
 		sessionMap.put(ResourceConstants.ATTR_RESOURCE, resource);
 		sessionMap.put(ResourceConstants.ATTR_TOOL_CONTENT_ID, contentId);
+		sessionMap.put(ResourceConstants.ATTR_REFLECT_LIST, relectList);
 		
 		return mapping.findForward(ResourceConstants.SUCCESS);
 	}
@@ -165,6 +174,34 @@
 		request.setAttribute(ResourceConstants.ATTR_USER_LIST, list);
 		return mapping.findForward(ResourceConstants.SUCCESS);
 	}
+	
+	private ActionForward viewReflection(ActionMapping mapping, ActionForm form, HttpServletRequest request,
+			HttpServletResponse response) {
+		
+		Long uid = WebUtil.readLongParam(request, ResourceConstants.ATTR_USER_UID); 
+		Long sessionID = WebUtil.readLongParam(request, AttributeNames.PARAM_TOOL_SESSION_ID);
+		
+		IResourceService service = getResourceService();
+		ResourceUser user = service.getUser(uid);
+		NotebookEntry notebookEntry = service.getEntry(sessionID, 
+				CoreNotebookConstants.NOTEBOOK_TOOL, 
+				ResourceConstants.TOOL_SIGNATURE, user.getUserId().intValue());
+		
+		ResourceSession session = service.getResourceSessionBySessionId(sessionID);
+		
+		ReflectDTO refDTO = new ReflectDTO(user);
+		if(notebookEntry == null){
+			refDTO.setFinishReflection(false);
+			refDTO.setReflect(null);
+		}else{
+			refDTO.setFinishReflection(true);
+			refDTO.setReflect(notebookEntry.getEntry());
+		}
+		refDTO.setReflectInstrctions(session.getResource().getReflectInstructions());
+		
+		request.setAttribute("userDTO", refDTO);
+		return mapping.findForward("success");
+	}	
 
 	// *************************************************************************************
 	// Private method
Index: lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/web/form/ReflectionForm.java
===================================================================
diff -u
--- lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/web/form/ReflectionForm.java	(revision 0)
+++ lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/web/form/ReflectionForm.java	(revision ed04e2e75728d635ec5c4530bcebe69ab32d9cdb)
@@ -0,0 +1,92 @@
+/****************************************************************
+ * Copyright (C) 2005 LAMS Foundation (http://lamsfoundation.org)
+ * =============================================================
+ * License Information: http://lamsfoundation.org/licensing/lams/2.0/
+ * 
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 2.0 
+ * as published by the Free Software Foundation.
+ * 
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ * 
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307
+ * USA
+ * 
+ * http://www.gnu.org/licenses/gpl.txt
+ * ****************************************************************
+ */
+
+/* $$Id$$ */	
+package org.lamsfoundation.lams.tool.rsrc.web.form;
+
+import org.apache.commons.lang.StringUtils;
+import org.apache.log4j.Logger;
+import org.apache.struts.action.ActionErrors;
+import org.apache.struts.action.ActionMapping;
+import org.apache.struts.action.ActionMessage;
+import org.apache.struts.action.ActionMessages;
+import org.apache.struts.validator.ValidatorForm;
+
+/**
+ *
+ * 	Reflection Form.
+ *	@struts.form name="reflectionForm"
+ *
+ */
+public class ReflectionForm extends ValidatorForm {
+	private static final long serialVersionUID = -9054365604649146735L;
+	private static Logger logger = Logger.getLogger(ReflectionForm.class.getName());
+	
+	private Integer userID;
+	private String sessionMapID;
+	private String entryText;
+
+    /**
+     * MessageForm validation method from STRUCT interface.
+     * 
+     */
+    public ActionErrors validate(ActionMapping mapping,
+                                 javax.servlet.http.HttpServletRequest request) {
+        ActionErrors errors = new ActionErrors(); 
+        try{
+            if (StringUtils.isBlank(entryText)) {
+               ActionMessage error = new ActionMessage("error.reflection.emtpy");
+               errors.add(ActionMessages.GLOBAL_MESSAGE, error);
+            }
+            
+        } catch (Exception e) {
+            logger.error(e);
+        }
+        return errors;
+    }
+
+	public String getEntryText() {
+		return entryText;
+	}
+
+	public void setEntryText(String entryText) {
+		this.entryText = entryText;
+	}
+
+	public Integer getUserID() {
+		return userID;
+	}
+
+	public void setUserID(Integer userUid) {
+		this.userID = userUid;
+	}
+
+	public String getSessionMapID() {
+		return sessionMapID;
+	}
+
+	public void setSessionMapID(String sessionMapID) {
+		this.sessionMapID = sessionMapID;
+	}
+
+}
Index: lams_tool_larsrc/web/error.jsp
===================================================================
diff -u -r88acb04987abd12bc87754fbbe1f835dcd99a5cc -red04e2e75728d635ec5c4530bcebe69ab32d9cdb
--- lams_tool_larsrc/web/error.jsp	(.../error.jsp)	(revision 88acb04987abd12bc87754fbbe1f835dcd99a5cc)
+++ lams_tool_larsrc/web/error.jsp	(.../error.jsp)	(revision ed04e2e75728d635ec5c4530bcebe69ab32d9cdb)
@@ -55,7 +55,7 @@
 <form action="${lams}errorpages/error.jsp" method="post" id="errorForm">
 	<input type="hidden" name="errorName" value="${errorName}"/>
 	<input type="hidden" name="errorMessage" value="${errorMessage}"/>
-	<input type="hidden" name="errorStack" value="${errorStack}"/>
+	<input type="hidden" name="errorStack" value='${errorStack}'/>
 </form>
 
 <script type="text/javascript">
Index: lams_tool_larsrc/web/pages/authoring/advance.jsp
===================================================================
diff -u -rae9f5cc7e20e299c6f3b1ce692e62a33fa5d33b3 -red04e2e75728d635ec5c4530bcebe69ab32d9cdb
--- lams_tool_larsrc/web/pages/authoring/advance.jsp	(.../advance.jsp)	(revision ae9f5cc7e20e299c6f3b1ce692e62a33fa5d33b3)
+++ lams_tool_larsrc/web/pages/authoring/advance.jsp	(.../advance.jsp)	(revision ed04e2e75728d635ec5c4530bcebe69ab32d9cdb)
@@ -6,14 +6,14 @@
 		<!-- Instructions Row -->
 		<tr>
 			<td colspan="2 class="formcontrol">
-				<html:checkbox property="resource.lockWhenFinished">
+				<html:checkbox property="resource.lockWhenFinished" styleClass="noBorder">
 					<fmt:message key="label.authoring.advance.lock.on.finished" />
 				</html:checkbox>
 			</td>
 		</tr>
 		<tr>
 			<td colspan="2 class="formcontrol">
-				<html:checkbox property="resource.runAuto">
+				<html:checkbox property="resource.runAuto" styleClass="noBorder">
 					<fmt:message key="label.authoring.advance.run.content.auto" />
 				</html:checkbox>
 			</td>
@@ -39,16 +39,28 @@
 		</tr>
 		<tr>
 			<td colspan="2 class="formcontrol">
-				<html:checkbox property="resource.allowAddUrls">
+				<html:checkbox property="resource.allowAddUrls" styleClass="noBorder">
 					<fmt:message key="label.authoring.advance.allow.learner.add.urls" />
 				</html:checkbox>
 			</td>
 		</tr>
 		<tr>
 			<td colspan="2 class="formcontrol">
-				<html:checkbox property="resource.allowAddFiles">
+				<html:checkbox property="resource.allowAddFiles" styleClass="noBorder">
 					<fmt:message key="label.authoring.advance.allow.learner.add.files" />
 				</html:checkbox>
 			</td>
 		</tr>
+		<tr>
+			<td colspan="2">
+				<html:checkbox property="resource.reflectOnActivity" styleClass="noBorder"  styleId="reflectOn">
+					<fmt:message key="label.authoring.advanced.reflectOnActivity" />
+				</html:checkbox>
+			</td>
+		</tr>
+		<tr>
+			<td colspan="2">
+				<lams:STRUTS-textarea property="resource.reflectInstructions" styleId="reflectInstructions" cols="30" rows="3" />
+			</td>
+		</tr>		
 	</table>
\ No newline at end of file
Index: lams_tool_larsrc/web/pages/authoring/authoring.jsp
===================================================================
diff -u -r671e0f31315ba99ec13f82789869ce2a01e0ea04 -red04e2e75728d635ec5c4530bcebe69ab32d9cdb
--- lams_tool_larsrc/web/pages/authoring/authoring.jsp	(.../authoring.jsp)	(revision 671e0f31315ba99ec13f82789869ce2a01e0ea04)
+++ lams_tool_larsrc/web/pages/authoring/authoring.jsp	(.../authoring.jsp)	(revision ed04e2e75728d635ec5c4530bcebe69ab32d9cdb)
@@ -51,7 +51,7 @@
 	    	
 	    	//for advanceTab
 	    	if(tabId == 2)
-	    		changeViewNum();	    	
+	    		changeViewNum(-1);	    	
         } 
 
         function doUploadOnline() {
@@ -65,25 +65,29 @@
         	myForm.submit();
         }
         
-        function changeViewNum(){
+        function changeViewNum(initVal){
 			var tb = document.getElementById("itemTable");
 			var num = tb.getElementsByTagName("tr");
 			var sel = document.getElementById("viewNumList");
 			var newField = sel.options;
 			var len = sel.length;
-			var selIdx=-1;
+			
+			//when first enter, it should get value from Resource
+			var selIdx=initVal < 0?-1:initVal;
 			//there is bug in Opera8.5: if add alert before this loop, it will work,weird.
 			for (var idx=0;idx<len;idx++)
 			{
-				if(newField[0].selected && selIdx == -1)
+				if(newField[0].selected && selIdx == -1 ){
 					selIdx = newField[0].value;
+				}
 				sel.removeChild(newField[0]);
 			}
 		
-			for(var i=1;i<=num.length;i++){
+			for(var i=0;i<=num.length;i++){
 				var opt = document.createElement("option");
 				var optT =document.createTextNode(i);
 				opt.value=i;
+				//get back user choosen value
 				if(selIdx > 0 && selIdx==i){
 					opt.selected = true;
 				}else{
@@ -152,5 +156,9 @@
 <lams:HTMLEditor />
 <!-- end page div -->
 </div>
+
+<script type="text/javascript">
+	changeViewNum(${formBean.resource.miniViewResourceNumber});
+</script>
 </body>
 </html:html>
Index: lams_tool_larsrc/web/pages/itemreview/instructionsnav.jsp
===================================================================
diff -u -r92e8b8d1454cfbd9ae8a20934a677340dac3b507 -red04e2e75728d635ec5c4530bcebe69ab32d9cdb
--- lams_tool_larsrc/web/pages/itemreview/instructionsnav.jsp	(.../instructionsnav.jsp)	(revision 92e8b8d1454cfbd9ae8a20934a677340dac3b507)
+++ lams_tool_larsrc/web/pages/itemreview/instructionsnav.jsp	(.../instructionsnav.jsp)	(revision ed04e2e75728d635ec5c4530bcebe69ab32d9cdb)
@@ -1,6 +1,8 @@
 <!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01//EN"
         "http://www.w3.org/TR/html4/strict.dtd">
 <%@ include file="/common/taglibs.jsp"%>
+<c:set var="sessionMap" value="${sessionScope[sessionMapID]}"/>
+
 <html>
 	<head>
 		<title>${instructions.title}</title>
@@ -30,6 +32,9 @@
 						window.parent.close();
 					}
 				}
+				function continueReflect(){
+					 window.parent.location.href='<c:url value="/learning/newReflection.do?sessionMapID=${sessionMapID}"/>';
+				}
 				function nextIns(currIns){
 					document.location.href="<c:url value='/nextInstruction.do'/>?mode=${mode}&insIdx=" + currIns + "&sessionMapID=${sessionMapID}&itemUid=${param.itemUid}&itemIndex=${param.itemIndex}";
 				}
@@ -62,6 +67,11 @@
 									<fmt:message key='label.next.instruction'/>
 								</a>
 							</c:when>
+							<c:when test="${sessionMap.reflectOn && sessionMap.runAuto}">
+								<a href="#" id="FinishInstruction" onClick="javascript:continueReflect()" class="button">
+									<fmt:message key='label.continue'/>
+								</a>
+							</c:when>
 							<c:otherwise>
 								<a href="#" id="FinishInstruction" onClick="javascript:finishIns()" class="button">
 									<fmt:message key='label.finish'/>
Index: lams_tool_larsrc/web/pages/learning/finish.jsp
===================================================================
diff -u -rfb406a57261a31898b62f3c808ffccb853089336 -red04e2e75728d635ec5c4530bcebe69ab32d9cdb
--- lams_tool_larsrc/web/pages/learning/finish.jsp	(.../finish.jsp)	(revision fb406a57261a31898b62f3c808ffccb853089336)
+++ lams_tool_larsrc/web/pages/learning/finish.jsp	(.../finish.jsp)	(revision ed04e2e75728d635ec5c4530bcebe69ab32d9cdb)
@@ -8,5 +8,6 @@
 	-->        
     </script>
 	
+	
 	<body>
 </html>
Index: lams_tool_larsrc/web/pages/learning/learning.jsp
===================================================================
diff -u -re76c3706b6db1d6af4fc1c15a0afabbf2ad04d4d -red04e2e75728d635ec5c4530bcebe69ab32d9cdb
--- lams_tool_larsrc/web/pages/learning/learning.jsp	(.../learning.jsp)	(revision e76c3706b6db1d6af4fc1c15a0afabbf2ad04d4d)
+++ lams_tool_larsrc/web/pages/learning/learning.jsp	(.../learning.jsp)	(revision ed04e2e75728d635ec5c4530bcebe69ab32d9cdb)
@@ -8,14 +8,13 @@
 	<%@ 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>
 	<c:if test="${not empty param.sessionMapID}">
 		<c:set var="sessionMapID" value="${param.sessionMapID}" />
 	</c:if>
 
 	<c:set var="sessionMap" value="${sessionScope[sessionMapID]}" />
+	
+	<c:set var="mode" value="${sessionMap.mode}" />
 	<c:set var="toolSessionID" value="${sessionMap.toolSessionID}"/>
 	<c:set var="resource" value="${sessionMap.resource}"/>
 	<c:set var="finishedLock" value="${sessionMap.finishedLock}"/>
@@ -51,12 +50,18 @@
 			document.location.href ='<c:url value="/learning/finish.do?sessionMapID=${sessionMapID}&mode=${mode}&toolSessionID=${toolSessionID}"/>';
 			return false;
 		}
+		function continueReflect(){
+			document.location.href='<c:url value="/learning/newReflection.do?sessionMapID=${sessionMapID}"/>';
+		}
+		
 		function showMessage(url) {
 			var area=document.getElementById("reourceInputArea");
-			area.style.width="100%";
-			area.style.height="100%";
-			area.src=url;
-			area.style.display="block";
+			if(area != null){
+				area.style.width="100%";
+				area.style.height="100%";
+				area.src=url;
+				area.style.display="block";
+			}
 		}
 	-->        
     </script>
@@ -133,15 +138,24 @@
 			</div>
 			<c:if test="${mode != 'teacher'}">
 				<div class="right-buttons">
-					<html:button property="FinishButton"  onclick="return finishSession()" disabled="${finishedLock}"  styleClass="button">
-						<fmt:message key="label.finished" />
-					</html:button>
+					<c:choose>
+						<c:when test="${sessionMap.reflectOn}">
+							<html:button property="FinishButton"  onclick="return continueReflect()" disabled="${finishedLock}"  styleClass="button">
+								<fmt:message key="label.continue" />
+							</html:button>
+						</c:when>
+						<c:otherwise>
+							<html:button property="FinishButton"  onclick="return finishSession()" disabled="${finishedLock}"  styleClass="button">
+								<fmt:message key="label.finished" />
+							</html:button>
+						</c:otherwise>
+					</c:choose>
 				</div>
 			</c:if>
-			<P>&nbsp;</P>
+			<div style="height: 70px"></div> 
 			<c:if test="${mode != 'teacher' && (not finishedLock)}">
 				<c:if test="${resource.allowAddFiles || resource.allowAddUrls}">
-					<table border="0" align="center" width="100%">
+					<table border="0" width="100%">
 						<tr>
 							<td style="align:left">
 								<fmt:message key="label.suggest.new" />
Index: lams_tool_larsrc/web/pages/learning/notebook.jsp
===================================================================
diff -u
--- lams_tool_larsrc/web/pages/learning/notebook.jsp	(revision 0)
+++ lams_tool_larsrc/web/pages/learning/notebook.jsp	(revision ed04e2e75728d635ec5c4530bcebe69ab32d9cdb)
@@ -0,0 +1,59 @@
+<%@ include file="/common/taglibs.jsp"%>
+<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01//EN"
+        "http://www.w3.org/TR/html4/strict.dtd">
+
+<%@ include file="/common/taglibs.jsp"%>
+<html:html locale="true">
+<head>
+	<title><fmt:message key="label.learning.title" /></title>
+	<%@ include file="/common/header.jsp"%>
+</head>
+<body>
+	<div id="page-learner">
+		<c:set var="sessionMapID" value="${param.sessionMapID}"/>
+		<c:set var="sessionMap" value="${sessionScope[sessionMapID]}"/>
+		
+		<html:form action="/learning/submitReflection" method="post">
+			<html:hidden property="userID" />
+			<html:hidden property="sessionMapID"/>
+			<h1 class="no-tabs-below">
+				${sessionMap.title}
+			</h1>
+			<div id="header-no-tabs-learner"></div>
+			<div id="content-learner">
+				<table>
+					<tr>
+						<td>
+							<%@ include file="/common/messages.jsp"%>
+						</td>
+					</tr>
+					<tr>
+						<td>
+							${sessionMap.reflectInstructions}
+						</td>
+					</tr>
+		
+					<tr>
+						<td>
+							<lams:STRUTS-textarea cols="66" rows="8" property="entryText"/>
+						</td>
+					</tr>
+		
+					<tr>
+						<td class="right-buttons">
+							<html:submit styleClass="button">
+								<fmt:message key="label.finish"/>
+							</html:submit>
+						</td>
+					</tr>
+				</table>
+			</div>
+		</html:form>
+		
+		<div id="footer-learner">
+		</div>
+		<!--closes footer-->
+
+	</div><!--closes page-->
+</body>
+</html:html>
Index: lams_tool_larsrc/web/pages/learning/runoffline.jsp
===================================================================
diff -u -r7d117b285f81648e6f16ad1e70cc84bcb59d80e4 -red04e2e75728d635ec5c4530bcebe69ab32d9cdb
--- lams_tool_larsrc/web/pages/learning/runoffline.jsp	(.../runoffline.jsp)	(revision 7d117b285f81648e6f16ad1e70cc84bcb59d80e4)
+++ lams_tool_larsrc/web/pages/learning/runoffline.jsp	(.../runoffline.jsp)	(revision ed04e2e75728d635ec5c4530bcebe69ab32d9cdb)
@@ -1,4 +1,6 @@
 <%@include file="/common/taglibs.jsp"%>
+<c:set var="sessionMap" value="${sessionScope[sessionMapID]}"/>
+
 <!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
 <html:html locale="true">
 <head>
@@ -7,23 +9,38 @@
 	<script type="text/javascript">
 	<!--
 		function finishSession(){
-			document.location.href ='<c:url value="/learning/finish.do?toolSessionID=${toolSessionID}&runOffline=true"/>';
+			document.location.href ='<c:url value="/learning/finish.do?sessionMapID=${sessionMapID}&runOffline=true"/>';
 			return false;
 		}
+		function continueReflect(){
+			document.location.href='<c:url value="/learning/newReflection.do?sessionMapID=${sessionMapID}"/>';
+		}
+		
 	-->        
     </script>
 </head>
 
 <body>
 	<h1>
-		${resource.title}
+		${sessionMap.title}
 	</h1>
 	<div align="center">
 		<h2>
 			<fmt:message key="run.offline.message" />
 		</h2>
 		<p>
-		<a href="#" class="button" onclick="return finishSession()"> <fmt:message key="label.finished" /> </a>
+		<c:choose>
+			<c:when test="${sessionMap.reflectOn}">
+				<html:button property="FinishButton"  onclick="return continueReflect()" disabled="${sessionMap.finishedLock}"  styleClass="button">
+					<fmt:message key="label.continue" />
+				</html:button>
+			</c:when>
+			<c:otherwise>
+				<html:button property="FinishButton"  onclick="return finishSession()" disabled="${sessionMap.finishedLock}"  styleClass="button">
+					<fmt:message key="label.finished" />
+				</html:button>
+			</c:otherwise>
+		</c:choose>		
 	</div>
 
 
Index: lams_tool_larsrc/web/pages/learning/start.jsp
===================================================================
diff -u -r92e8b8d1454cfbd9ae8a20934a677340dac3b507 -red04e2e75728d635ec5c4530bcebe69ab32d9cdb
--- lams_tool_larsrc/web/pages/learning/start.jsp	(.../start.jsp)	(revision 92e8b8d1454cfbd9ae8a20934a677340dac3b507)
+++ lams_tool_larsrc/web/pages/learning/start.jsp	(.../start.jsp)	(revision ed04e2e75728d635ec5c4530bcebe69ab32d9cdb)
@@ -1,9 +1,10 @@
 <%@ include file="/common/taglibs.jsp"%>
+<c:set var="sessionMap" value="${sessionScope[sessionMapID]}"/>
 <html>
 	<body>
 
 		<c:choose>
-			<c:when test="${runAuto}">
+			<c:when test="${sessionMap.runAuto}">
 				<script type="text/javascript">
 					document.location = "<c:url value="/reviewItem.do"/>?sessionMapID=${sessionMapID}&mode=${mode}&toolSessionID=${toolSessionID}&itemUid=${itemUid}";
 				</script>
Index: lams_tool_larsrc/web/pages/monitoring/notebook.jsp
===================================================================
diff -u
--- lams_tool_larsrc/web/pages/monitoring/notebook.jsp	(revision 0)
+++ lams_tool_larsrc/web/pages/monitoring/notebook.jsp	(revision ed04e2e75728d635ec5c4530bcebe69ab32d9cdb)
@@ -0,0 +1,59 @@
+<%@ include file="/common/taglibs.jsp"%>
+<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01//EN"
+        "http://www.w3.org/TR/html4/strict.dtd">
+
+<%@ include file="/common/taglibs.jsp"%>
+<html:html locale="true">
+<head>
+	<title><fmt:message key="label.learning.title" /></title>
+	<%@ include file="/common/header.jsp"%>
+</head>
+<body>
+	<div id="page-learner">
+			<h1 class="no-tabs-below">
+				<fmt:message key="page.title.monitoring.view.reflection"/>
+			</h1>
+			<div id="header-no-tabs-learner"></div>
+			<div id="content-learner">
+				<table>
+					<tr>
+						<td>
+							<h1>
+								${userDTO.fullName}
+							</h1>
+						</td>
+					</tr>
+					<tr>
+						<td>
+							<h1>
+								${userDTO.reflectInstrctions}
+							</h1>
+						</td>
+					</tr>
+					<tr>
+						<td>
+							<c:choose>
+								<c:when test="${userDTO.finishReflection}">
+									<c:out value="${userDTO.reflect}" escapeXml="false"/>
+								</c:when>
+								<c:otherwise>
+									<fmt:message key="message.not.avaliable" />
+								</c:otherwise>
+							</c:choose>
+						</td>
+					</tr>
+				</table>
+				<table cellpadding="0">
+					<tr>
+						<td>
+							<a href="javascript:window.close();" class="button"><fmt:message key="button.close"/></a>
+						</td>
+					</tr>
+				</table>
+			</div>
+		<div id="footer-learner">
+		</div>
+		<!--closes footer-->
+	</div><!--closes page-->
+</body>
+</html:html>
Index: lams_tool_larsrc/web/pages/monitoring/summary.jsp
===================================================================
diff -u -r71cccb6c7558909803c2886472385427cdc4a281 -red04e2e75728d635ec5c4530bcebe69ab32d9cdb
--- lams_tool_larsrc/web/pages/monitoring/summary.jsp	(.../summary.jsp)	(revision 71cccb6c7558909803c2886472385427cdc4a281)
+++ lams_tool_larsrc/web/pages/monitoring/summary.jsp	(.../summary.jsp)	(revision ed04e2e75728d635ec5c4530bcebe69ab32d9cdb)
@@ -13,12 +13,15 @@
 		<c:set var="groupSize" value="${fn:length(group)}" />
 		<c:forEach var="item" items="${group}" varStatus="status">
 			<%-- display group name on first row--%>
-			<c:if test="${status.index == 0}">
+			<c:if test="${status.first}">
 				<tr>
 					<td colspan="5">
-						<B><fmt:message key="monitoring.label.group" /> ${item.sessionName}</B> <SPAN style="font-size: 12px;"> <c:if test="${firstGroup.index==0}">
+						<B><fmt:message key="monitoring.label.group" /> ${item.sessionName}</B> 
+						<SPAN style="font-size: 12px;"> 
+							<c:if test="${firstGroup.index==0}">
 								<fmt:message key="monitoring.summary.note" />
-							</c:if> </SPAN>
+							</c:if> 
+						</SPAN>
 					</td>
 				</tr>
 				<tr>
@@ -84,8 +87,8 @@
 								<a href="#" onclick="launchPopup('${listUrl}','listuser')"> ${item.viewNumber}<a>
 							</c:when>
 							<c:otherwise>
-												0
-											</c:otherwise>
+									0
+							</c:otherwise>
 						</c:choose>
 					</td>
 					<td align="center">
@@ -100,6 +103,49 @@
 					</td>
 				</tr>
 			</c:if>
+			
+				<%-- Reflection list  --%>
+				<c:if test="${sessionMap.resource.reflectOnActivity && status.last}">
+					<c:set var="userList" value="${sessionMap.reflectList[item.sessionId]}"/>
+					<c:forEach var="user" items="${userList}" varStatus="refStatus">
+						<c:if test="${refStatus.first}">
+							<tr>
+								<td colspan="5">
+									<h2><fmt:message key="title.reflection"/>	</h2>
+								</td>
+							</tr>
+							<tr>
+								<th colspan="2">
+									<fmt:message key="monitoring.user.fullname"/>
+								</th>
+								<th colspan="2">
+									<fmt:message key="monitoring.label.user.loginname"/>
+								</th>
+								<th>
+									<fmt:message key="monitoring.user.reflection"/>
+								</th>
+							</tr>
+						</c:if>
+						<tr>
+							<td colspan="2">
+								${user.fullName}
+							</td>
+							<td colspan="2">
+								${user.loginName}
+							</td>
+							<td >
+								<c:set var="viewReflection">
+									<c:url value="/monitoring/viewReflection.do?toolSessionID=${item.sessionId}&userUid=${user.userUid}"/>
+								</c:set>
+								<html:link href="javascript:launchPopup('${viewReflection}')">
+									<fmt:message key="label.view" />
+								</html:link>
+							</td>
+						</tr>
+					</c:forEach>
+				</c:if>
+			
 		</c:forEach>
+		
 	</c:forEach>
 </table>