Index: lams_tool_survey/conf/language/lams/ApplicationResources_en_AU.properties
===================================================================
diff -u -rca0f8f67547691b7a216d696046b36ae086fd619 -r098d686653002409f33aff0f69b1183e07c1eab3
--- lams_tool_survey/conf/language/lams/ApplicationResources_en_AU.properties	(.../ApplicationResources_en_AU.properties)	(revision ca0f8f67547691b7a216d696046b36ae086fd619)
+++ lams_tool_survey/conf/language/lams/ApplicationResources_en_AU.properties	(.../ApplicationResources_en_AU.properties)	(revision 098d686653002409f33aff0f69b1183e07c1eab3)
@@ -47,8 +47,6 @@
 error.survey.item.question.blank  =Question can not be blank
 error.survey.item.question.less.option  =You must input at least two options.
 label.submit.survey  =Done
-label.retake.survey  =Retake survey
-label.retake  =[Retake]
 label.append.text  =Append text:
 label.next  =Next
 label.previous  =Previous
@@ -137,8 +135,6 @@
 label.monitoring.button.export.excel  =Export Report
 errors.maxfilesize  =The uploaded file has exceeded the maximum file size limit of {0} bytes
 message.not.available  =Not Available
-message.warnLockOnFinish  =Note: After you click on "Next Activity" and you come back to this Survey, you won't be able to change your answers.
-message.activityLocked  == The instructor has set this activity not to allow you to retake the survey after you have finished it. As you are returning to this activity, you are able to see your answers but not allowed to change them.
 label.export.reflection  =Notebook entries
 label.on  =On
 label.off  =Off
@@ -181,5 +177,15 @@
 label.submit  =Finish
 label.monitoring.heading  =Survey Monitoring
 
+label.show.answers.from.other.users	 =Show answers from other learners 
+label.view.all.responses  =View All Responses
+label.sort.by.answer  =Sort by answer
+label.other.answers  =Answers from other Learners
+label.total.responses  =Total responses so far: {0}
 
+label.retake.survey  =Change answers
+label.retake  =[Change answer]
+message.activityLocked  =The instructor has set this activity not to allow you to retake the survey after you have finished it. As you are returning to this activity, you are able to see your answers but not allowed to change them.
+message.warnLockOnFinish  =Note: After you click on "{0}" and you come back to this Survey, you won't be able to change your answers.
+
 #======= End labels: Exported 174 labels for en AU =====
Index: lams_tool_survey/conf/xdoclet/struts-actions.xml
===================================================================
diff -u -r8b97231e320c0c5b674f07c14da711f232ba9e1c -r098d686653002409f33aff0f69b1183e07c1eab3
--- lams_tool_survey/conf/xdoclet/struts-actions.xml	(.../struts-actions.xml)	(revision 8b97231e320c0c5b674f07c14da711f232ba9e1c)
+++ lams_tool_survey/conf/xdoclet/struts-actions.xml	(.../struts-actions.xml)	(revision 098d686653002409f33aff0f69b1183e07c1eab3)
@@ -157,6 +157,7 @@
 			<forward name="definelater" path="/pages/learning/definelater.jsp" />
 			<forward name="submissionDeadline" path="/pages/learning/submissionDeadline.jsp" />	
 			<forward name="result" path="/pages/learning/result.jsp" />	
+			<forward name="resultOtherUsers" path="/learning/showOtherUsersAnswers.do" />
 		</action>
 		
 		<action path="/learning/nextQuestion"
@@ -188,6 +189,23 @@
 			<forward name="success" path="/pages/learning/learning.jsp" />
 		</action>
 		
+		<action path="/learning/showOtherUsersAnswers"
+			type="org.lamsfoundation.lams.tool.survey.web.action.LearningAction"
+			input="/pages/learning/learning.jsp"
+			scope="request" 
+			validate="false"			
+			parameter="showOtherUsersAnswers" >
+			<forward name="success" path="/pages/learning/resultOtherUsers.jsp" />
+		</action>
+		
+		<action path="/learning/getOpenResponses"
+			type="org.lamsfoundation.lams.tool.survey.web.action.LearningAction"
+			input="/pages/learning/learning.jsp"
+			scope="request" 
+			validate="false"			
+			parameter="getOpenResponses" >
+		</action>
+		
 		<action path="/learning/doSurvey"
 			type="org.lamsfoundation.lams.tool.survey.web.action.LearningAction"
 			input="/pages/learning/learning.jsp"
Index: lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/SurveyConstants.java
===================================================================
diff -u -r8b97231e320c0c5b674f07c14da711f232ba9e1c -r098d686653002409f33aff0f69b1183e07c1eab3
--- lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/SurveyConstants.java	(.../SurveyConstants.java)	(revision 8b97231e320c0c5b674f07c14da711f232ba9e1c)
+++ lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/SurveyConstants.java	(.../SurveyConstants.java)	(revision 098d686653002409f33aff0f69b1183e07c1eab3)
@@ -62,6 +62,8 @@
     public static final String PARAM_TITLE = "title";
 
     public static final String ATTR_USER_UID = "userUid";
+    
+    public static final String ATTR_USER = "user";
 
     // error message keys
     public static final String ERROR_MSG_TITLE_BLANK = "error.survey.item.title.blank";
@@ -143,6 +145,8 @@
     public static final String FORWARD_CHOICE_QUESTION = "question";
 
     public static final String ATTR_SHOW_ON_ONE_PAGE = "showOnOnePage";
+    
+    public static final String ATTR_SHOW_OTHER_USERS_ANSWERS = "showOtherUsersAnswers";
 
     public static final String PREFIX_QUESTION_CHOICE = "optionChoice";
 
@@ -164,6 +168,12 @@
     
     public static final String ATTR_IS_GROUPED_ACTIVITY = "isGroupedActivity";
     
+    public static final String ATTR_COUNT_FINISHED_USERS = "countFinishedUser";
+    
+    public static final int SORT_BY_DEAFAULT = 0;
+    public static final int SORT_BY_ANSWER_ASC = 1;
+    public static final int SORT_BY_ANSWER_DESC = 2;
+    
     // POSITION
     public static int POSITION_INSIDE = 0;
 
Index: lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/dao/SurveyAnswerDAO.java
===================================================================
diff -u -r15045d0b9b0dbb930f42a9da7757207e7a325341 -r098d686653002409f33aff0f69b1183e07c1eab3
--- lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/dao/SurveyAnswerDAO.java	(.../SurveyAnswerDAO.java)	(revision 15045d0b9b0dbb930f42a9da7757207e7a325341)
+++ lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/dao/SurveyAnswerDAO.java	(.../SurveyAnswerDAO.java)	(revision 098d686653002409f33aff0f69b1183e07c1eab3)
@@ -24,18 +24,26 @@
 
 import java.util.List;
 
+import org.lamsfoundation.lams.tool.survey.SurveyConstants;
 import org.lamsfoundation.lams.tool.survey.model.SurveyAnswer;
 
 public interface SurveyAnswerDAO extends DAO {
 
-	SurveyAnswer getAnswer(Long uid, Long userUid);
-	
-	/**
-	 * Get all answers from same session for same question. 
-	 * @param sessionId
-	 * @param questionUid
-	 * @return
-	 */
-	List<SurveyAnswer> getSessionAnswer(Long sessionId,Long questionUid);
-	List<SurveyAnswer> getByToolContentIdAndUserId(Long toolContentId, Long userId);
+    SurveyAnswer getAnswer(Long uid, Long userUid);
+
+    /**
+     * Get all answers from same session for same question.
+     * 
+     * @param sessionId
+     * @param questionUid
+     * @return
+     */
+    List<SurveyAnswer> getSessionAnswer(Long sessionId, Long questionUid);
+
+    List<SurveyAnswer> getByToolContentIdAndUserId(Long toolContentId, Long userId);
+
+    List<String> getOpenResponsesForTablesorter(final Long sessionId, final Long questionUid, int page, int size,
+	    int sorting);
+
+    int getCountResponsesBySessionAndQuestion(final Long sessionId, final Long questionId);
 }
Index: lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/dao/SurveyUserDAO.java
===================================================================
diff -u -rf0c3f41d54ef4f390ef3f93f28c0679c46b6730c -r098d686653002409f33aff0f69b1183e07c1eab3
--- lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/dao/SurveyUserDAO.java	(.../SurveyUserDAO.java)	(revision f0c3f41d54ef4f390ef3f93f28c0679c46b6730c)
+++ lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/dao/SurveyUserDAO.java	(.../SurveyUserDAO.java)	(revision 098d686653002409f33aff0f69b1183e07c1eab3)
@@ -29,9 +29,11 @@
 
 public interface SurveyUserDAO extends DAO {
 
-	SurveyUser getUserByUserIDAndSessionID(Long userID, Long sessionId);
+    SurveyUser getUserByUserIDAndSessionID(Long userID, Long sessionId);
 
-	SurveyUser getUserByUserIDAndContentID(Long userId, Long contentId);
+    SurveyUser getUserByUserIDAndContentID(Long userId, Long contentId);
 
-	List<SurveyUser> getBySessionID(Long sessionId);
+    List<SurveyUser> getBySessionID(Long sessionId);
+    
+    int getCountFinishedUsers(Long sessionId);
 }
Index: lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/dao/hibernate/SurveyAnswerDAOHibernate.java
===================================================================
diff -u -r2aaccb954583cd8431759dca8e912a8fe863aa0e -r098d686653002409f33aff0f69b1183e07c1eab3
--- lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/dao/hibernate/SurveyAnswerDAOHibernate.java	(.../SurveyAnswerDAOHibernate.java)	(revision 2aaccb954583cd8431759dca8e912a8fe863aa0e)
+++ lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/dao/hibernate/SurveyAnswerDAOHibernate.java	(.../SurveyAnswerDAOHibernate.java)	(revision 098d686653002409f33aff0f69b1183e07c1eab3)
@@ -24,31 +24,85 @@
 
 import java.util.List;
 
+import org.hibernate.HibernateException;
+import org.hibernate.Session;
+import org.lamsfoundation.lams.tool.survey.SurveyConstants;
 import org.lamsfoundation.lams.tool.survey.dao.SurveyAnswerDAO;
 import org.lamsfoundation.lams.tool.survey.model.SurveyAnswer;
+import org.springframework.orm.hibernate4.HibernateCallback;
 
 public class SurveyAnswerDAOHibernate extends BaseDAOHibernate implements SurveyAnswerDAO {
-	private static final String GET_LEARNER_ANSWER = "from "+SurveyAnswer.class.getName()+" as a where a.surveyQuestion.uid=? and a.user.uid=?";
-	private static final String GET_SESSION_ANSWER = "from "+SurveyAnswer.class.getName()+" as a " +
-							" where a.user.session.sessionId=? and a.surveyQuestion.uid=?";
-	private static final String GET_BY_TOOL_CONTENT_ID_AND_USER_ID = "from "+SurveyAnswer.class.getName()+" as a " +
-		" where a.user.session.survey.contentId = ? and a.user.userId = ?";
-	
-	public SurveyAnswer getAnswer(Long questionUid, Long userUid) {
-		List list = getHibernateTemplate().find(GET_LEARNER_ANSWER,new Object[]{questionUid,userUid});
-		if(list.size() > 0)
-			return (SurveyAnswer) list.get(0);
-		else
-			return null;
-	}
+    private static final String GET_LEARNER_ANSWER = "FROM " + SurveyAnswer.class.getName()
+	    + " AS a WHERE a.surveyQuestion.uid=? AND a.user.uid=?";
+    private static final String GET_SESSION_ANSWER = "FROM " + SurveyAnswer.class.getName() + " AS a "
+	    + " WHERE a.user.session.sessionId=? AND a.surveyQuestion.uid=?";
+    private static final String GET_BY_TOOL_CONTENT_ID_AND_USER_ID = "FROM " + SurveyAnswer.class.getName() + " AS a "
+	    + " WHERE a.user.session.survey.contentId = ? AND a.user.userId = ?";
 
-	@SuppressWarnings("unchecked")
-	public List<SurveyAnswer> getSessionAnswer(Long sessionId, Long questionUid) {
-		return (List<SurveyAnswer>) getHibernateTemplate().find(GET_SESSION_ANSWER,new Object[]{sessionId,questionUid});
-	}
-	
+    private static final String LOAD_ATTEMPT_FOR_SESSION_AND_QUESTION_LIMIT = "SELECT r.answerText FROM "
+	    + SurveyAnswer.class.getName()
+	    + " AS r "
+	    + "WHERE r.user.session.sessionId=:sessionId AND r.surveyQuestion.uid=:questionUid AND r.answerText<>'' order by ";
+
+    private static final String GET_COUNT_RESPONSES_FOR_SESSION_AND_QUESTION = "SELECT COUNT(*) FROM "
+	    + SurveyAnswer.class.getName() + " AS r "
+	    + "WHERE r.user.session.sessionId=? AND r.surveyQuestion.uid=? AND r.answerText<>''";
+
+    @Override
+    public SurveyAnswer getAnswer(Long questionUid, Long userUid) {
+	List list = getHibernateTemplate().find(GET_LEARNER_ANSWER, new Object[] { questionUid, userUid });
+	if (list.size() > 0)
+	    return (SurveyAnswer) list.get(0);
+	else
+	    return null;
+    }
+
     @SuppressWarnings("unchecked")
+    public List<SurveyAnswer> getSessionAnswer(Long sessionId, Long questionUid) {
+	return (List<SurveyAnswer>) getHibernateTemplate().find(GET_SESSION_ANSWER,
+		new Object[] { sessionId, questionUid });
+    }
+
+    @SuppressWarnings("unchecked")
     public List<SurveyAnswer> getByToolContentIdAndUserId(Long toolContentId, Long userId) {
-	return (List<SurveyAnswer>) getHibernateTemplate().find(GET_BY_TOOL_CONTENT_ID_AND_USER_ID, new Object[] { toolContentId, userId });
+	return (List<SurveyAnswer>) getHibernateTemplate().find(GET_BY_TOOL_CONTENT_ID_AND_USER_ID,
+		new Object[] { toolContentId, userId });
     }
+
+    @Override
+    public List<String> getOpenResponsesForTablesorter(final Long sessionId, final Long questionUid, final int page,
+	    final int size, final int sorting) {
+	String sortingOrder = "";
+	switch (sorting) {
+	case SurveyConstants.SORT_BY_DEAFAULT:
+	    sortingOrder = "r.updateDate";
+	    break;
+	case SurveyConstants.SORT_BY_ANSWER_ASC:
+	    sortingOrder = "r.answerText ASC";
+	    break;
+	case SurveyConstants.SORT_BY_ANSWER_DESC:
+	    sortingOrder = "r.answerText DESC";
+	    break;
+	}
+	final String sqlQuery = LOAD_ATTEMPT_FOR_SESSION_AND_QUESTION_LIMIT + sortingOrder;
+
+	return (List<String>) getHibernateTemplate().execute(new HibernateCallback() {
+	    public Object doInHibernate(Session session) throws HibernateException {
+		return session.createQuery(sqlQuery).setLong("sessionId", sessionId.longValue())
+			.setLong("questionUid", questionUid.longValue()).setFirstResult(page * size)
+			.setMaxResults(size).list();
+	    }
+	});
+    }
+
+    @Override
+    public int getCountResponsesBySessionAndQuestion(final Long sessionId, final Long questionId) {
+
+	List list = getHibernateTemplate().find(GET_COUNT_RESPONSES_FOR_SESSION_AND_QUESTION,
+		new Object[] { sessionId, questionId });
+	if (list == null || list.size() == 0) {
+	    return 0;
+	}
+	return ((Number) list.get(0)).intValue();
+    }
 }
Index: lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/dao/hibernate/SurveyUserDAOHibernate.java
===================================================================
diff -u -r2aaccb954583cd8431759dca8e912a8fe863aa0e -r098d686653002409f33aff0f69b1183e07c1eab3
--- lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/dao/hibernate/SurveyUserDAOHibernate.java	(.../SurveyUserDAOHibernate.java)	(revision 2aaccb954583cd8431759dca8e912a8fe863aa0e)
+++ lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/dao/hibernate/SurveyUserDAOHibernate.java	(.../SurveyUserDAOHibernate.java)	(revision 098d686653002409f33aff0f69b1183e07c1eab3)
@@ -28,31 +28,43 @@
 import org.lamsfoundation.lams.tool.survey.dao.SurveyUserDAO;
 import org.lamsfoundation.lams.tool.survey.model.SurveyUser;
 
+public class SurveyUserDAOHibernate extends BaseDAOHibernate implements SurveyUserDAO {
 
-public class SurveyUserDAOHibernate extends BaseDAOHibernate implements SurveyUserDAO{
-	
-	private static final String FIND_BY_USER_ID_CONTENT_ID = "from " + SurveyUser.class.getName() + " as u where u.userId =? and u.survey.contentId=?";
-	private static final String FIND_BY_USER_ID_SESSION_ID = "from " + SurveyUser.class.getName() + " as u where u.userId =? and u.session.sessionId=?";
-	private static final String FIND_BY_SESSION_ID = "from " + SurveyUser.class.getName() + " as u where u.session.sessionId=?";
+    private static final String FIND_BY_USER_ID_CONTENT_ID = "FROM " + SurveyUser.class.getName()
+	    + " AS u WHERE u.userId =? AND u.survey.contentId=?";
+    private static final String FIND_BY_USER_ID_SESSION_ID = "FROM " + SurveyUser.class.getName()
+	    + " AS u WHERE u.userId =? AND u.session.sessionId=?";
+    private static final String FIND_BY_SESSION_ID = "FROM " + SurveyUser.class.getName()
+	    + " AS u WHERE u.session.sessionId=?";
+    private static final String GET_COUNT_FINISHED_USERS_FOR_SESSION = "SELECT COUNT(*) FROM "
+	    + SurveyUser.class.getName()
+	    + " AS u WHERE u.session.sessionId=? AND (u.sessionFinished is true OR u.responseFinalized is true)";
 
-	public SurveyUser getUserByUserIDAndSessionID(Long userID, Long sessionId) {
-		List list = this.getHibernateTemplate().find(FIND_BY_USER_ID_SESSION_ID,new Object[]{userID,sessionId});
-		if(list == null || list.size() == 0)
-			return null;
-		return (SurveyUser) list.get(0);
-	}
+    public SurveyUser getUserByUserIDAndSessionID(Long userID, Long sessionId) {
+	List list = this.getHibernateTemplate().find(FIND_BY_USER_ID_SESSION_ID, new Object[] { userID, sessionId });
+	if (list == null || list.size() == 0)
+	    return null;
+	return (SurveyUser) list.get(0);
+    }
 
-	public SurveyUser getUserByUserIDAndContentID(Long userId, Long contentId) {
-		List list = this.getHibernateTemplate().find(FIND_BY_USER_ID_CONTENT_ID,new Object[]{userId,contentId});
-		if(list == null || list.size() == 0)
-			return null;
-		return (SurveyUser) list.get(0);
-	}
+    public SurveyUser getUserByUserIDAndContentID(Long userId, Long contentId) {
+	List list = this.getHibernateTemplate().find(FIND_BY_USER_ID_CONTENT_ID, new Object[] { userId, contentId });
+	if (list == null || list.size() == 0)
+	    return null;
+	return (SurveyUser) list.get(0);
+    }
 
-	@SuppressWarnings("unchecked")
-	public List<SurveyUser> getBySessionID(Long sessionId) {
-		return (List<SurveyUser>) this.getHibernateTemplate().find(FIND_BY_SESSION_ID,sessionId);
+    @SuppressWarnings("unchecked")
+    public List<SurveyUser> getBySessionID(Long sessionId) {
+	return (List<SurveyUser>) this.getHibernateTemplate().find(FIND_BY_SESSION_ID, sessionId);
+    }
+
+    public int getCountFinishedUsers(Long sessionId) {
+	List list = getHibernateTemplate().find(GET_COUNT_FINISHED_USERS_FOR_SESSION, new Object[] { sessionId });
+	if (list == null || list.size() == 0) {
+	    return 0;
 	}
+	return ((Number) list.get(0)).intValue();
+    }
 
-
 }
Index: lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/dbupdates/patch20141023.sql
===================================================================
diff -u
--- lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/dbupdates/patch20141023.sql	(revision 0)
+++ lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/dbupdates/patch20141023.sql	(revision 098d686653002409f33aff0f69b1183e07c1eab3)
@@ -0,0 +1,15 @@
+-- Turn off autocommit, so nothing is committed if there is an error
+SET AUTOCOMMIT = 0;
+SET FOREIGN_KEY_CHECKS=0;
+----------------------Put all sql statements below here-------------------------
+
+-- LDEV-3326 Option to include everyone results after survey responses are sent
+ALTER TABLE tl_lasurv11_survey ADD COLUMN show_other_users_answers TINYINT(1) NOT NULL DEFAULT 0;
+ALTER TABLE tl_lasurv11_user ADD COLUMN response_finalized TINYINT(1) NOT NULL DEFAULT 0;
+
+----------------------Put all sql statements above here-------------------------
+
+-- If there were no errors, commit and restore autocommit to on
+COMMIT;
+SET AUTOCOMMIT = 1;
+SET FOREIGN_KEY_CHECKS=1;
\ No newline at end of file
Index: lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/dto/AnswerDTO.java
===================================================================
diff -u -r103eacc68afe9e0df0c7bae49c482eb34873165a -r098d686653002409f33aff0f69b1183e07c1eab3
--- lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/dto/AnswerDTO.java	(.../AnswerDTO.java)	(revision 103eacc68afe9e0df0c7bae49c482eb34873165a)
+++ lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/dto/AnswerDTO.java	(.../AnswerDTO.java)	(revision 098d686653002409f33aff0f69b1183e07c1eab3)
@@ -35,83 +35,91 @@
 import org.lamsfoundation.lams.tool.survey.model.SurveyUser;
 import org.lamsfoundation.lams.tool.survey.util.SurveyOptionComparator;
 
+public class AnswerDTO extends SurveyQuestion {
 
+    private static final int SHORT_TITLE_LENGTH = 60;
 
-public class AnswerDTO extends SurveyQuestion{
+    // ***********************************************
+    // DTO fields:
+    // this is DTO field. For answer, which is user and session level. For question, which is content level.
+    private SurveyAnswer answer;
+    // Open text entry response percentage if this question has open text entry.
+    private double openResponse;
+    private String openResponseFormatStr;
+    private int openResponseCount;
 
-	private static final int SHORT_TITLE_LENGTH = 60;
-	
-	//***********************************************
-	//DTO fields:
-	//this is DTO field. For answer, which is user and session level. For question, which is content level.
-	private SurveyAnswer answer;
-	//Open text entry response percentage  if this question has open text entry.
-	private double openResponse; 
-	private String openResponseFormatStr;
-	private int openResponseCount; 
-	
-	//this field could have value even this user does not reply this question (the answer is null)
-	private SurveyUser replier;
-	
-	static Logger logger = Logger.getLogger(AnswerDTO.class.getName());
-	public AnswerDTO(SurveyQuestion question){
-		try {
-			PropertyUtils.copyProperties(this, question);
-		} catch (Exception e) {
-			logger.error("Error occurs during creating AnswerDTO");
-		}
-		
-		//clone options 
-		Set<SurveyOption> optList = question.getOptions();
-		if(optList != null){
-			SortedSet<SurveyOption> newOptions = new TreeSet<SurveyOption>(new SurveyOptionComparator());
-			for (SurveyOption option : optList) {
-				SurveyOption newOption = (SurveyOption) option.clone();
-				//clone does not copy the UID, here copy it back
-				newOption.setUid(option.getUid());
-				newOptions.add(newOption);
-			}
-			this.setOptions(newOptions);
-		}
-		
-		String desc = this.getDescription();
-		desc = desc.replaceAll("<(.|\n)*?>", "");
-		this.setShortTitle(StringUtils.abbreviate(desc,SHORT_TITLE_LENGTH));
+    // this field could have value even this user does not reply this question (the answer is null)
+    private SurveyUser replier;
 
+    private static Logger logger = Logger.getLogger(AnswerDTO.class.getName());
+
+    public AnswerDTO(SurveyQuestion question) {
+	try {
+	    PropertyUtils.copyProperties(this, question);
+	} catch (Exception e) {
+	    logger.error("Error occurs during creating AnswerDTO");
 	}
-	//****************************************************************
-	// DTO fields
-	//****************************************************************
-	public SurveyAnswer getAnswer() {
-		return answer;
+
+	// clone options
+	Set<SurveyOption> optList = question.getOptions();
+	if (optList != null) {
+	    SortedSet<SurveyOption> newOptions = new TreeSet<SurveyOption>(new SurveyOptionComparator());
+	    for (SurveyOption option : optList) {
+		SurveyOption newOption = (SurveyOption) option.clone();
+		// clone does not copy the UID, here copy it back
+		newOption.setUid(option.getUid());
+		newOptions.add(newOption);
+	    }
+	    this.setOptions(newOptions);
 	}
-	public void setAnswer(SurveyAnswer answer) {
-		this.answer = answer;
-	}
-	public double getOpenResponse() {
-		return openResponse;
-	}
-	public void setOpenResponse(double openResponse) {
-		this.openResponse = openResponse;
-	}
-	public int getOpenResponseCount() {
-		return openResponseCount;
-	}
-	public void setOpenResponseCount(int openResponseCount) {
-		this.openResponseCount = openResponseCount;
-	}
-	public String getOpenResponseFormatStr() {
-		return openResponseFormatStr;
-	}
-	public void setOpenResponseFormatStr(String openResponseFormatStr) {
-		this.openResponseFormatStr = openResponseFormatStr;
-	}
-	
-	public SurveyUser getReplier() {
-		return replier;
-	}
-	public void setReplier(SurveyUser replier) {
-		this.replier = replier;
-	}
 
+	String desc = this.getDescription();
+	desc = desc.replaceAll("<(.|\n)*?>", "");
+	this.setShortTitle(StringUtils.abbreviate(desc, SHORT_TITLE_LENGTH));
+
+    }
+
+    // ****************************************************************
+    // DTO fields
+    // ****************************************************************
+    public SurveyAnswer getAnswer() {
+	return answer;
+    }
+
+    public void setAnswer(SurveyAnswer answer) {
+	this.answer = answer;
+    }
+
+    public double getOpenResponse() {
+	return openResponse;
+    }
+
+    public void setOpenResponse(double openResponse) {
+	this.openResponse = openResponse;
+    }
+
+    public int getOpenResponseCount() {
+	return openResponseCount;
+    }
+
+    public void setOpenResponseCount(int openResponseCount) {
+	this.openResponseCount = openResponseCount;
+    }
+
+    public String getOpenResponseFormatStr() {
+	return openResponseFormatStr;
+    }
+
+    public void setOpenResponseFormatStr(String openResponseFormatStr) {
+	this.openResponseFormatStr = openResponseFormatStr;
+    }
+
+    public SurveyUser getReplier() {
+	return replier;
+    }
+
+    public void setReplier(SurveyUser replier) {
+	this.replier = replier;
+    }
+
 }
Index: lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/model/Survey.java
===================================================================
diff -u -r8b97231e320c0c5b674f07c14da711f232ba9e1c -r098d686653002409f33aff0f69b1183e07c1eab3
--- lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/model/Survey.java	(.../Survey.java)	(revision 8b97231e320c0c5b674f07c14da711f232ba9e1c)
+++ lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/model/Survey.java	(.../Survey.java)	(revision 098d686653002409f33aff0f69b1183e07c1eab3)
@@ -23,21 +23,16 @@
 /* $Id$ */
 package org.lamsfoundation.lams.tool.survey.model;
 
-import java.util.ArrayList;
 import java.util.Date;
 import java.util.HashSet;
 import java.util.Iterator;
-import java.util.List;
 import java.util.Set;
 import java.util.TreeSet;
 
-import org.apache.commons.lang.StringUtils;
 import org.apache.commons.lang.builder.EqualsBuilder;
 import org.apache.commons.lang.builder.HashCodeBuilder;
 import org.apache.log4j.Logger;
-import org.lamsfoundation.lams.contentrepository.client.IToolContentHandler;
 import org.lamsfoundation.lams.learningdesign.TextSearchConditionComparator;
-import org.lamsfoundation.lams.tool.survey.util.SurveyToolContentHandler;
 
 /**
  * Survey
@@ -59,6 +54,7 @@
     private String instructions;
     // advance
     private boolean showOnePage;
+    private boolean showOtherUsersAnswers;
     private boolean lockWhenFinished;
 
     private boolean reflectOnActivity;
@@ -384,7 +380,19 @@
     public void setShowOnePage(boolean showOnePage) {
 	this.showOnePage = showOnePage;
     }
+    
+    /**
+     * @hibernate.property column="show_other_users_answers"
+     * @return
+     */
+    public boolean isShowOtherUsersAnswers() {
+	return showOtherUsersAnswers;
+    }
 
+    public void setShowOtherUsersAnswers(boolean showOtherUsersAnswers) {
+	this.showOtherUsersAnswers = showOtherUsersAnswers;
+    }
+
     /**
      * @hibernate.property column="answer_submit_notify"
      * @return
Index: lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/model/SurveyAnswer.java
===================================================================
diff -u -r920894ca746cba5e080023c5cc80167d64d1653d -r098d686653002409f33aff0f69b1183e07c1eab3
--- lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/model/SurveyAnswer.java	(.../SurveyAnswer.java)	(revision 920894ca746cba5e080023c5cc80167d64d1653d)
+++ lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/model/SurveyAnswer.java	(.../SurveyAnswer.java)	(revision 098d686653002409f33aff0f69b1183e07c1eab3)
@@ -25,103 +25,111 @@
 package org.lamsfoundation.lams.tool.survey.model;
 
 import java.util.Date;
+
 /**
  * Survey
+ * 
  * @author Dapeng Ni
- *
- * @hibernate.class  table="tl_lasurv11_answer"
- *
+ * 
+ * @hibernate.class table="tl_lasurv11_answer"
+ * 
  */
 public class SurveyAnswer {
 
-	private Long uid;
-	
-	private SurveyUser user;
-	
-	private SurveyQuestion surveyQuestion;
-	//options choice string: option UIDs are conjunctioned by &. Such as 2&5&3
-	private String answerChoices;
-	private String answerText;
-	private Date updateDate;
-	
-	//************************************************
-	// DTO fields
-	//************************************************
-	//it is list of optional UIDs. Uid is long type, but here just save them by String format
-	private String[] choices;
-	
-	/**
-	 * @hibernate.many-to-one  column="question_uid"
- 	 * cascade="none"
-	 * @return
-	 */
-	public SurveyQuestion getSurveyQuestion() {
-		return surveyQuestion;
-	}
-	public void setSurveyQuestion(SurveyQuestion item) {
-		this.surveyQuestion = item;
-	}
-	
-	/**
-	 * @hibernate.id generator-class="native" type="java.lang.Long" column="uid"
-	 * @return Returns the log Uid.
-	 */
-	public Long getUid() {
-		return uid;
-	}
-	public void setUid(Long uid) {
-		this.uid = uid;
-	}
-	/**
-	 * @hibernate.many-to-one  column="user_uid"
- 	 * cascade="none"
-	 * @return
-	 */
-	public SurveyUser getUser() {
-		return user;
-	}
-	public void setUser(SurveyUser user) {
-		this.user = user;
-	}
-	/**
-	 * @hibernate.property column="answer_choices"
-	 * @return
-	 */
-	public String getAnswerChoices() {
-		return answerChoices;
-	}
-	public void setAnswerChoices(String answers) {
-		this.answerChoices = answers;
-	}
-	/**
-	 * @hibernate.property column="udpate_date"
-	 * @return
-	 */
-	public Date getUpdateDate() {
-		return updateDate;
-	}
-	public void setUpdateDate(Date updateDate) {
-		this.updateDate = updateDate;
-	}
-	/**
-	 * @hibernate.property column="answer_text"
-	 * @return
-	 */
-	public String getAnswerText() {
-		return answerText;
-	}
-	public void setAnswerText(String textEntry) {
-		this.answerText = textEntry;
-	}
-	public String[] getChoices() {
-		return choices;
-	}
-	public void setChoices(String[] choices) {
-		this.choices = choices;
-	}
-	
+    private Long uid;
 
+    private SurveyUser user;
 
-	
-	
+    private SurveyQuestion surveyQuestion;
+    // options choice string: option UIDs are conjunctioned by &. Such as 2&5&3
+    private String answerChoices;
+    private String answerText;
+    private Date updateDate;
+
+    // ************************************************
+    // DTO fields
+    // ************************************************
+    // it is list of optional UIDs. Uid is long type, but here just save them by String format
+    private String[] choices;
+
+    /**
+     * @hibernate.many-to-one column="question_uid" cascade="none"
+     * @return
+     */
+    public SurveyQuestion getSurveyQuestion() {
+	return surveyQuestion;
+    }
+
+    public void setSurveyQuestion(SurveyQuestion item) {
+	this.surveyQuestion = item;
+    }
+
+    /**
+     * @hibernate.id generator-class="native" type="java.lang.Long" column="uid"
+     * @return Returns the log Uid.
+     */
+    public Long getUid() {
+	return uid;
+    }
+
+    public void setUid(Long uid) {
+	this.uid = uid;
+    }
+
+    /**
+     * @hibernate.many-to-one column="user_uid" cascade="none"
+     * @return
+     */
+    public SurveyUser getUser() {
+	return user;
+    }
+
+    public void setUser(SurveyUser user) {
+	this.user = user;
+    }
+
+    /**
+     * @hibernate.property column="answer_choices"
+     * @return
+     */
+    public String getAnswerChoices() {
+	return answerChoices;
+    }
+
+    public void setAnswerChoices(String answers) {
+	this.answerChoices = answers;
+    }
+
+    /**
+     * @hibernate.property column="udpate_date"
+     * @return
+     */
+    public Date getUpdateDate() {
+	return updateDate;
+    }
+
+    public void setUpdateDate(Date updateDate) {
+	this.updateDate = updateDate;
+    }
+
+    /**
+     * @hibernate.property column="answer_text"
+     * @return
+     */
+    public String getAnswerText() {
+	return answerText;
+    }
+
+    public void setAnswerText(String textEntry) {
+	this.answerText = textEntry;
+    }
+
+    public String[] getChoices() {
+	return choices;
+    }
+
+    public void setChoices(String[] choices) {
+	this.choices = choices;
+    }
+
 }
Index: lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/model/SurveyOption.java
===================================================================
diff -u -r920894ca746cba5e080023c5cc80167d64d1653d -r098d686653002409f33aff0f69b1183e07c1eab3
--- lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/model/SurveyOption.java	(.../SurveyOption.java)	(revision 920894ca746cba5e080023c5cc80167d64d1653d)
+++ lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/model/SurveyOption.java	(.../SurveyOption.java)	(revision 098d686653002409f33aff0f69b1183e07c1eab3)
@@ -23,8 +23,6 @@
 /* $$Id$$ */
 package org.lamsfoundation.lams.tool.survey.model;
 
-import java.util.Set;
-
 import org.apache.log4j.Logger;
 /**
  * @hibernate.class table="tl_lasurv11_option"
Index: lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/model/SurveyQuestion.java
===================================================================
diff -u -r8b97231e320c0c5b674f07c14da711f232ba9e1c -r098d686653002409f33aff0f69b1183e07c1eab3
--- lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/model/SurveyQuestion.java	(.../SurveyQuestion.java)	(revision 8b97231e320c0c5b674f07c14da711f232ba9e1c)
+++ lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/model/SurveyQuestion.java	(.../SurveyQuestion.java)	(revision 098d686653002409f33aff0f69b1183e07c1eab3)
@@ -27,197 +27,205 @@
 import java.util.HashSet;
 import java.util.Iterator;
 import java.util.Set;
-import java.util.SortedMap;
 
 import org.apache.log4j.Logger;
 
 /**
  * Survey
+ * 
  * @author Dapeng Ni
- *
- * @hibernate.class  table="tl_lasurv11_question"
- *
+ * 
+ * @hibernate.class table="tl_lasurv11_question"
+ * 
  */
-public class SurveyQuestion  implements Cloneable{
-	private static final Logger log = Logger.getLogger(SurveyQuestion.class);
-	
-	private Long uid;
-	
-	//Survey Type:1=Single Choice,2=Multiple Choice,3=Text Entry
-	private short type;
-	
-	private String description;
-	private int sequenceId;
-	
-	//option of Question
-	private boolean appendText;
-	private boolean optional;
-	private boolean allowMultipleAnswer;
-	
-	private Set<SurveyOption> options;
-	
-	private Date createDate;
-	private SurveyUser createBy;
-	
-//	***********************************************
-	//DTO fields:
-	private String shortTitle;
-	
-    public Object clone(){
-    	SurveyQuestion obj = null;
-		try {
-			obj = (SurveyQuestion) super.clone();
-//			clone options
-  			if(options != null){
-  				Iterator iter = options.iterator();
-  				Set set = new HashSet();
-  				while(iter.hasNext()){
-  					SurveyOption instruct = (SurveyOption)iter.next(); 
-  					SurveyOption newInsruct = (SurveyOption) instruct.clone();
-					set.add(newInsruct);
-  				}
-  				obj.options = set;
-  			}
-			((SurveyQuestion)obj).setUid(null);
-  			//clone ReourceUser as well
-  			if(this.createBy != null)
-  				((SurveyQuestion)obj).setCreateBy((SurveyUser) this.createBy.clone());
-  			
-		} catch (CloneNotSupportedException e) {
-			log.error("When clone " + SurveyQuestion.class + " failed");
-		}
-		
-		return obj;
-	}	
-//    **********************************************************
-	  	//		Get/Set methods
-//	  **********************************************************
-		/**
-		 * @hibernate.id generator-class="native" type="java.lang.Long" column="uid"
-		 * @return Returns the uid.
-		 */
-		public Long getUid() {
-			return uid;
-		}
-		/**
-		 * @param uid The uid to set.
-		 */
-		public void setUid(Long userID) {
-			this.uid = userID;
-		}
-		
-		/**
-		 * @hibernate.property column="description" type="text"
-		 * @return
-		 */
-		public String getDescription() {
-			return description;
-		}
-		public void setDescription(String description) {
-			this.description = description;
-		}
-		
-		/**
-	     * @hibernate.set   lazy="false"
-	     * 					cascade="all-delete-orphan"
-	     * 					inverse="false"
-	     * 					order-by="sequence_id asc"
-	     * @hibernate.collection-key column="question_uid"
-	     * @hibernate.collection-one-to-many
-	     * 			class="org.lamsfoundation.lams.tool.survey.model.SurveyOption"
-	     * @return
-		 */
-		public Set<SurveyOption> getOptions() {
-			return options;
-		}
-		public void setOptions(Set<SurveyOption> itemInstructions) {
-			this.options = itemInstructions;
-		}
-		
-		/**
-	     * @hibernate.many-to-one
-	     *     cascade="none"
-	     * 		column="create_by"
-		 * 
-		 * @return
-		 */
-		public SurveyUser getCreateBy() {
-			return createBy;
-		}
-		public void setCreateBy(SurveyUser createBy) {
-			this.createBy = createBy;
-		}
-		/**
-		 * @hibernate.property column="create_date" 
-		 * @return
-		 */
-		public Date getCreateDate() {
-			return createDate;
-		}
-		public void setCreateDate(Date createDate) {
-			this.createDate = createDate;
-		}
+public class SurveyQuestion implements Cloneable {
+    private static final Logger log = Logger.getLogger(SurveyQuestion.class);
 
-		/**
-		 * @hibernate.property column="question_type" 
-		 * @return
-		 */
-		public short getType() {
-			return type;
+    private Long uid;
+
+    // Survey Type:1=Single Choice,2=Multiple Choice,3=Text Entry
+    private short type;
+
+    private String description;
+    private int sequenceId;
+
+    // option of Question
+    private boolean appendText;
+    private boolean optional;
+    private boolean allowMultipleAnswer;
+
+    private Set<SurveyOption> options;
+
+    private Date createDate;
+    private SurveyUser createBy;
+
+    // ***********************************************
+    // DTO fields:
+    private String shortTitle;
+
+    public Object clone() {
+	SurveyQuestion obj = null;
+	try {
+	    obj = (SurveyQuestion) super.clone();
+	    // clone options
+	    if (options != null) {
+		Iterator iter = options.iterator();
+		Set set = new HashSet();
+		while (iter.hasNext()) {
+		    SurveyOption instruct = (SurveyOption) iter.next();
+		    SurveyOption newInsruct = (SurveyOption) instruct.clone();
+		    set.add(newInsruct);
 		}
-		public void setType(short type) {
-			this.type = type;
-		}
-	    /**
-	     * @hibernate.property  column="append_text" 
-	     * @return
-	     */
-	    public boolean isAppendText() {
-	    	return appendText;
+		obj.options = set;
 	    }
-	    public void setAppendText(boolean appendText) {
-	    	this.appendText = appendText;
-	    }
-	    
-	    /**
-	     * @hibernate.property  column="optional" 
-	     * @return
-	     */
-		public boolean isOptional() {
-			return optional;
-		}
-		public void setOptional(boolean compulsory) {
-			this.optional = compulsory;
-		}
-	    /**
-	     * @hibernate.property  column="allow_multiple_answer" 
-	     * @return
-	     */
-		public boolean isAllowMultipleAnswer() {
-			return allowMultipleAnswer;
-		}
-		public void setAllowMultipleAnswer(boolean allowMultipleAnswer) {
-			this.allowMultipleAnswer = allowMultipleAnswer;
-		}
-		  /**
-	     * @hibernate.property  column="sequence_id" 
-	     * @return
-	     */
-		public int getSequenceId() {
-			return sequenceId;
-		}
-		public void setSequenceId(int sequenceId) {
-			this.sequenceId = sequenceId;
-		}
-		public String getShortTitle() {
-			return shortTitle;
-		}
-		public void setShortTitle(String shortTitle) {
-			this.shortTitle = shortTitle;
-		}
+	    ((SurveyQuestion) obj).setUid(null);
+	    // clone ReourceUser as well
+	    if (this.createBy != null)
+		((SurveyQuestion) obj).setCreateBy((SurveyUser) this.createBy.clone());
 
-	
-		
-	
+	} catch (CloneNotSupportedException e) {
+	    log.error("When clone " + SurveyQuestion.class + " failed");
+	}
 
+	return obj;
+    }
+
+    // **********************************************************
+    // Get/Set methods
+    // **********************************************************
+    /**
+     * @hibernate.id generator-class="native" type="java.lang.Long" column="uid"
+     * @return Returns the uid.
+     */
+    public Long getUid() {
+	return uid;
+    }
+
+    /**
+     * @param uid
+     *            The uid to set.
+     */
+    public void setUid(Long userID) {
+	this.uid = userID;
+    }
+
+    /**
+     * @hibernate.property column="description" type="text"
+     * @return
+     */
+    public String getDescription() {
+	return description;
+    }
+
+    public void setDescription(String description) {
+	this.description = description;
+    }
+
+    /**
+     * @hibernate.set lazy="false" cascade="all-delete-orphan" inverse="false" order-by="sequence_id asc"
+     * @hibernate.collection-key column="question_uid"
+     * @hibernate.collection-one-to-many class="org.lamsfoundation.lams.tool.survey.model.SurveyOption"
+     * @return
+     */
+    public Set<SurveyOption> getOptions() {
+	return options;
+    }
+
+    public void setOptions(Set<SurveyOption> itemInstructions) {
+	this.options = itemInstructions;
+    }
+
+    /**
+     * @hibernate.many-to-one cascade="none" column="create_by"
+     * 
+     * @return
+     */
+    public SurveyUser getCreateBy() {
+	return createBy;
+    }
+
+    public void setCreateBy(SurveyUser createBy) {
+	this.createBy = createBy;
+    }
+
+    /**
+     * @hibernate.property column="create_date"
+     * @return
+     */
+    public Date getCreateDate() {
+	return createDate;
+    }
+
+    public void setCreateDate(Date createDate) {
+	this.createDate = createDate;
+    }
+
+    /**
+     * @hibernate.property column="question_type"
+     * @return
+     */
+    public short getType() {
+	return type;
+    }
+
+    public void setType(short type) {
+	this.type = type;
+    }
+
+    /**
+     * @hibernate.property column="append_text"
+     * @return
+     */
+    public boolean isAppendText() {
+	return appendText;
+    }
+
+    public void setAppendText(boolean appendText) {
+	this.appendText = appendText;
+    }
+
+    /**
+     * @hibernate.property column="optional"
+     * @return
+     */
+    public boolean isOptional() {
+	return optional;
+    }
+
+    public void setOptional(boolean compulsory) {
+	this.optional = compulsory;
+    }
+
+    /**
+     * @hibernate.property column="allow_multiple_answer"
+     * @return
+     */
+    public boolean isAllowMultipleAnswer() {
+	return allowMultipleAnswer;
+    }
+
+    public void setAllowMultipleAnswer(boolean allowMultipleAnswer) {
+	this.allowMultipleAnswer = allowMultipleAnswer;
+    }
+
+    /**
+     * @hibernate.property column="sequence_id"
+     * @return
+     */
+    public int getSequenceId() {
+	return sequenceId;
+    }
+
+    public void setSequenceId(int sequenceId) {
+	this.sequenceId = sequenceId;
+    }
+
+    public String getShortTitle() {
+	return shortTitle;
+    }
+
+    public void setShortTitle(String shortTitle) {
+	this.shortTitle = shortTitle;
+    }
+
 }
Index: lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/model/SurveySession.java
===================================================================
diff -u -r920894ca746cba5e080023c5cc80167d64d1653d -r098d686653002409f33aff0f69b1183e07c1eab3
--- lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/model/SurveySession.java	(.../SurveySession.java)	(revision 920894ca746cba5e080023c5cc80167d64d1653d)
+++ lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/model/SurveySession.java	(.../SurveySession.java)	(revision 098d686653002409f33aff0f69b1183e07c1eab3)
@@ -24,10 +24,7 @@
 package org.lamsfoundation.lams.tool.survey.model;
 
 import java.util.Date;
-import java.util.Set;
 
-import org.apache.log4j.Logger;
-
 /**
  * Survey
  * @author Dapeng Ni
Index: lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/model/SurveyUser.java
===================================================================
diff -u -r920894ca746cba5e080023c5cc80167d64d1653d -r098d686653002409f33aff0f69b1183e07c1eab3
--- lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/model/SurveyUser.java	(.../SurveyUser.java)	(revision 920894ca746cba5e080023c5cc80167d64d1653d)
+++ lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/model/SurveyUser.java	(.../SurveyUser.java)	(revision 098d686653002409f33aff0f69b1183e07c1eab3)
@@ -23,190 +23,213 @@
 /* $$Id$$ */
 package org.lamsfoundation.lams.tool.survey.model;
 
-import java.util.Date;
-
 import org.apache.commons.lang.builder.EqualsBuilder;
 import org.apache.commons.lang.builder.HashCodeBuilder;
 import org.apache.log4j.Logger;
 import org.lamsfoundation.lams.usermanagement.dto.UserDTO;
+
 /**
  * Survey
+ * 
  * @author Dapeng Ni
- *
- * @hibernate.class  table="tl_lasurv11_user"
- *
+ * 
+ * @hibernate.class table="tl_lasurv11_user"
  */
-public class SurveyUser implements Cloneable{
-	private static final long serialVersionUID = -7043502180037866257L;
-	private static Logger log = Logger.getLogger(SurveyUser.class);
-	
+public class SurveyUser implements Cloneable {
+    private static final long serialVersionUID = -7043502180037866257L;
+    private static Logger log = Logger.getLogger(SurveyUser.class);
+
     private Long uid;
-	private Long userId;
-	private String firstName;
-	private String lastName;
-	private String loginName;
-	private boolean sessionFinished;
-	
-	private SurveySession session;
-	private Survey survey;
-	
-	
-	public SurveyUser(){
-	}
-	public SurveyUser(UserDTO user, SurveySession session){
-		this.userId = new Long(user.getUserID().intValue());
-		this.firstName = user.getFirstName();
-		this.lastName = user.getLastName();
-		this.loginName = user.getLogin();
-		this.session = session;
-		this.survey = null;
-		this.sessionFinished = false;
-	}
-	public SurveyUser(UserDTO user, Survey content){
-		this.userId = new Long(user.getUserID().intValue());
-		this.firstName = user.getFirstName();
-		this.lastName = user.getLastName();
-		this.loginName = user.getLogin();
-		this.session = null;
-		this.survey = content;
-		this.sessionFinished = false;
-	}
-	/**
-	 * Clone method from <code>java.lang.Object</code>
-	 */
-	public Object clone(){
-  		
-		SurveyUser user = null;
-  		try{
-  			user = (SurveyUser) super.clone();
-  			user.setUid(null);
-  			//never clone session
-  			user.setSession(null);
-		} catch (CloneNotSupportedException e) {
-			log.error("When clone " + SurveyUser.class + " failed");
-		}
-  		
-  		return user;
-  	}
-//  **********************************************************
-  	//		Get/Set methods
-//  **********************************************************
-	/**
-	 * @hibernate.id generator-class="native" type="java.lang.Long" column="uid"
-	 * @return Returns the uid.
-	 */
-	public Long getUid() {
-		return uid;
-	}
-	/**
-	 * @param uid The uid to set.
-	 */
-	public void setUid(Long userID) {
-		this.uid = userID;
-	}
+    private Long userId;
+    private String firstName;
+    private String lastName;
+    private String loginName;
+    private boolean sessionFinished;
+    private boolean responseFinalized;
 
-	/**
-	 * @hibernate.property column="user_id" length="20"
-	 * @return Returns the userId.
-	 */
-	public Long getUserId() {
-		return userId;
-	}
-	/**
-	 * @param userId
-	 *            The userId to set.
-	 */
-	public void setUserId(Long userID) {
-		this.userId = userID;
-	}
+    private SurveySession session;
+    private Survey survey;
 
-	/**
-	 * @hibernate.property length="255" column="last_name"
-	 * @return
-	 */
-	public String getLastName() {
-		return lastName;
-	}
-	public void setLastName(String lastName) {
-		this.lastName = lastName;
-	}
-	/**
-	 * @hibernate.property length="255" column="first_name"
-	 * @return
-	 */
-	public String getFirstName() {
-		return firstName;
-	}
-	public void setFirstName(String firstName) {
-		this.firstName = firstName;
-	}
-	
+    public SurveyUser() {
+    }
 
-	/**
-	 * @hibernate.property column="login_name"
-	 * @return
-	 */
-	public String getLoginName() {
-		return loginName;
-	}
+    public SurveyUser(UserDTO user, SurveySession session) {
+	this.userId = new Long(user.getUserID().intValue());
+	this.firstName = user.getFirstName();
+	this.lastName = user.getLastName();
+	this.loginName = user.getLogin();
+	this.session = session;
+	this.survey = null;
+	this.sessionFinished = false;
+	this.responseFinalized = false;
+    }
 
-	public void setLoginName(String loginName) {
-		this.loginName = loginName;
-	}
-	/**
-	 * @hibernate.many-to-one column="session_uid"
-	 * 			cascade="none"
-	 * @return
-	 */
-	public SurveySession getSession() {
-		return session;
-	}
+    public SurveyUser(UserDTO user, Survey content) {
+	this.userId = new Long(user.getUserID().intValue());
+	this.firstName = user.getFirstName();
+	this.lastName = user.getLastName();
+	this.loginName = user.getLogin();
+	this.session = null;
+	this.survey = content;
+	this.responseFinalized = false;
+    }
 
-	public void setSession(SurveySession session) {
-		this.session = session;
-	}
-	/**
-	 * @hibernate.many-to-one column="survey_uid"
-	 * 			cascade="none"
-	 * @return
-	 */
-	public Survey getSurvey() {
-		return survey;
-	}
-	public void setSurvey(Survey content) {
-		this.survey = content;
-	}
-	/**
-	 * @hibernate.property column="session_finished"
-	 * @return
-	 */
-	public boolean isSessionFinished() {
-		return sessionFinished;
-	}
+    /**
+     * Clone method from <code>java.lang.Object</code>
+     */
+    public Object clone() {
 
-	public void setSessionFinished(boolean sessionFinished) {
-		this.sessionFinished = sessionFinished;
+	SurveyUser user = null;
+	try {
+	    user = (SurveyUser) super.clone();
+	    user.setUid(null);
+	    // never clone session
+	    user.setSession(null);
+	} catch (CloneNotSupportedException e) {
+	    log.error("When clone " + SurveyUser.class + " failed");
 	}
-	
-	public boolean equals(Object obj) {
-		if (this == obj)
-			return true;
-		if (!(obj instanceof SurveyUser))
-			return false;
 
-		final SurveyUser user = (SurveyUser) obj;
-		
-		return new EqualsBuilder().append(this.uid, user.uid).append(this.firstName, user.firstName)
-			.append(this.lastName,user.lastName).append(this.loginName, user.loginName).isEquals();
-		
-	}
+	return user;
+    }
 
-	public int hashCode() {
-		return new HashCodeBuilder().append(uid).append(firstName)
-		.append(lastName).append(loginName)
-		.toHashCode();
-	}
+    // **********************************************************
+    // Get/Set methods
+    // **********************************************************
+    /**
+     * @hibernate.id generator-class="native" type="java.lang.Long" column="uid"
+     * @return Returns the uid.
+     */
+    public Long getUid() {
+	return uid;
+    }
 
+    /**
+     * @param uid
+     *            The uid to set.
+     */
+    public void setUid(Long userID) {
+	this.uid = userID;
+    }
 
+    /**
+     * @hibernate.property column="user_id" length="20"
+     * @return Returns the userId.
+     */
+    public Long getUserId() {
+	return userId;
+    }
 
+    /**
+     * @param userId
+     *            The userId to set.
+     */
+    public void setUserId(Long userID) {
+	this.userId = userID;
+    }
+
+    /**
+     * @hibernate.property length="255" column="last_name"
+     * @return
+     */
+    public String getLastName() {
+	return lastName;
+    }
+
+    public void setLastName(String lastName) {
+	this.lastName = lastName;
+    }
+
+    /**
+     * @hibernate.property length="255" column="first_name"
+     * @return
+     */
+    public String getFirstName() {
+	return firstName;
+    }
+
+    public void setFirstName(String firstName) {
+	this.firstName = firstName;
+    }
+
+    /**
+     * @hibernate.property column="login_name"
+     * @return
+     */
+    public String getLoginName() {
+	return loginName;
+    }
+
+    public void setLoginName(String loginName) {
+	this.loginName = loginName;
+    }
+
+    /**
+     * @hibernate.many-to-one column="session_uid" cascade="none"
+     * @return
+     */
+    public SurveySession getSession() {
+	return session;
+    }
+
+    public void setSession(SurveySession session) {
+	this.session = session;
+    }
+
+    /**
+     * @hibernate.many-to-one column="survey_uid" cascade="none"
+     * @return
+     */
+    public Survey getSurvey() {
+	return survey;
+    }
+
+    public void setSurvey(Survey content) {
+	this.survey = content;
+    }
+
+    /**
+     * @hibernate.property column="session_finished"
+     * @return
+     */
+    public boolean isSessionFinished() {
+	return sessionFinished;
+    }
+
+    public void setSessionFinished(boolean sessionFinished) {
+	this.sessionFinished = sessionFinished;
+    }
+    
+    /**
+     * @hibernate.property column="response_finalized"
+     * @return Returns the responseFinalized.
+     */
+    public boolean isResponseFinalized() {
+	return responseFinalized;
+    }
+
+    /**
+     * @param responseFinalized
+     *                The responseFinalized to set.
+     */
+    public void setResponseFinalized(boolean responseFinalized) {
+	this.responseFinalized = responseFinalized;
+    }
+
+    public boolean equals(Object obj) {
+	if (this == obj)
+	    return true;
+	if (!(obj instanceof SurveyUser))
+	    return false;
+
+	final SurveyUser user = (SurveyUser) obj;
+
+	return new EqualsBuilder().append(this.uid, user.uid).append(this.firstName, user.firstName)
+		.append(this.lastName, user.lastName).append(this.loginName, user.loginName).isEquals();
+
+    }
+
+    public int hashCode() {
+	return new HashCodeBuilder().append(uid).append(firstName).append(lastName).append(loginName).toHashCode();
+    }
+
 }
Index: lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/service/ISurveyService.java
===================================================================
diff -u -rbb597b8155375e6ac4dfe280f630d323b6e5e575 -r098d686653002409f33aff0f69b1183e07c1eab3
--- lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/service/ISurveyService.java	(.../ISurveyService.java)	(revision bb597b8155375e6ac4dfe280f630d323b6e5e575)
+++ lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/service/ISurveyService.java	(.../ISurveyService.java)	(revision 098d686653002409f33aff0f69b1183e07c1eab3)
@@ -29,7 +29,6 @@
 import java.util.Set;
 import java.util.SortedMap;
 
-import org.lamsfoundation.lams.events.IEventNotificationService;
 import org.lamsfoundation.lams.notebook.model.NotebookEntry;
 import org.lamsfoundation.lams.tool.survey.dto.AnswerDTO;
 import org.lamsfoundation.lams.tool.survey.dto.ReflectDTO;
@@ -39,12 +38,11 @@
 import org.lamsfoundation.lams.tool.survey.model.SurveyQuestion;
 import org.lamsfoundation.lams.tool.survey.model.SurveySession;
 import org.lamsfoundation.lams.tool.survey.model.SurveyUser;
-import org.lamsfoundation.lams.usermanagement.User;
 
 /**
  * @author Dapeng.Ni
  * 
- * Interface that defines the contract that all Survey service provider must follow.
+ *         Interface that defines the contract that all Survey service provider must follow.
  */
 public interface ISurveyService {
 
@@ -118,6 +116,11 @@
      */
     AnswerDTO getQuestionResponse(Long sessionId, Long questionUid);
 
+    List<String> getOpenResponsesForTablesorter(final Long qaSessionId, final Long questionId, int page, int size,
+	    int sorting);
+
+    int getCountResponsesBySessionAndQuestion(final Long qaSessionId, final Long questionId);
+
     /**
      * Commit answers for a group of question together.
      * 
@@ -165,6 +168,14 @@
      * @return
      */
     List<SurveyUser> getSessionUsers(Long sessionId);
+    
+    /**
+     * Get number of finished users.
+     * 
+     * @param sessionId
+     * @return
+     */
+    int getCountFinishedUsers(Long sessionId);
 
     // ******************************************************************************************
     // ********** Session methods ***********************
@@ -193,6 +204,13 @@
      * @return
      */
     String finishToolSession(Long toolSessionId, Long userId) throws SurveyApplicationException;
+    
+    /**
+     * Mark user as finalized response which will signify he has submitted response.
+     * 
+     * @param user
+     */
+    void setResponseFinalized(Long userUid);
 
     // ******************************************************************************************
     // Monitoring summary /statistic methods
@@ -261,21 +279,21 @@
      * @return
      */
     Map<Long, Set<ReflectDTO>> getReflectList(Long contentId, boolean setEntry);
-    
+
     void notifyTeachersOnAnswerSumbit(Long sessionId, SurveyUser surveyUser);
 
     /**
      * Creates an unique name for a SurveyCondition. It consists of the tool output definition name and a unique
      * positive integer number.
      * 
      * @param existingConditions
-     *                existing conditions; required to check if a condition with the same name does not exist.
+     *            existing conditions; required to check if a condition with the same name does not exist.
      * @return unique SurveyCondition name
      */
     String createConditionName(Collection<SurveyCondition> existingConditions);
 
     void deleteCondition(SurveyCondition condition);
-    
+
     /**
      * Returns whether activity is grouped and therefore it is expected more than one tool session.
      * 
Index: lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/service/SurveyServiceImpl.java
===================================================================
diff -u -r40eb54374e84591563d8b6a679ac719dbc85c8f7 -r098d686653002409f33aff0f69b1183e07c1eab3
--- lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/service/SurveyServiceImpl.java	(.../SurveyServiceImpl.java)	(revision 40eb54374e84591563d8b6a679ac719dbc85c8f7)
+++ lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/service/SurveyServiceImpl.java	(.../SurveyServiceImpl.java)	(revision 098d686653002409f33aff0f69b1183e07c1eab3)
@@ -174,6 +174,11 @@
     public SurveyUser getUserByIDAndSession(Long userId, Long sessionId) {
 	return surveyUserDao.getUserByUserIDAndSessionID(userId, sessionId);
     }
+    
+    @Override
+    public int getCountFinishedUsers(Long sessionId) {
+	return surveyUserDao.getCountFinishedUsers(sessionId);
+    }
 
     @Override
     public void saveOrUpdateSurvey(Survey survey) {
@@ -219,6 +224,13 @@
 	}
 	return nextUrl;
     }
+    
+    @Override
+    public void setResponseFinalized(Long userUid) {
+	SurveyUser user = (SurveyUser) surveyUserDao.getObject(SurveyUser.class, userUid);
+	user.setResponseFinalized(true);
+	surveyUserDao.saveObject(user);
+    }
 
     @Override
     public Map<Long, Set<ReflectDTO>> getReflectList(Long contentId, boolean setEntry) {
@@ -340,57 +352,70 @@
 	}
 
 	// initial a array to hold how many time chose has been done for a option or open text.
-	int optSize = options.size();
+	int numberAvailableOptions = options.size();
 	// for appendText and open Text Entry will be the last one of choose[] array.
 	if (answerDto.isAppendText() || answerDto.getType() == SurveyConstants.QUESTION_TYPE_TEXT_ENTRY) {
-	    optSize++;
+	    numberAvailableOptions++;
 	}
 
-	int[] choose = new int[optSize];
-	Arrays.fill(choose, 0);
+	int[] choiceArray = new int[numberAvailableOptions];
+	Arrays.fill(choiceArray, 0);
 
 	// sum up all option and open text (if has) have been selected count list
-	int answerSum = 0;
+	int numberAnswers = 0;
 	if (answsers != null) {
 	    for (SurveyAnswer answer : answsers) {
-		String[] choseOpt = SurveyWebUtils.getChoiceList(answer.getAnswerChoices());
-		for (String optUid : choseOpt) {
+		String[] choiceList = SurveyWebUtils.getChoiceList(answer.getAnswerChoices());
+		for (String optUid : choiceList) {
 		    // if option has been chosen, the relative index of choose[] array will increase.
 		    if (optMap.containsKey(optUid)) {
-			choose[optMap.get(optUid)]++;
-			answerSum++;
+			choiceArray[optMap.get(optUid)]++;
+			numberAnswers++;
 		    }
 		}
 		// handle appendText or Open Text Entry
 		if ((answerDto.isAppendText() || answerDto.getType() == SurveyConstants.QUESTION_TYPE_TEXT_ENTRY)
 			&& !StringUtils.isBlank(answer.getAnswerText())) {
-		    choose[optSize - 1]++;
-		    answerSum++;
+		    choiceArray[numberAvailableOptions - 1]++;
+		    numberAnswers++;
 		}
 	    }
 	}
 	// caculate the percentage of answer response
 	idx = 0;
-	if (answerSum == 0) {
-	    answerSum = 1;
+	if (numberAnswers == 0) {
+	    numberAnswers = 1;
 	}
 	for (SurveyOption option : options) {
-	    option.setResponse((double) choose[idx] / (double) answerSum * 100d);
-	    option.setResponseFormatStr(new Long(Math.round(option.getResponse())).toString());
-	    option.setResponseCount(choose[idx]);
+	    double percentage = (double) choiceArray[idx] / (double) numberAnswers * 100d;
+	    option.setResponse(percentage);
+	    option.setResponseFormatStr(new Long(Math.round(percentage)).toString());
+	    option.setResponseCount(choiceArray[idx]);
 	    idx++;
 	}
 	if (answerDto.isAppendText() || answerDto.getType() == SurveyConstants.QUESTION_TYPE_TEXT_ENTRY) {
-	    answerDto.setOpenResponse((double) choose[idx] / (double) answerSum * 100d);
-	    answerDto.setOpenResponseFormatStr(new Long(Math.round(answerDto.getOpenResponse())).toString());
-	    answerDto.setOpenResponseCount(choose[idx]);
+	    double percentage = (double) choiceArray[idx] / (double) numberAnswers * 100d;
+	    answerDto.setOpenResponse(percentage);
+	    answerDto.setOpenResponseFormatStr(new Long(Math.round(percentage)).toString());
+	    answerDto.setOpenResponseCount(choiceArray[idx]);
 	}
 
 	return answerDto;
+    }
 
+    @Override
+    public List<String> getOpenResponsesForTablesorter(final Long qaSessionId, final Long questionId, int page,
+	    int size, int sorting) {
+	return surveyAnswerDao.getOpenResponsesForTablesorter(qaSessionId, questionId, page, size,
+		sorting);
     }
 
     @Override
+    public int getCountResponsesBySessionAndQuestion(final Long qaSessionId, final Long questionId) {
+	return surveyAnswerDao.getCountResponsesBySessionAndQuestion(qaSessionId, questionId);
+    }
+
+    @Override
     public SortedMap<SurveySession, List<AnswerDTO>> getSummary(Long toolContentId) {
 
 	SortedMap<SurveySession, List<AnswerDTO>> summary = new TreeMap<SurveySession, List<AnswerDTO>>(
@@ -502,11 +527,11 @@
 
 	return summary;
     }
-    
+
     public boolean isGroupedActivity(long toolContentID) {
 	return toolService.isGroupedActivity(toolContentID);
     }
-    
+
     @Override
     public String createConditionName(Collection<SurveyCondition> existingConditions) {
 	String uniqueNumber = null;
@@ -531,15 +556,15 @@
     public Class[] getSupportedToolOutputDefinitionClasses(int definitionType) {
 	return getSurveyOutputFactory().getSupportedDefinitionClasses(definitionType);
     }
-    
+
     @Override
     public void notifyTeachersOnAnswerSumbit(Long sessionId, SurveyUser surveyUser) {
-	
-	//it appears surveyUser can be null (?)
+
+	// it appears surveyUser can be null (?)
 	if (surveyUser == null) {
 	    return;
 	}
-	
+
 	String userName = surveyUser.getLastName() + " " + surveyUser.getFirstName();
 	String message = getLocalisedMessage("event.answer.submit.body", new Object[] { userName });
 	eventNotificationService.notifyLessonMonitors(sessionId, message, false);
@@ -633,7 +658,7 @@
 	try {
 	    // register version filter class
 	    exportContentService.registerImportVersionFilterClass(SurveyImportContentVersionFilter.class);
-	
+
 	    Object toolPOJO = exportContentService.importToolContent(toolContentPath, surveyToolContentHandler,
 		    fromVersion, toVersion);
 	    if (!(toolPOJO instanceof Survey)) {
@@ -719,11 +744,11 @@
 	    }
 	}
     }
-    
+
     public String getToolContentTitle(Long toolContentId) {
 	return getSurveyByContentId(toolContentId).getTitle();
     }
-    
+
     @Override
     public void resetDefineLater(Long toolContentId) throws DataMissingException, ToolException {
 	Survey survey = surveyDao.getByContentId(toolContentId);
@@ -732,7 +757,7 @@
 	}
 	survey.setDefineLater(false);
     }
-    
+
     @Override
     public boolean isContentEdited(Long toolContentId) {
 	return getSurveyByContentId(toolContentId).isDefineLater();
@@ -837,31 +862,33 @@
     /* ===============Methods implemented from ToolContentImport102Manager =============== */
 
     /*
-     * Sample content for import: <struct> <var name='objectType'><string>content</string></var> <var name='id'><number>34.0</number></var>
-     * <var name='questions'><array length='3'> <struct> <var name='order'><number>1.0</number></var> <var
-     * name='isOptional'><boolean value='true'/></var> <var name='question'><string>Sample Multiple choice - only one
-     * response allowed?</string></var> <var name='questionType'><string>simpleChoice</string></var> <var
-     * name='isTextBoxEnabled'><boolean value='false'/></var> <var name='candidates'><array length='3'> <struct> <var
-     * name='order'><number>1.0</number></var> <var name='answer'><string>Option 1</string></var> </struct>
-     * <struct> <var name='order'><number>2.0</number></var> <var name='answer'><string>Option 2</string></var>
-     * </struct> <struct> <var name='order'><number>3.0</number></var> <var name='answer'><string>Option 3</string></var>
+     * Sample content for import: <struct> <var name='objectType'><string>content</string></var> <var
+     * name='id'><number>34.0</number></var> <var name='questions'><array length='3'> <struct> <var
+     * name='order'><number>1.0</number></var> <var name='isOptional'><boolean value='true'/></var> <var
+     * name='question'><string>Sample Multiple choice - only one response allowed?</string></var> <var
+     * name='questionType'><string>simpleChoice</string></var> <var name='isTextBoxEnabled'><boolean
+     * value='false'/></var> <var name='candidates'><array length='3'> <struct> <var
+     * name='order'><number>1.0</number></var> <var name='answer'><string>Option 1</string></var> </struct> <struct>
+     * <var name='order'><number>2.0</number></var> <var name='answer'><string>Option 2</string></var> </struct>
+     * <struct> <var name='order'><number>3.0</number></var> <var name='answer'><string>Option 3</string></var>
      * </struct> </array></var> </struct> <struct> <var name='order'><number>2.0</number></var> <var
-     * name='isOptional'><boolean value='false'/></var> <var name='question'><string>Sample Multiple choice -
-     * multiple response allowed?</string></var> <var name='questionType'><string>choiceMultiple</string></var>
-     * <var name='isTextBoxEnabled'><boolean value='true'/></var> <var name='candidates'><array length='3'> <struct>
-     * <var name='order'><number>1.0</number></var> <var name='answer'><string>Option 1</string></var> </struct><struct>
-     * <var name='order'><number>2.0</number></var> <var name='answer'><string>Option 2</string></var> </struct><struct>
-     * <var name='order'><number>3.0</number></var> <var name='answer'><string>Option 3</string></var> </struct>
+     * name='isOptional'><boolean value='false'/></var> <var name='question'><string>Sample Multiple choice - multiple
+     * response allowed?</string></var> <var name='questionType'><string>choiceMultiple</string></var> <var
+     * name='isTextBoxEnabled'><boolean value='true'/></var> <var name='candidates'><array length='3'> <struct> <var
+     * name='order'><number>1.0</number></var> <var name='answer'><string>Option 1</string></var> </struct><struct> <var
+     * name='order'><number>2.0</number></var> <var name='answer'><string>Option 2</string></var> </struct><struct> <var
+     * name='order'><number>3.0</number></var> <var name='answer'><string>Option 3</string></var> </struct>
      * </array></var> </struct> <struct> <var name='order'><number>3.0</number></var> <var name='isOptional'><boolean
      * value='true'/></var> <var name='question'><string>Sample Free text question?</string></var> <var
      * name='questionType'><string>textEntry</string></var> <var name='isTextBoxEnabled'><boolean value='false'/></var>
      * <var name='candidates'><array length='0'></array></var> </struct> </array></var> <var
-     * name='contentDefineLater'><boolean value='false'/></var> <var name='body'><string>Put instructions here.</string></var>
-     * <var name='contentShowUser'><boolean value='false'/></var> <var name='isHTML'><boolean value='false'/></var>
-     * <var name='summary'><string>Thank you for your participation!</string></var> xxxxxxxxxx <var name='title'><string>Put
-     * Title Here</string></var> <var name='description'><string>Survey Questions</string></var> <var
-     * name='contentType'><string>surveycontent</string></var> <var name='isReusable'><boolean value='false'/></var>
-     * </struct></array></var> <var name='firstActivity'><number>31.0</number></var>
+     * name='contentDefineLater'><boolean value='false'/></var> <var name='body'><string>Put instructions
+     * here.</string></var> <var name='contentShowUser'><boolean value='false'/></var> <var name='isHTML'><boolean
+     * value='false'/></var> <var name='summary'><string>Thank you for your participation!</string></var> xxxxxxxxxx
+     * <var name='title'><string>Put Title Here</string></var> <var name='description'><string>Survey
+     * Questions</string></var> <var name='contentType'><string>surveycontent</string></var> <var
+     * name='isReusable'><boolean value='false'/></var> </struct></array></var> <var
+     * name='firstActivity'><number>31.0</number></var>
      */
 
     /**
Index: lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/surveyApplicationContext.xml
===================================================================
diff -u -rf3fbbb8ec1732de7e5ce51567287b42b55ab33e1 -r098d686653002409f33aff0f69b1183e07c1eab3
--- lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/surveyApplicationContext.xml	(.../surveyApplicationContext.xml)	(revision f3fbbb8ec1732de7e5ce51567287b42b55ab33e1)
+++ lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/surveyApplicationContext.xml	(.../surveyApplicationContext.xml)	(revision 098d686653002409f33aff0f69b1183e07c1eab3)
@@ -105,6 +105,7 @@
                 <prop key="remove*">PROPAGATION_REQUIRED,-java.lang.Exception</prop>
                 <prop key="save*">PROPAGATION_REQUIRED,-java.lang.Exception</prop>
                 <prop key="update*">PROPAGATION_REQUIRED,-java.lang.Exception</prop>
+                <prop key="setResponseFinalized">PROPAGATION_REQUIRED,-java.lang.Exception</prop>
                 <prop key="resetDefineLater">PROPAGATION_REQUIRED,-java.lang.Exception</prop>
                 <prop key="notifyTeachersOnAnswerSumbit">PROPAGATION_REQUIRED,-java.lang.Exception</prop>
                 <prop key="copyToolContent">PROPAGATION_REQUIRED,-java.lang.Exception</prop>
Index: lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/util/SurveyWebUtils.java
===================================================================
diff -u -rde91e6d736e56d514d840e128bd06c9238b5871b -r098d686653002409f33aff0f69b1183e07c1eab3
--- lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/util/SurveyWebUtils.java	(.../SurveyWebUtils.java)	(revision de91e6d736e56d514d840e128bd06c9238b5871b)
+++ lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/util/SurveyWebUtils.java	(.../SurveyWebUtils.java)	(revision 098d686653002409f33aff0f69b1183e07c1eab3)
@@ -21,53 +21,49 @@
  * ****************************************************************
  */
 
-/* $$Id$$ */	
+/* $$Id$$ */
 package org.lamsfoundation.lams.tool.survey.util;
 
-import org.apache.commons.lang.StringUtils;
 import org.lamsfoundation.lams.tool.survey.model.Survey;
-import org.lamsfoundation.lams.tool.survey.model.SurveyQuestion;
 import org.lamsfoundation.lams.tool.survey.web.action.MonitoringAction;
 
-
-
 /**
  * Contains helper methods used by the Action Servlets
  * 
  * @author Anthony Sukkar
- *
  */
 public class SurveyWebUtils {
 
-	public static boolean isSurveyEditable(Survey survey) {
-	        if ( (survey.isDefineLater() == true) && (survey.isContentInUse()==true) )
-	        {
-	//            throw new SurveyApplicationException("An exception has occurred: There is a bug in this tool, conflicting flags are set");
-	        	 MonitoringAction.log.error("An exception has occurred: There is a bug in this tool, conflicting flags are set");
-	             return false;
-	        }
-	        else if ( (survey.isDefineLater() == true) && (survey.isContentInUse() == false))
-	            return true;
-	        else if ( (survey.isDefineLater() == false) && (survey.isContentInUse() == false))
-	            return true;
-	        else //  (content.isContentInUse()==true && content.isDefineLater() == false)
-	            return false;
-		}
+    public static boolean isSurveyEditable(Survey survey) {
+	if ((survey.isDefineLater() == true) && (survey.isContentInUse() == true)) {
+	    // throw new
+	    // SurveyApplicationException("An exception has occurred: There is a bug in this tool, conflicting flags are set");
+	    MonitoringAction.log
+		    .error("An exception has occurred: There is a bug in this tool, conflicting flags are set");
+	    return false;
+	} else if ((survey.isDefineLater() == true) && (survey.isContentInUse() == false))
+	    return true;
+	else if ((survey.isDefineLater() == false) && (survey.isContentInUse() == false))
+	    return true;
+	else
+	    // (content.isContentInUse()==true && content.isDefineLater() == false)
+	    return false;
+    }
 
-	public static String getChoicesStr(String[] choiceList) {
-		String choices = "";
-		if(choiceList == null)
-			return choices;
-		
-		for(String c: choiceList)
-			choices = choices + c + "&";
-		return choices;
-	}
-	
-	public static String[] getChoiceList(String choiceList) {
-		if(choiceList == null)
-			return new String[]{};
-		
-		return choiceList.split("&");
-	}
+    public static String getChoicesStr(String[] choiceList) {
+	String choices = "";
+	if (choiceList == null)
+	    return choices;
+
+	for (String c : choiceList)
+	    choices = choices + c + "&";
+	return choices;
+    }
+
+    public static String[] getChoiceList(String choiceList) {
+	if (choiceList == null)
+	    return new String[] {};
+
+	return choiceList.split("&");
+    }
 }
Index: lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/web/action/AuthoringAction.java
===================================================================
diff -u -r8b97231e320c0c5b674f07c14da711f232ba9e1c -r098d686653002409f33aff0f69b1183e07c1eab3
--- lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/web/action/AuthoringAction.java	(.../AuthoringAction.java)	(revision 8b97231e320c0c5b674f07c14da711f232ba9e1c)
+++ lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/web/action/AuthoringAction.java	(.../AuthoringAction.java)	(revision 098d686653002409f33aff0f69b1183e07c1eab3)
@@ -29,7 +29,6 @@
 import java.util.ArrayList;
 import java.util.Date;
 import java.util.HashMap;
-import java.util.HashSet;
 import java.util.Iterator;
 import java.util.LinkedHashSet;
 import java.util.List;
@@ -54,9 +53,7 @@
 import org.apache.struts.action.ActionMapping;
 import org.apache.struts.action.ActionMessage;
 import org.apache.struts.action.ActionMessages;
-import org.apache.struts.upload.FormFile;
 import org.lamsfoundation.lams.authoring.web.AuthoringConstants;
-import org.lamsfoundation.lams.contentrepository.client.IToolContentHandler;
 import org.lamsfoundation.lams.learningdesign.TextSearchConditionComparator;
 import org.lamsfoundation.lams.tool.ToolAccessMode;
 import org.lamsfoundation.lams.tool.survey.SurveyConstants;
@@ -67,13 +64,11 @@
 import org.lamsfoundation.lams.tool.survey.model.SurveyUser;
 import org.lamsfoundation.lams.tool.survey.service.ISurveyService;
 import org.lamsfoundation.lams.tool.survey.service.SurveyApplicationException;
-import org.lamsfoundation.lams.tool.survey.service.UploadSurveyFileException;
 import org.lamsfoundation.lams.tool.survey.util.QuestionsComparator;
 import org.lamsfoundation.lams.tool.survey.util.SurveyWebUtils;
 import org.lamsfoundation.lams.tool.survey.web.form.QuestionForm;
 import org.lamsfoundation.lams.tool.survey.web.form.SurveyForm;
 import org.lamsfoundation.lams.usermanagement.dto.UserDTO;
-import org.lamsfoundation.lams.util.FileValidatorUtil;
 import org.lamsfoundation.lams.util.WebUtil;
 import org.lamsfoundation.lams.web.session.SessionManager;
 import org.lamsfoundation.lams.web.util.AttributeNames;
Index: lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/web/action/ChartAction.java
===================================================================
diff -u -rd25b9e0ce0aa1cc320186300d26c05ef385b780c -r098d686653002409f33aff0f69b1183e07c1eab3
--- lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/web/action/ChartAction.java	(.../ChartAction.java)	(revision d25b9e0ce0aa1cc320186300d26c05ef385b780c)
+++ lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/web/action/ChartAction.java	(.../ChartAction.java)	(revision 098d686653002409f33aff0f69b1183e07c1eab3)
@@ -54,7 +54,7 @@
  */
 public class ChartAction extends Action {
 
-    static Logger logger = Logger.getLogger(ChartAction.class);
+    private static Logger logger = Logger.getLogger(ChartAction.class);
 
     private static ISurveyService surveyService;
     private MessageResources resource;
@@ -67,6 +67,7 @@
 	Long sessionId = WebUtil.readLongParam(request, AttributeNames.PARAM_TOOL_SESSION_ID);
 	Long questionUid = WebUtil.readLongParam(request, SurveyConstants.ATTR_QUESTION_UID);
 
+	// if excludeUserId received exclude this user's answers
 	AnswerDTO answer = getSurveyService().getQuestionResponse(sessionId, questionUid);
 	if (answer.getType() == SurveyConstants.QUESTION_TYPE_TEXT_ENTRY) {
 	    ChartAction.logger.error("Error question type : Text entry can not generate chart.");
@@ -94,7 +95,7 @@
 	} catch (JSONException e) {
 	    ChartAction.logger.error("Error while generating pie chart for Survey Tool: " + sessionId);
 	}
-	
+
 	response.setContentType("application/json;charset=utf-8");
 	response.getWriter().write(responseJSON.toString());
 	return null;
Index: lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/web/action/LearningAction.java
===================================================================
diff -u -rdb8cd59179f23576cd809e7657885236cfafa60b -r098d686653002409f33aff0f69b1183e07c1eab3
--- lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/web/action/LearningAction.java	(.../LearningAction.java)	(revision db8cd59179f23576cd809e7657885236cfafa60b)
+++ lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/web/action/LearningAction.java	(.../LearningAction.java)	(revision 098d686653002409f33aff0f69b1183e07c1eab3)
@@ -32,14 +32,15 @@
 import java.util.List;
 import java.util.Map;
 import java.util.SortedMap;
-import java.util.TreeMap;
 import java.util.TimeZone;
+import java.util.TreeMap;
 
 import javax.servlet.ServletException;
 import javax.servlet.http.HttpServletRequest;
 import javax.servlet.http.HttpServletResponse;
 import javax.servlet.http.HttpSession;
 
+import org.apache.commons.lang.StringEscapeUtils;
 import org.apache.commons.lang.StringUtils;
 import org.apache.log4j.Logger;
 import org.apache.struts.action.Action;
@@ -48,8 +49,10 @@
 import org.apache.struts.action.ActionForward;
 import org.apache.struts.action.ActionMapping;
 import org.apache.struts.action.ActionMessage;
-import org.lamsfoundation.lams.events.DeliveryMethodMail;
-import org.lamsfoundation.lams.events.IEventNotificationService;
+import org.apache.struts.action.ActionRedirect;
+import org.apache.tomcat.util.json.JSONArray;
+import org.apache.tomcat.util.json.JSONException;
+import org.apache.tomcat.util.json.JSONObject;
 import org.lamsfoundation.lams.learning.web.bean.ActivityPositionDTO;
 import org.lamsfoundation.lams.learning.web.util.LearningWebUtil;
 import org.lamsfoundation.lams.notebook.model.NotebookEntry;
@@ -59,6 +62,7 @@
 import org.lamsfoundation.lams.tool.survey.dto.AnswerDTO;
 import org.lamsfoundation.lams.tool.survey.model.Survey;
 import org.lamsfoundation.lams.tool.survey.model.SurveyAnswer;
+import org.lamsfoundation.lams.tool.survey.model.SurveyQuestion;
 import org.lamsfoundation.lams.tool.survey.model.SurveySession;
 import org.lamsfoundation.lams.tool.survey.model.SurveyUser;
 import org.lamsfoundation.lams.tool.survey.service.ISurveyService;
@@ -67,7 +71,6 @@
 import org.lamsfoundation.lams.tool.survey.util.SurveyWebUtils;
 import org.lamsfoundation.lams.tool.survey.web.form.AnswerForm;
 import org.lamsfoundation.lams.tool.survey.web.form.ReflectionForm;
-import org.lamsfoundation.lams.usermanagement.User;
 import org.lamsfoundation.lams.usermanagement.dto.UserDTO;
 import org.lamsfoundation.lams.util.DateUtil;
 import org.lamsfoundation.lams.util.WebUtil;
@@ -89,7 +92,7 @@
 
     @Override
     public ActionForward execute(ActionMapping mapping, ActionForm form, HttpServletRequest request,
-	    HttpServletResponse response) throws IOException, ServletException {
+	    HttpServletResponse response) throws IOException, ServletException, JSONException {
 
 	String param = mapping.getParameter();
 	// -----------------------Survey Learner function ---------------------------
@@ -110,6 +113,14 @@
 	    return retake(mapping, form, request, response);
 	}
 
+	if (param.equals("showOtherUsersAnswers")) {
+	    return showOtherUsersAnswers(mapping, form, request, response);
+	}
+	
+	if (param.equals("getOpenResponses")) {
+	    return getOpenResponses(mapping, form, request, response);
+	}
+
 	if (param.equals("finish")) {
 	    return finish(mapping, form, request, response);
 	}
@@ -138,19 +149,20 @@
 	AnswerForm answerForm = (AnswerForm) form;
 	// initial Session Map
 	SessionMap<String, Object> sessionMap = new SessionMap<String, Object>();
-	request.getSession().setAttribute(sessionMap.getSessionID(), sessionMap);
-	answerForm.setSessionMapID(sessionMap.getSessionID());
+	String sessionMapID = sessionMap.getSessionID();
+	request.getSession().setAttribute(sessionMapID, sessionMap);
+	answerForm.setSessionMapID(sessionMapID);
 
 	// save toolContentID into HTTPSession
 	ToolAccessMode mode = WebUtil.readToolAccessModeParam(request, AttributeNames.PARAM_MODE, true);
 	Long sessionId = new Long(request.getParameter(AttributeNames.PARAM_TOOL_SESSION_ID));
 	// it will be use when submissionDeadline or lock on finish page.
-	request.setAttribute(SurveyConstants.ATTR_SESSION_MAP_ID, sessionMap.getSessionID());
+	request.setAttribute(SurveyConstants.ATTR_SESSION_MAP_ID, sessionMapID);
 
 	// get back the survey and question list and display them on page
 	ISurveyService service = getSurveyService();
 	SurveyUser surveyUser = null;
-	if (mode != null && mode.isTeacher()) {
+	if ((mode != null) && mode.isTeacher()) {
 	    // monitoring mode - user is specified in URL
 	    surveyUser = getSpecifiedUser(service, sessionId,
 		    WebUtil.readIntParam(request, AttributeNames.PARAM_USER_ID, false));
@@ -161,12 +173,9 @@
 	    surveyUser = getCurrentUser(service, sessionId);
 	}
 
-	Survey survey;
 	List<AnswerDTO> answers = service.getQuestionAnswers(sessionId, surveyUser.getUid());
-	survey = service.getSurveyBySessionId(sessionId);
+	Survey survey = service.getSurveyBySessionId(sessionId);
 
-	// check whehter finish lock is on/off
-	boolean lock = survey.getLockWhenFinished() && surveyUser.isSessionFinished();
 
 	// get notebook entry
 	String entryText = new String();
@@ -183,10 +192,14 @@
 	// basic information
 	sessionMap.put(SurveyConstants.ATTR_TITLE, survey.getTitle());
 	sessionMap.put(SurveyConstants.ATTR_SURVEY_INSTRUCTION, survey.getInstructions());
+	// check whehter finish lock is on/off
+	boolean lock = survey.getLockWhenFinished() && surveyUser.isSessionFinished();
 	sessionMap.put(SurveyConstants.ATTR_FINISH_LOCK, lock);
 	sessionMap.put(SurveyConstants.ATTR_LOCK_ON_FINISH, survey.getLockWhenFinished());
 	sessionMap.put(SurveyConstants.ATTR_SHOW_ON_ONE_PAGE, survey.isShowOnePage());
+	sessionMap.put(SurveyConstants.ATTR_SHOW_OTHER_USERS_ANSWERS, survey.isShowOtherUsersAnswers());
 	sessionMap.put(SurveyConstants.ATTR_USER_FINISHED, surveyUser.isSessionFinished());
+	sessionMap.put(SurveyConstants.ATTR_USER, surveyUser);
 
 	sessionMap.put(AttributeNames.PARAM_TOOL_SESSION_ID, sessionId);
 	sessionMap.put(AttributeNames.ATTR_MODE, mode);
@@ -249,11 +262,20 @@
 	} else {
 	    answerForm.setPosition(SurveyConstants.POSITION_FIRST);
 	}
-	// if page is locked, only go to result pages.
-	if (lock) {
+	
+	// if session is finished go to result pages.
+	if (surveyUser.isSessionFinished() && !survey.isShowOtherUsersAnswers()) {
 	    return mapping.findForward(SurveyConstants.FORWARD_RESULT);
+	 
+	//if show other users is ON and response is finalized - show results page with other users answers
+	} else if (survey.isShowOtherUsersAnswers() && surveyUser.isResponseFinalized()) {
+	    ActionRedirect redirect = new ActionRedirect(mapping.findForwardConfig("resultOtherUsers"));
+	    redirect.addParameter(SurveyConstants.ATTR_SESSION_MAP_ID, sessionMapID);
+	    return redirect;
+	    
+	}  else {
+	    return mapping.findForward(SurveyConstants.SUCCESS);    
 	}
-	return mapping.findForward(SurveyConstants.SUCCESS);
     }
 
     private ActionForward nextQuestion(ActionMapping mapping, ActionForm form, HttpServletRequest request,
@@ -262,7 +284,8 @@
 	Integer questionSeqID = answerForm.getQuestionSeqID();
 	String sessionMapID = answerForm.getSessionMapID();
 
-	SessionMap sessionMap = (SessionMap) request.getSession().getAttribute(sessionMapID);
+	SessionMap<String, Object> sessionMap = (SessionMap<String, Object>) request.getSession().getAttribute(
+		sessionMapID);
 	SortedMap<Integer, AnswerDTO> surveyItemMap = getQuestionList(sessionMap);
 
 	ActionErrors errors = getAnswer(request, surveyItemMap.get(questionSeqID));
@@ -303,7 +326,8 @@
 	Integer questionSeqID = answerForm.getQuestionSeqID();
 	String sessionMapID = answerForm.getSessionMapID();
 
-	SessionMap sessionMap = (SessionMap) request.getSession().getAttribute(sessionMapID);
+	SessionMap<String, Object> sessionMap = (SessionMap<String, Object>) request.getSession().getAttribute(
+		sessionMapID);
 	SortedMap<Integer, AnswerDTO> surveyItemMap = getQuestionList(sessionMap);
 
 	ActionErrors errors = getAnswer(request, surveyItemMap.get(questionSeqID));
@@ -337,8 +361,9 @@
 	Integer questionSeqID = answerForm.getQuestionSeqID();
 	answerForm.setPosition(SurveyConstants.POSITION_ONLY_ONE);
 	String sessionMapID = answerForm.getSessionMapID();
+	SessionMap<String, Object> sessionMap = (SessionMap<String, Object>) request.getSession().getAttribute(
+		sessionMapID);
 
-	SessionMap sessionMap = (SessionMap) request.getSession().getAttribute(sessionMapID);
 	SortedMap<Integer, AnswerDTO> surveyItemMap = getQuestionList(sessionMap);
 	// get current question index of total questions
 	int currIdx = new ArrayList<Integer>(surveyItemMap.keySet()).indexOf(questionSeqID) + 1;
@@ -347,30 +372,98 @@
 	return mapping.findForward(SurveyConstants.SUCCESS);
     }
 
+    private ActionForward showOtherUsersAnswers(ActionMapping mapping, ActionForm form, HttpServletRequest request,
+	    HttpServletResponse response) {
+	ISurveyService service = getSurveyService();
+	String sessionMapID = request.getParameter("sessionMapID");
+	request.setAttribute("sessionMapID", sessionMapID);
+	SessionMap<String, Object> sessionMap = (SessionMap<String, Object>) request.getSession().getAttribute(
+		sessionMapID);
+
+	SortedMap<Integer, AnswerDTO> surveyItemMap = getQuestionList(sessionMap);
+	Long sessionId = (Long) sessionMap.get(AttributeNames.PARAM_TOOL_SESSION_ID);
+
+	List<AnswerDTO> answerDtos = new ArrayList<AnswerDTO>();
+	for (SurveyQuestion question : surveyItemMap.values()) {
+	    AnswerDTO answerDto = service.getQuestionResponse(sessionId, question.getUid());
+	    answerDtos.add(answerDto);
+	}
+	request.setAttribute("answerDtos", answerDtos);
+
+	SurveyUser surveyLearner = (SurveyUser) sessionMap.get(SurveyConstants.ATTR_USER);
+	service.setResponseFinalized(surveyLearner.getUid());
+	
+	int countFinishedUser = service.getCountFinishedUsers(sessionId);
+	request.setAttribute(SurveyConstants.ATTR_COUNT_FINISHED_USERS, countFinishedUser);
+
+	return mapping.findForward(SurveyConstants.SUCCESS);
+    }
+    
+    /**
+     * Get OpenResponses.
+     */
+    private ActionForward getOpenResponses(ActionMapping mapping, ActionForm form, HttpServletRequest request,
+	    HttpServletResponse res) throws IOException, ServletException, JSONException {
+	ISurveyService service = getSurveyService();
+	
+	Long questionUid = WebUtil.readLongParam(request, "questionUid");
+	Long sessionId = WebUtil.readLongParam(request, "sessionId");
+	
+	//paging parameters of tablesorter
+	int size = WebUtil.readIntParam(request, "size");
+	int page = WebUtil.readIntParam(request, "page");
+	Integer isSort1 = WebUtil.readIntParam(request, "column[0]", true);
+	
+	int sorting = SurveyConstants.SORT_BY_DEAFAULT;
+	if (isSort1 != null && isSort1.equals(0)) {
+	    sorting = SurveyConstants.SORT_BY_ANSWER_ASC;
+	} else if (isSort1 != null && isSort1.equals(1)) {
+	    sorting = SurveyConstants.SORT_BY_ANSWER_DESC;
+	}
+	
+	List<String> responses = service.getOpenResponsesForTablesorter(sessionId, questionUid, page, size,
+		sorting);
+	
+	JSONArray rows = new JSONArray();
+
+	JSONObject responcedata = new JSONObject();
+	responcedata.put("total_rows", service.getCountResponsesBySessionAndQuestion(sessionId, questionUid));
+	
+	for (String response : responses) {
+	    //JSONArray cell=new JSONArray();
+	    //cell.put(StringEscapeUtils.escapeHtml(user.getFirstName()) + " " + StringEscapeUtils.escapeHtml(user.getLastName()) + " [" + StringEscapeUtils.escapeHtml(user.getLogin()) + "]");
+	    
+	    JSONObject responseRow = new JSONObject();
+	    responseRow.put("answer", StringEscapeUtils.escapeCsv(response));
+//	    responseRow.put("attemptTime", response.getAttemptTime());
+	    
+	    rows.put(responseRow);
+	}
+	responcedata.put("rows", rows);
+	res.setContentType("application/json;charset=utf-8");
+	res.getWriter().print(new String(responcedata.toString()));
+	return null;
+     }
+
     private ActionForward doSurvey(ActionMapping mapping, ActionForm form, HttpServletRequest request,
 	    HttpServletResponse response) {
 	ISurveyService service = getSurveyService();
 
 	AnswerForm answerForm = (AnswerForm) form;
 	Integer questionSeqID = answerForm.getQuestionSeqID();
 	String sessionMapID = answerForm.getSessionMapID();
-	SessionMap sessionMap = (SessionMap) request.getSession().getAttribute(sessionMapID);
+	SessionMap<String, Object> sessionMap = (SessionMap<String, Object>) request.getSession().getAttribute(
+		sessionMapID);
 
 	// validate
 	SortedMap<Integer, AnswerDTO> surveyItemMap = getQuestionList(sessionMap);
 	Collection<AnswerDTO> surveyItemList = surveyItemMap.values();
 
-	SurveyUser surveyLearner = null;
+	SurveyUser surveyLearner = (SurveyUser) sessionMap.get(SurveyConstants.ATTR_USER);
 	Long sessionId = (Long) sessionMap.get(AttributeNames.PARAM_TOOL_SESSION_ID);
 
-	Long userID = WebUtil.readLongParam(request, AttributeNames.PARAM_USER_ID, true);
-	if (userID != null && userID != 0) {
-	    surveyLearner = service.getUserByIDAndSession(userID, sessionId);
-	    request.setAttribute(AttributeNames.PARAM_USER_ID, userID);
-	}
-
 	ActionErrors errors;
-	if (questionSeqID == null || questionSeqID.equals(0)) {
+	if ((questionSeqID == null) || questionSeqID.equals(0)) {
 	    errors = getAnswers(request);
 	} else {
 	    errors = getAnswer(request, surveyItemMap.get(questionSeqID));
@@ -382,8 +475,7 @@
 	List<SurveyAnswer> answerList = new ArrayList<SurveyAnswer>();
 	for (AnswerDTO question : surveyItemList) {
 	    if (question.getAnswer() != null) {
-		if (userID != null && userID != 0)
-		    question.getAnswer().setUser(surveyLearner);
+		question.getAnswer().setUser(surveyLearner);
 		answerList.add(question.getAnswer());
 	    }
 	}
@@ -394,12 +486,9 @@
 
 	Survey survey = service.getSurveyBySessionId(sessionId);
 	if (survey.isNotifyTeachersOnAnswerSumbit()) {
-	    if (surveyLearner == null) {
-		surveyLearner = getCurrentUser(service, sessionId);
-	    }
 	    service.notifyTeachersOnAnswerSumbit(sessionId, surveyLearner);
 	}
-	
+
 	return mapping.findForward(SurveyConstants.SUCCESS);
     }
 
@@ -417,7 +506,8 @@
 
 	// get back SessionMap
 	String sessionMapID = request.getParameter(SurveyConstants.ATTR_SESSION_MAP_ID);
-	SessionMap sessionMap = (SessionMap) request.getSession().getAttribute(sessionMapID);
+	SessionMap<String, Object> sessionMap = (SessionMap<String, Object>) request.getSession().getAttribute(
+		sessionMapID);
 
 	// get mode and ToolSessionID from sessionMAP
 	Long sessionId = (Long) sessionMap.get(AttributeNames.PARAM_TOOL_SESSION_ID);
@@ -464,7 +554,7 @@
 	// get the existing reflection entry
 	ISurveyService submitFilesService = getSurveyService();
 
-	SessionMap map = (SessionMap) request.getSession().getAttribute(sessionMapID);
+	SessionMap<String, Object> map = (SessionMap<String, Object>) request.getSession().getAttribute(sessionMapID);
 	Long toolSessionID = (Long) map.get(AttributeNames.PARAM_TOOL_SESSION_ID);
 	NotebookEntry entry = submitFilesService.getEntry(toolSessionID, CoreNotebookConstants.NOTEBOOK_TOOL,
 		SurveyConstants.TOOL_SIGNATURE, user.getUserID());
@@ -491,7 +581,8 @@
 	Integer userId = refForm.getUserID();
 
 	String sessionMapID = WebUtil.readStrParam(request, SurveyConstants.ATTR_SESSION_MAP_ID);
-	SessionMap sessionMap = (SessionMap) request.getSession().getAttribute(sessionMapID);
+	SessionMap<String, Object> sessionMap = (SessionMap<String, Object>) request.getSession().getAttribute(
+		sessionMapID);
 	Long sessionId = (Long) sessionMap.get(AttributeNames.PARAM_TOOL_SESSION_ID);
 
 	ISurveyService service = getSurveyService();
@@ -524,7 +615,8 @@
 	ActionErrors errors = new ActionErrors();
 	// get sessionMap
 	String sessionMapID = request.getParameter(SurveyConstants.ATTR_SESSION_MAP_ID);
-	SessionMap sessionMap = (SessionMap) request.getSession().getAttribute(sessionMapID);
+	SessionMap<String, Object> sessionMap = (SessionMap<String, Object>) request.getSession().getAttribute(
+		sessionMapID);
 	Long sessionID = (Long) sessionMap.get(AttributeNames.PARAM_TOOL_SESSION_ID);
 
 	SurveyAnswer answer = getAnswerFromPage(request, answerDto, sessionID);
@@ -546,7 +638,8 @@
 	ActionErrors errors = new ActionErrors();
 	// get sessionMap
 	String sessionMapID = request.getParameter(SurveyConstants.ATTR_SESSION_MAP_ID);
-	SessionMap sessionMap = (SessionMap) request.getSession().getAttribute(sessionMapID);
+	SessionMap<String, Object> sessionMap = (SessionMap<String, Object>) request.getSession().getAttribute(
+		sessionMapID);
 	Long sessionID = (Long) sessionMap.get(AttributeNames.PARAM_TOOL_SESSION_ID);
 	Collection<AnswerDTO> answerDtoList = getQuestionList(sessionMap).values();
 
@@ -563,14 +656,14 @@
 
     private void validateAnswers(HttpServletRequest request, AnswerDTO question, ActionErrors errors,
 	    SurveyAnswer answer) {
-	boolean isAnswerEmpty = (answer.getChoices() == null && StringUtils.isBlank(answer.getAnswerText()));
+	boolean isAnswerEmpty = ((answer.getChoices() == null) && StringUtils.isBlank(answer.getAnswerText()));
 
 	// for mandatory questions, answer can not be null.
 	if (!question.isOptional() && isAnswerEmpty) {
 	    errors.add(SurveyConstants.ERROR_MSG_KEY + question.getUid(), new ActionMessage(
 		    SurveyConstants.ERROR_MSG_MANDATORY_QUESTION));
 	}
-	if (question.getType() == SurveyConstants.QUESTION_TYPE_SINGLE_CHOICE && question.isAppendText()
+	if ((question.getType() == SurveyConstants.QUESTION_TYPE_SINGLE_CHOICE) && question.isAppendText()
 		&& !isAnswerEmpty) {
 	    // for single choice, user only can choose one option or open text (if it has)
 	    if (!StringUtils.isBlank(answer.getAnswerChoices()) && !StringUtils.isBlank(answer.getAnswerText())) {
@@ -613,7 +706,7 @@
      * @param request
      * @return
      */
-    private SortedMap<Integer, AnswerDTO> getQuestionList(SessionMap sessionMap) {
+    private SortedMap<Integer, AnswerDTO> getQuestionList(SessionMap<String, Object> sessionMap) {
 	SortedMap<Integer, AnswerDTO> list = (SortedMap<Integer, AnswerDTO>) sessionMap
 		.get(SurveyConstants.ATTR_QUESTION_LIST);
 	if (list == null) {
Index: lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/web/action/MonitoringAction.java
===================================================================
diff -u -r8b97231e320c0c5b674f07c14da711f232ba9e1c -r098d686653002409f33aff0f69b1183e07c1eab3
--- lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/web/action/MonitoringAction.java	(.../MonitoringAction.java)	(revision 8b97231e320c0c5b674f07c14da711f232ba9e1c)
+++ lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/web/action/MonitoringAction.java	(.../MonitoringAction.java)	(revision 098d686653002409f33aff0f69b1183e07c1eab3)
@@ -27,14 +27,14 @@
 import java.io.ByteArrayOutputStream;
 import java.io.IOException;
 import java.io.PrintWriter;
+import java.util.Date;
 import java.util.List;
 import java.util.Map;
+import java.util.Map.Entry;
 import java.util.Set;
 import java.util.SortedMap;
-import java.util.TreeMap;
-import java.util.Map.Entry;
-import java.util.Date;
 import java.util.TimeZone;
+import java.util.TreeMap;
 
 import javax.servlet.ServletException;
 import javax.servlet.http.HttpServletRequest;
@@ -66,12 +66,12 @@
 import org.lamsfoundation.lams.tool.survey.util.SurveyUserComparator;
 import org.lamsfoundation.lams.tool.survey.util.SurveyWebUtils;
 import org.lamsfoundation.lams.usermanagement.dto.UserDTO;
+import org.lamsfoundation.lams.util.DateUtil;
 import org.lamsfoundation.lams.util.MessageService;
 import org.lamsfoundation.lams.util.WebUtil;
-import org.lamsfoundation.lams.util.DateUtil;
+import org.lamsfoundation.lams.web.session.SessionManager;
 import org.lamsfoundation.lams.web.util.AttributeNames;
 import org.lamsfoundation.lams.web.util.SessionMap;
-import org.lamsfoundation.lams.web.session.SessionManager;
 import org.springframework.web.context.WebApplicationContext;
 import org.springframework.web.context.support.WebApplicationContextUtils;
 
@@ -456,8 +456,7 @@
      * @param string
      * @return
      */
-    private String removeHTMLTags(String string)
-    {
+    private String removeHTMLTags(String string) {
 	return string.replaceAll("\\<.*?>", "").replaceAll("&nbsp;", " ");
     }
 
Index: lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/web/form/ReflectionForm.java
===================================================================
diff -u -r1ee653aae7ac1b2ae1e30254a8e6bd2dcdf95bad -r098d686653002409f33aff0f69b1183e07c1eab3
--- lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/web/form/ReflectionForm.java	(.../ReflectionForm.java)	(revision 1ee653aae7ac1b2ae1e30254a8e6bd2dcdf95bad)
+++ lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/web/form/ReflectionForm.java	(.../ReflectionForm.java)	(revision 098d686653002409f33aff0f69b1183e07c1eab3)
@@ -24,12 +24,7 @@
 /* $$Id$$ */	
 package org.lamsfoundation.lams.tool.survey.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;
 
 /**
Index: lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/web/form/SurveyForm.java
===================================================================
diff -u -r8b97231e320c0c5b674f07c14da711f232ba9e1c -r098d686653002409f33aff0f69b1183e07c1eab3
--- lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/web/form/SurveyForm.java	(.../SurveyForm.java)	(revision 8b97231e320c0c5b674f07c14da711f232ba9e1c)
+++ lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/web/form/SurveyForm.java	(.../SurveyForm.java)	(revision 098d686653002409f33aff0f69b1183e07c1eab3)
@@ -33,101 +33,95 @@
 import org.lamsfoundation.lams.tool.survey.model.Survey;
 
 /**
- *
- * 	Survey Form.
- *	@struts.form name="surveyForm"
- *
+ * Survey Form.
+ * 
+ * @struts.form name="surveyForm"
+ * 
  * User: Dapeng.Ni
  */
-public class SurveyForm extends ActionForm  {
-	private static final long serialVersionUID = 3599879328307492312L;
+public class SurveyForm extends ActionForm {
+    private static final long serialVersionUID = 3599879328307492312L;
 
-	private static Logger logger = Logger.getLogger(SurveyForm.class.getName());
+    private static Logger logger = Logger.getLogger(SurveyForm.class.getName());
 
-	//Forum fields
-	private String sessionMapID;
-	private String contentFolderID;
-	private int currentTab;
+    // Forum fields
+    private String sessionMapID;
+    private String contentFolderID;
+    private int currentTab;
     private FormFile offlineFile;
     private FormFile onlineFile;
 
     private Survey survey;
-    
-    public SurveyForm(){
-    	survey = new Survey();
-    	survey.setTitle("Survey");
-    	currentTab = 1;
+
+    public SurveyForm() {
+	survey = new Survey();
+	survey.setTitle("Survey");
+	currentTab = 1;
     }
-	
-	public void setSurvey(Survey survey) {
-        this.survey = survey;
-        //set Form special varaible from given forum
-        if(survey == null){
-        	logger.error("Initial SurveyForum failed by null value of Survey.");
-        }
+
+    public void setSurvey(Survey survey) {
+	this.survey = survey;
+	// set Form special varaible from given forum
+	if (survey == null) {
+	    logger.error("Initial SurveyForum failed by null value of Survey.");
 	}
-    public void reset(ActionMapping mapping, HttpServletRequest request){
-    	String param = mapping.getParameter();
-    	//if it is start page, all data read out from database or current session
-    	//so need not reset checkbox to refresh value!
-    	if(!StringUtils.equals(param,"start") && !StringUtils.equals(param,"initPage")){
-	    	survey.setLockWhenFinished(false);
-	    	survey.setDefineLater(false);
-	    	survey.setReflectOnActivity(false);
-	    	survey.setShowOnePage(false);
-	    	
-    	}
     }
 
-	public int getCurrentTab() {
-		return currentTab;
+    public void reset(ActionMapping mapping, HttpServletRequest request) {
+	String param = mapping.getParameter();
+	// if it is start page, all data read out from database or current session
+	// so need not reset checkbox to refresh value!
+	if (!StringUtils.equals(param, "start") && !StringUtils.equals(param, "initPage")) {
+	    survey.setLockWhenFinished(false);
+	    survey.setDefineLater(false);
+	    survey.setReflectOnActivity(false);
+	    survey.setShowOnePage(false);
+	    survey.setShowOtherUsersAnswers(false);
 	}
+    }
 
+    public int getCurrentTab() {
+	return currentTab;
+    }
 
-	public void setCurrentTab(int currentTab) {
-		this.currentTab = currentTab;
-	}
+    public void setCurrentTab(int currentTab) {
+	this.currentTab = currentTab;
+    }
 
+    public FormFile getOfflineFile() {
+	return offlineFile;
+    }
 
-	public FormFile getOfflineFile() {
-		return offlineFile;
-	}
+    public void setOfflineFile(FormFile offlineFile) {
+	this.offlineFile = offlineFile;
+    }
 
+    public FormFile getOnlineFile() {
+	return onlineFile;
+    }
 
-	public void setOfflineFile(FormFile offlineFile) {
-		this.offlineFile = offlineFile;
-	}
+    public void setOnlineFile(FormFile onlineFile) {
+	this.onlineFile = onlineFile;
+    }
 
+    public Survey getSurvey() {
+	return survey;
+    }
 
-	public FormFile getOnlineFile() {
-		return onlineFile;
-	}
+    public String getSessionMapID() {
+	return sessionMapID;
+    }
 
+    public void setSessionMapID(String sessionMapID) {
+	this.sessionMapID = sessionMapID;
+    }
 
-	public void setOnlineFile(FormFile onlineFile) {
-		this.onlineFile = onlineFile;
-	}
+    public String getContentFolderID() {
+	return contentFolderID;
+    }
 
+    public void setContentFolderID(String contentFolderID) {
+	this.contentFolderID = contentFolderID;
+    }
 
-	public Survey getSurvey() {
-		return survey;
-	}
-
-	public String getSessionMapID() {
-		return sessionMapID;
-	}
-
-	public void setSessionMapID(String sessionMapID) {
-		this.sessionMapID = sessionMapID;
-	}
-
-	public String getContentFolderID() {
-		return contentFolderID;
-	}
-
-	public void setContentFolderID(String contentFolderID) {
-		this.contentFolderID = contentFolderID;
-	}
-
-
 }
Index: lams_tool_survey/web/common/header.jsp
===================================================================
diff -u -r7db98069c542476a8a3aea28e0822b751e120085 -r098d686653002409f33aff0f69b1183e07c1eab3
--- lams_tool_survey/web/common/header.jsp	(.../header.jsp)	(revision 7db98069c542476a8a3aea28e0822b751e120085)
+++ lams_tool_survey/web/common/header.jsp	(.../header.jsp)	(revision 098d686653002409f33aff0f69b1183e07c1eab3)
@@ -8,7 +8,6 @@
 
 <!-- ********************  javascript ********************** -->
 <script type="text/javascript" src="${lams}includes/javascript/common.js"></script>
-<script type="text/javascript" src="${lams}includes/javascript/prototype.js"></script>
 <script type="text/javascript" src="${lams}includes/javascript/tabcontroller.js"></script>
 
 	
Index: lams_tool_survey/web/pages/authoring/addCondition.jsp
===================================================================
diff -u -rc2ba0dc3e2d99441897de8d185e83c419a416f2d -r098d686653002409f33aff0f69b1183e07c1eab3
--- lams_tool_survey/web/pages/authoring/addCondition.jsp	(.../addCondition.jsp)	(revision c2ba0dc3e2d99441897de8d185e83c419a416f2d)
+++ lams_tool_survey/web/pages/authoring/addCondition.jsp	(.../addCondition.jsp)	(revision 098d686653002409f33aff0f69b1183e07c1eab3)
@@ -5,6 +5,7 @@
 <lams:html>
 	<lams:head>
 		<%@ include file="/common/header.jsp"%>
+		<script type="text/javascript" src="${lams}includes/javascript/prototype.js"></script>
 		<lams:css style="tabbed" />
 		<script type="text/javascript"> 
 			function callHideConditionMessage() {
Index: lams_tool_survey/web/pages/authoring/advance.jsp
===================================================================
diff -u -r05ab55ef4acdc0c374af7405c58fb0e0fe8eb6b7 -r098d686653002409f33aff0f69b1183e07c1eab3
--- lams_tool_survey/web/pages/authoring/advance.jsp	(.../advance.jsp)	(revision 05ab55ef4acdc0c374af7405c58fb0e0fe8eb6b7)
+++ lams_tool_survey/web/pages/authoring/advance.jsp	(.../advance.jsp)	(revision 098d686653002409f33aff0f69b1183e07c1eab3)
@@ -1,46 +1,45 @@
 <%@ include file="/common/taglibs.jsp"%>
-<c:set var="formBean"
-	value="<%=request
-									.getAttribute(org.apache.struts.taglib.html.Constants.BEAN_KEY)%>" />
+<c:set var="formBean" value="<%=request.getAttribute(org.apache.struts.taglib.html.Constants.BEAN_KEY)%>" />
 
 <!-- Advance Tab Content -->
 
 <p class="small-space-top">
-	<html:checkbox property="survey.lockWhenFinished" styleClass="noBorder" styleId="lockWhenFinished">
-	</html:checkbox>
+	<html:checkbox property="survey.lockWhenFinished" styleClass="noBorder" styleId="lockWhenFinished"/>
 	<label for="lockWhenFinished">
 		<fmt:message key="label.authoring.advance.lock.on.finished" />
 	</label>
 </p>
 
 <p>
-	<html:checkbox property="survey.showOnePage" styleClass="noBorder" styleId="showOnePage">
-	</html:checkbox>
+	<html:checkbox property="survey.showOnePage" styleClass="noBorder" styleId="showOnePage"/>
 	<label for="showOnePage">
 		<fmt:message key="label.authoring.advance.show.on.one.page" />
 	</label>
 </p>
+
 <p>
-	<html:checkbox property="survey.notifyTeachersOnAnswerSumbit"
-		styleClass="noBorder" styleId="notifyTeachersOnAnswerSumbit">
-	</html:checkbox>
+	<html:checkbox property="survey.showOtherUsersAnswers" styleClass="noBorder" styleId="showOtherUsersAnswers"/>
+	<label for="showOtherUsersAnswers">
+		<fmt:message key="label.show.answers.from.other.users" />
+	</label>
+</p>
+
+<p>
+	<html:checkbox property="survey.notifyTeachersOnAnswerSumbit" styleClass="noBorder" styleId="notifyTeachersOnAnswerSumbit"/>
 	<label for="notifyTeachersOnAnswerSumbit">
 		<fmt:message key="label.authoring.advanced.notify.onanswersubmit" />
 	</label>
 </p>
 
 <p>
-	<html:checkbox property="survey.reflectOnActivity"
-		styleClass="noBorder" styleId="reflectOn">
-	</html:checkbox>
+	<html:checkbox property="survey.reflectOnActivity" styleClass="noBorder" styleId="reflectOn"/>
 	<label for="reflectOn">
 		<fmt:message key="label.authoring.advanced.reflectOnActivity" />
 	</label>
 </p>
 
 <p>
-	<html:textarea property="survey.reflectInstructions"
-		styleId="reflectInstructions" cols="30" rows="3" />
+	<html:textarea property="survey.reflectInstructions" styleId="reflectInstructions" cols="30" rows="3" />
 </p>
 
 <script type="text/javascript">
Index: lams_tool_survey/web/pages/export/exportportfolio.jsp
===================================================================
diff -u -r77bcef245dda8b166cdf40c12c62a7a5ab8d6fde -r098d686653002409f33aff0f69b1183e07c1eab3
--- lams_tool_survey/web/pages/export/exportportfolio.jsp	(.../exportportfolio.jsp)	(revision 77bcef245dda8b166cdf40c12c62a7a5ab8d6fde)
+++ lams_tool_survey/web/pages/export/exportportfolio.jsp	(.../exportportfolio.jsp)	(revision 098d686653002409f33aff0f69b1183e07c1eab3)
@@ -1,16 +1,15 @@
-<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01//EN"
-        "http://www.w3.org/TR/html4/strict.dtd">
+<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01//EN" "http://www.w3.org/TR/html4/strict.dtd">
 
-
 <%@ include file="/common/taglibs.jsp"%>
 <%@ page import="org.lamsfoundation.lams.tool.survey.SurveyConstants"%>
 <c:set var="tool"><lams:WebAppURL/></c:set>
 <c:set var="sessionMapID" value="${param.sessionMapID}"/>
 <c:set var="sessionMap" value="${sessionScope[sessionMapID]}"/>
 <lams:html>
 <lams:head>
-	    <%@ include file="/common/header.jsp" %>
+	<%@ include file="/common/header.jsp" %>
 </lams:head>
+
 <body class="stripes">
 		<div id="content">
 		<h1>
Index: lams_tool_survey/web/pages/learning/learning.jsp
===================================================================
diff -u -r9cde79c9a334706527edef8b9cac05931f6649a8 -r098d686653002409f33aff0f69b1183e07c1eab3
--- lams_tool_survey/web/pages/learning/learning.jsp	(.../learning.jsp)	(revision 9cde79c9a334706527edef8b9cac05931f6649a8)
+++ lams_tool_survey/web/pages/learning/learning.jsp	(.../learning.jsp)	(revision 098d686653002409f33aff0f69b1183e07c1eab3)
@@ -1,48 +1,43 @@
-<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01//EN"
-        "http://www.w3.org/TR/html4/strict.dtd">
+<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01//EN" "http://www.w3.org/TR/html4/strict.dtd">
 
 <%@ include file="/common/taglibs.jsp"%>
 <lams:html>
 <lams:head>
 	<title><fmt:message key="label.learning.title" />
 	</title>
 	<%@ include file="/common/header.jsp"%>
-
+	
+	<script type="text/javascript" src="${lams}includes/javascript/prototype.js"></script>
 	<script type="text/javascript">
-	<!--
-		
 		function previousQuestion(sessionMapID){
 			$("surveyForm").action = '<c:url value="/learning/previousQuestion.do"/>';
 			$("surveyForm").submit();
 		}
+		
 		function nextQuestion(sessionMapID){
 			$("surveyForm").action = '<c:url value="/learning/nextQuestion.do"/>';
 			$("surveyForm").submit();
 		}
+		
 		function singleChoice(choiceName){
 			var rs = document.getElementsByName(choiceName);
 			for(idx=0;idx<rs.length;idx++)
 				rs[idx].checked=false;
-		}
-	-->        
+		}  
     </script>
 </lams:head>
 <body class="stripes">
-	<html:form action="/learning/doSurvey" method="post"
-		styleId="surveyForm">
-		<c:set var="formBean"
-			value="<%=request.getAttribute(org.apache.struts.taglib.html.Constants.BEAN_KEY)%>" />
+	<html:form action="/learning/doSurvey" method="post" styleId="surveyForm">
+		<c:set var="formBean" value="<%=request.getAttribute(org.apache.struts.taglib.html.Constants.BEAN_KEY)%>" />
 		<html:hidden property="questionSeqID" />
 		<html:hidden property="sessionMapID" />
 		<html:hidden property="position" />
 		<html:hidden property="currentIdx" />
-		<html:hidden property="userID" />
 		<c:set var="sessionMapID" value="${formBean.sessionMapID}" />
 		<c:set var="sessionMap" value="${sessionScope[sessionMapID]}" />
 		<c:set var="position" value="${formBean.position}" />
 		<c:set var="questionSeqID" value="${formBean.questionSeqID}" />
 		<c:set var="currentIdx" value="${formBean.currentIdx}" />
-		<c:set var="userID" value="${formBean.userID}" />
 
 		<div id="content">
 			<h1>
@@ -60,16 +55,14 @@
 			</c:if>	
 			<c:choose>
 				<%-- Show on one page or when learner does not choose edit one question --%>
-				<c:when
-					test="${sessionMap.showOnOnePage && (empty questionSeqID or questionSeqID == 0)}">
+				<c:when test="${sessionMap.showOnOnePage && (empty questionSeqID or questionSeqID == 0)}">
 					<c:forEach var="element" items="${sessionMap.questionList}">
 						<c:set var="question" value="${element.value}" />
 						<%@ include file="/pages/learning/question.jsp"%>
 					</c:forEach>
 				</c:when>
 				<c:otherwise>
-					<c:set var="question"
-						value="${sessionMap.questionList[questionSeqID]}" />
+					<c:set var="question" value="${sessionMap.questionList[questionSeqID]}" />
 					<fmt:message key="label.question" /> ${currentIdx} <fmt:message
 						key="label.of" /> ${sessionMap.totalQuestions}
 						<%@ include file="/pages/learning/question.jsp"%>
Index: lams_tool_survey/web/pages/learning/notebook.jsp
===================================================================
diff -u -r40c565b9307a4b3fcc2f724b1baf104c6c8a2dcf -r098d686653002409f33aff0f69b1183e07c1eab3
--- lams_tool_survey/web/pages/learning/notebook.jsp	(.../notebook.jsp)	(revision 40c565b9307a4b3fcc2f724b1baf104c6c8a2dcf)
+++ lams_tool_survey/web/pages/learning/notebook.jsp	(.../notebook.jsp)	(revision 098d686653002409f33aff0f69b1183e07c1eab3)
@@ -10,10 +10,10 @@
 		function disableFinishButton() {
 			document.getElementById("finishButton").disabled = true;
 		}
-	         function submitForm(methodName){
-	                var f = document.getElementById('messageForm');
-	                f.submit();
-	        }
+		function submitForm(methodName){
+			var f = document.getElementById('messageForm');
+			f.submit();
+		}
 	</script>	
 </lams:head>
 <body class="stripes">
Index: lams_tool_survey/web/pages/learning/question.jsp
===================================================================
diff -u -r40c565b9307a4b3fcc2f724b1baf104c6c8a2dcf -r098d686653002409f33aff0f69b1183e07c1eab3
--- lams_tool_survey/web/pages/learning/question.jsp	(.../question.jsp)	(revision 40c565b9307a4b3fcc2f724b1baf104c6c8a2dcf)
+++ lams_tool_survey/web/pages/learning/question.jsp	(.../question.jsp)	(revision 098d686653002409f33aff0f69b1183e07c1eab3)
@@ -45,8 +45,7 @@
 
 	<c:if test="${question.type == 3}">
 		<p>
-			<textarea name="optionText${question.uid}" rows="7" cols="50" class="text-area"
-				type="_moz">${answerText}</textarea>
+			<lams:textarea name="optionText${question.uid}" rows="7" cols="50" class="text-area">${answerText}</lams:textarea>
 		</p>
 	</c:if>
 	<c:if test="${question.appendText}">
Index: lams_tool_survey/web/pages/learning/result.jsp
===================================================================
diff -u -r61d957b69e2230effe4ab96242233c281e0bba44 -r098d686653002409f33aff0f69b1183e07c1eab3
--- lams_tool_survey/web/pages/learning/result.jsp	(.../result.jsp)	(revision 61d957b69e2230effe4ab96242233c281e0bba44)
+++ lams_tool_survey/web/pages/learning/result.jsp	(.../result.jsp)	(revision 098d686653002409f33aff0f69b1183e07c1eab3)
@@ -1,39 +1,39 @@
-<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01//EN"
-        "http://www.w3.org/TR/html4/strict.dtd">
+<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01//EN" "http://www.w3.org/TR/html4/strict.dtd">
 
 <%@ include file="/common/taglibs.jsp"%>
 <lams:html>
 <c:set var="sessionMap" value="${sessionScope[sessionMapID]}" />
 <lams:head>
-	<title><fmt:message key="label.learning.title" />
-	</title>
+	<title><fmt:message key="label.learning.title" /></title>
 	<%@ include file="/common/header.jsp"%>
 
 	<script type="text/javascript">
-	<!--
-		
 		function finishSession(){
 			document.getElementById("finishButton").disabled = true;
 			document.location.href ='<c:url value="/learning/finish.do?sessionMapID=${sessionMapID}"/>';
 			return false;
 		}
+		
+		function showOtherUsersAnswers(){
+			document.location.href='<c:url value="/learning/showOtherUsersAnswers.do?sessionMapID=${sessionMapID}"/>';
+		}
+		
 		function continueReflect(){
 			document.location.href='<c:url value="/learning/newReflection.do?sessionMapID=${sessionMapID}"/>';
 		}
+		
 		function retakeSurvey(questionSeqId){
 			//retake for all questions
-			if(questionSeqId == -1)
-				document.location.href='<c:url value="/learning/start.do?mode=${sessionMap.mode}&toolSessionID=${sessionMap.toolSessionID}&userID=${userID}"/>';
-			else
+			if(questionSeqId == -1) {
+				document.location.href='<c:url value="/learning/retake.do?sessionMapID=${sessionMapID}"/>';
+			} else {
 				document.location.href='<c:url value="/learning/retake.do?sessionMapID=${sessionMapID}&questionSeqID="/>' +questionSeqId ;
+			}
 		}
-	-->        
     </script>
-
 </lams:head>
 <body class="stripes">
 
-
 	<div id="content">
 		<h1>
 			<c:out value="${sessionMap.title}" escapeXml="true"/>
@@ -43,16 +43,37 @@
 		</p>
 
 		<c:if test="${sessionMap.lockOnFinish and sessionMap.mode != 'teacher'}">
-				<div class="info">
-					<c:choose>
-						<c:when test="${sessionMap.userFinished}">
-							<fmt:message key="message.activityLocked" />
-						</c:when>
-						<c:otherwise>
-							<fmt:message key="message.warnLockOnFinish" />
-						</c:otherwise>
-					</c:choose>
-				</div>
+			<div class="info space-bottom">
+				<c:choose>
+					<c:when test="${sessionMap.showOtherUsersAnswers}">
+						<c:set var="buttonLabel"><fmt:message key="label.view.all.responses" /></c:set>
+						<fmt:message key="message.warnLockOnFinish" >
+							<fmt:param>${buttonLabel}</fmt:param>
+						</fmt:message>
+					</c:when>
+								
+					<c:when test="${sessionMap.userFinished}">
+						<fmt:message key="message.activityLocked" />
+					</c:when>
+					
+					<c:otherwise>
+						<c:set var="buttonLabel">
+							<c:choose>
+							 	<c:when test="${sessionMap.activityPosition.last}">
+							 		<fmt:message key="label.submit" />
+							 	</c:when>
+							 					
+							 	<c:otherwise>
+							 		 <fmt:message key="label.finished" />
+							 	</c:otherwise>
+							 </c:choose>
+						</c:set>
+						<fmt:message key="message.warnLockOnFinish" >
+							<fmt:param>${buttonLabel}</fmt:param>
+						</fmt:message>
+					</c:otherwise>
+				</c:choose>
+			</div>
 		</c:if>
 		
 		<c:forEach var="element" items="${sessionMap.questionList}">
@@ -65,10 +86,10 @@
 
 				<c:out value="${question.description}" escapeXml="false" />
 
-				<c:if
-					test="${(not sessionMap.finishedLock) && (not sessionMap.showOnOnePage)}">
-					<a href="#" onclick="retakeSurvey(${question.sequenceId})"><fmt:message
-							key="label.retake" /> </a>
+				<c:if test="${(not sessionMap.finishedLock) && (not sessionMap.showOnOnePage)}">
+					<a href="#" onclick="retakeSurvey(${question.sequenceId})">
+						<fmt:message key="label.retake" /> 
+					</a>
 				</c:if>
 
 				<p>
@@ -96,7 +117,7 @@
 
 				</p>
 				<c:if test="${question.type == 3}">
-					<lams:out value="${answerText}" escapeHtml="true"/>
+					<lams:out value="${answerText}" escapeHtml="false"/>
 				</c:if>
 
 				<c:if test="${question.appendText && (not empty answerText)}">
@@ -107,13 +128,12 @@
 		</c:forEach>
 
 		<c:if test="${not sessionMap.finishedLock}">
-			<html:button property="RetakeButton"
-				onclick="return retakeSurvey(-1)" styleClass="button">
+			<html:button property="RetakeButton" onclick="return retakeSurvey(-1)" styleClass="button">
 				<fmt:message key="label.retake.survey" />
 			</html:button>
 		</c:if>
 
-		<c:if test="${sessionMap.userFinished and sessionMap.reflectOn}">
+		<c:if test="${sessionMap.userFinished and sessionMap.reflectOn and !sessionMap.showOtherUsersAnswers}">
 			<div class="small-space-top">
 				<h2><fmt:message key="title.reflection" /></h2>
 				<strong>
@@ -123,8 +143,7 @@
 				<c:choose>
 					<c:when test="${empty sessionMap.reflectEntry}">
 						<p>
-							<em> <fmt:message key="message.no.reflection.available" />
-							</em>
+							<em> <fmt:message key="message.no.reflection.available" /></em>
 						</p>
 					</c:when>
 					<c:otherwise>
@@ -134,8 +153,7 @@
 					</c:otherwise>
 				</c:choose>
 
-				<html:button property="ContinueButton"
-					onclick="return continueReflect()" styleClass="button">
+				<html:button property="ContinueButton" onclick="return continueReflect()" styleClass="button">
 					<fmt:message key="label.edit" />
 				</html:button>
 			</div>
@@ -145,13 +163,20 @@
 			<table>
 				<tr>
 					<td>
-						<span class="right-buttons"> <c:choose>
+						<span class="right-buttons">
+							<c:choose>
+								<c:when test="${sessionMap.showOtherUsersAnswers}">
+									<html:button property="otherUsersAnswersButton" onclick="return showOtherUsersAnswers()" styleClass="button">
+										<fmt:message key="label.view.all.responses" />
+									</html:button>
+								</c:when>
+							
 								<c:when test="${sessionMap.reflectOn}">
-									<html:button property="ContinueButton"
-										onclick="return continueReflect()" styleClass="button">
+									<html:button property="ContinueButton" onclick="return continueReflect()" styleClass="button">
 										<fmt:message key="label.continue" />
 									</html:button>
 								</c:when>
+								
 								<c:otherwise>
 									<html:link href="#nogo" property="FinishButton" styleId="finishButton"
 										onclick="return finishSession()" styleClass="button">
@@ -160,14 +185,16 @@
 							 					<c:when test="${sessionMap.activityPosition.last}">
 							 						<fmt:message key="label.submit" />
 							 					</c:when>
+							 					
 							 					<c:otherwise>
 							 		 				<fmt:message key="label.finished" />
 							 					</c:otherwise>
 							 				</c:choose>
 							 			</span>
 									</html:link>
 								</c:otherwise>
-							</c:choose> </span>
+							</c:choose> 
+						</span>
 					</td>
 				</tr>
 			</table>
Index: lams_tool_survey/web/pages/learning/resultOtherUsers.jsp
===================================================================
diff -u
--- lams_tool_survey/web/pages/learning/resultOtherUsers.jsp	(revision 0)
+++ lams_tool_survey/web/pages/learning/resultOtherUsers.jsp	(revision 098d686653002409f33aff0f69b1183e07c1eab3)
@@ -0,0 +1,365 @@
+<!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="lams"><lams:LAMSURL /></c:set>
+<c:set var="sessionMap" value="${sessionScope[sessionMapID]}" />
+
+<lams:html>
+<lams:head>
+	<title><fmt:message key="label.learning.title" /></title>
+	<%@ include file="/common/header.jsp"%>
+	<link rel="stylesheet" href="${lams}css/jquery.tablesorter.theme-blue.css">
+	<link rel="stylesheet" href="${lams}css/jquery.tablesorter.pager.css">
+	<style media="screen,projection" type="text/css">
+		#other-users-answers-title {
+			margin-top: 20px;
+		}
+		
+		table.alternative-color td:first-child {
+			width: 25%;
+		}
+	</style>
+	
+	<script type="text/javascript" src="${lams}includes/javascript/jquery.js"></script>
+	<script type="text/javascript" src="${lams}includes/javascript/raphael/raphael.js"></script>
+	<script type="text/javascript" src="${lams}includes/javascript/raphael/g.raphael.js"></script>
+	<script type="text/javascript" src="${lams}includes/javascript/raphael/g.pie.js"></script>
+	<script type="text/javascript" src="${lams}includes/javascript/raphael/chart.js"></script>
+	<script type="text/javascript" src="${lams}includes/javascript/jquery.tablesorter.js"></script>
+	<script type="text/javascript" src="${lams}includes/javascript/jquery.tablesorter-pager.js"></script>
+
+	<script type="text/javascript">
+		var chartDataUrl = '<lams:WebAppURL />showChart.do';
+		var alwaysFetchChartValues = true;
+	
+		$(document).ready(function(){
+	
+			$(".tablesorter").tablesorter({
+				theme: 'blue',
+			    widthFixed: true,
+			    widgets: ['zebra']
+			});
+			
+			$(".tablesorter").each(function() {
+				$(this).tablesorterPager({
+					// set to false otherwise it remembers setting from other jsFiddle demos
+					savePages: false,
+				      // use this format: "http:/mydatabase.com?page={page}&size={size}&{sortList:col}"
+				      // where {page} is replaced by the page number (or use {page+1} to get a one-based index),
+				      // {size} is replaced by the number of records to show,
+				      // {sortList:col} adds the sortList to the url into a "col" array, and {filterList:fcol} adds
+				      // the filterList to the url into an "fcol" array.
+				      // So a sortList = [[2,0],[3,0]] becomes "&col[2]=0&col[3]=0" in the url
+					ajaxUrl : "<c:url value='/learning/getOpenResponses.do'/>?page={page}&size={size}&{sortList:column}&sessionId=${sessionMap.toolSessionID}&questionUid=" + $(this).attr('data-question-uid'),
+					ajaxProcessing: function (data) {
+						
+				    	if (data && data.hasOwnProperty('rows')) {
+				    		var rows = [],
+				            json = {};
+				    		
+							for (i = 0; i < data.rows.length; i++){
+								var userData = data.rows[i];
+								
+								rows += '<tr>';
+								rows += 	'<td>';
+								rows += 		'<div class="user-answer">';
+								rows += 			userData["answer"];
+								rows += 		'</div>';
+								rows += 	'</td>';
+								rows += '</tr>';
+							}
+				            
+							json.total = data.total_rows; // only allow 100 rows in total
+							//json.filteredRows = 100; // no filtering
+							json.rows = $(rows);
+							return json;
+				    	}
+					},
+				    container: $(this).next(".pager"),
+				    output: '{startRow} to {endRow} ({totalRows})',// possible variables: {page}, {totalPages}, {filteredPages}, {startRow}, {endRow}, {filteredRows} and {totalRows}
+				    // if true, the table will remain the same height no matter how many records are displayed. The space is made up by an empty
+				    // table row set to a height to compensate; default is false
+				    fixedHeight: true,
+				    // remove rows from the table to speed up the sort of large tables.
+				    // setting this to false, only hides the non-visible rows; needed if you plan to add/remove rows with the pager enabled.
+				    removeRows: false,
+				    // css class names of pager arrows
+				    cssNext: '.tablesorter-next', // next page arrow
+					cssPrev: '.tablesorter-prev', // previous page arrow
+					cssFirst: '.tablesorter-first', // go to first page arrow
+					cssLast: '.tablesorter-last', // go to last page arrow
+					cssGoto: '.gotoPage', // select dropdown to allow choosing a page
+					cssPageDisplay: '.pagedisplay', // location of where the "output" is displayed
+					cssPageSize: '.pagesize', // page size selector - select dropdown that sets the "size" option
+					// class added to arrows when at the extremes (i.e. prev/first arrows are "disabled" when on the first page)
+					cssDisabled: 'disabled' // Note there is no period "." in front of this class name
+				})
+			});
+		 });
+	
+		function finishSession(){
+			document.getElementById("finishButton").disabled = true;
+			document.location.href ='<c:url value="/learning/finish.do?sessionMapID=${sessionMapID}"/>';
+			return false;
+		}
+		
+		function continueReflect(){
+			document.location.href='<c:url value="/learning/newReflection.do?sessionMapID=${sessionMapID}"/>';
+		}
+		
+		function retakeSurvey(){
+			document.location.href='<c:url value="/learning/retake.do?sessionMapID=${sessionMapID}"/>';
+		}
+    </script>
+</lams:head>
+<body class="stripes">
+
+	<div id="content">
+		<h1>
+			<c:out value="${sessionMap.title}" escapeXml="true"/>
+		</h1>
+		<p>
+			<c:out value="${sessionMap.instructions}" escapeXml="false" />
+		</p>
+
+		<c:if test="${sessionMap.lockOnFinish and sessionMap.mode != 'teacher'}">
+			<div class="info">
+				<fmt:message key="message.activityLocked" />
+			</div>
+		</c:if>
+		
+		<%-- user personal results--%>
+		<c:forEach var="element" items="${sessionMap.questionList}">
+			<div class="shading-bg">
+				<c:set var="question" value="${element.value}" />
+
+				<c:if test="${not question.optional}">
+					<span style="color: #FF0000">*</span>
+				</c:if>
+
+				<c:out value="${question.description}" escapeXml="false" />
+
+				<p>
+					<c:set var="answerText" value="" />
+					<c:if test="${not empty question.answer}">
+						<c:set var="answerText" value="${question.answer.answerText}" />
+					</c:if>
+					<c:forEach var="option" items="${question.options}">
+						<div>
+							<c:set var="checked" value="false" />
+
+							<c:if test="${not empty question.answer}">
+								<c:forEach var="choice" items="${question.answer.choices}">
+									<c:if test="${choice == option.uid}">
+										<c:set var="checked" value="true" />
+									</c:if>
+								</c:forEach>
+							</c:if>
+
+							<c:if test="${checked}">
+								<c:out value="${option.description}" escapeXml="true"/>
+							</c:if>
+						</div>
+					</c:forEach>
+
+				</p>
+				<c:if test="${question.type == 3}">
+					<lams:out value="${answerText}" escapeHtml="false"/>
+				</c:if>
+
+				<c:if test="${question.appendText && (not empty answerText)}">
+					<fmt:message key="label.append.text" />
+					<lams:out value="${answerText}" escapeHtml="true" />
+				</c:if>
+			</div>
+		</c:forEach>
+
+		<c:if test="${not sessionMap.lockOnFinish}">
+			<html:button property="RetakeButton" onclick="return retakeSurvey()" styleClass="button">
+				<fmt:message key="label.retake.survey" />
+			</html:button>
+		</c:if>
+
+		<c:if test="${sessionMap.userFinished and sessionMap.reflectOn}">
+			<div class="small-space-top">
+				<h2><fmt:message key="title.reflection" /></h2>
+				<strong>
+					<lams:out value="${sessionMap.reflectInstructions}" escapeHtml="true"/>
+				</strong>
+
+				<c:choose>
+					<c:when test="${empty sessionMap.reflectEntry}">
+						<p>
+							<em> <fmt:message key="message.no.reflection.available" /></em>
+						</p>
+					</c:when>
+					<c:otherwise>
+						<p>
+							<lams:out escapeHtml="true" value="${sessionMap.reflectEntry}" />
+						</p>
+					</c:otherwise>
+				</c:choose>
+
+				<html:button property="ContinueButton" onclick="return continueReflect()" styleClass="button">
+					<fmt:message key="label.edit" />
+				</html:button>
+			</div>
+		</c:if>
+		
+		<%-- other users personal results--%>
+		<c:if test="${sessionMap.showOtherUsersAnswers}">
+			<h2 id="other-users-answers-title">
+				<fmt:message key="label.other.answers" />
+			</h2>
+			
+			<div>
+				<fmt:message key="label.total.responses" >
+					<fmt:param>${countFinishedUser}</fmt:param>
+				</fmt:message>
+			</div>
+		
+			<c:forEach var="question" items="${answerDtos}" varStatus="queStatus">
+			<table class="alternative-color">
+				<tr>
+					<th class="first" colspan="2">
+						<c:out value="${question.shortTitle}"/> 
+						
+						<%-- Only show pie/bar chart when question is single/multiple choics type --%>
+						<c:if test="${question.type != 3}">
+							<div style="float:right">
+								<img src='<c:out value="${tool}"/>includes/images/piechart.gif'
+									title="<fmt:message key='message.view.pie.chart'/>"
+									style="cursor: pointer; width: 30px; border: none"
+									onclick="javascript:drawChart('pie', ${queStatus.index}, {'toolSessionID' : '${sessionMap.toolSessionID}','questionUid' : '${question.uid}'})">
+							</div>
+						</c:if>
+					</th>
+				</tr>
+				
+				<c:set var="optSize" value="${fn:length(question.options)}" />
+				<c:forEach var="option" items="${question.options}"  varStatus="optStatus">
+					<tr>
+						<td><c:out value="${option.description}" escapeXml="true"/></td>
+						<td>
+							<c:set var="imgTitle">
+								<fmt:message key="message.learner.choose.answer.percentage">
+									<fmt:param>${option.response}</fmt:param>
+								</fmt:message>
+							</c:set>
+							<c:set var="imgIdx">
+								${optStatus.index % 5 + 1}
+							</c:set>			
+							<img src="${tool}/includes/images/bar${imgIdx}.gif" height="10" width="${option.response * 2}" title="${imgTitle}">
+							${option.responseCount} (${option.responseFormatStr}%)
+						</td>
+					</tr>
+				</c:forEach>
+				<tr>
+					<td id="chartDiv${queStatus.index}" style="height: 220px; display: none; padding: 0;" colspan="2">
+					</td>
+				</tr>
+				<c:if test="${question.appendText}">
+					<tr>
+						<td><fmt:message key="label.open.response"/></td>
+						<td>
+							<c:set var="imgTitle">
+								<fmt:message key="message.learner.choose.answer.percentage">
+									<fmt:param>${question.openResponseFormatStr}</fmt:param>
+								</fmt:message>
+							</c:set>
+							<c:set var="imgIdx">
+								${(optSize % 5)  + 1}
+							</c:set>						
+							<img src="${tool}/includes/images/bar${imgIdx}.gif" height="10" width="${question.openResponse * 2}" title="${imgTitle}">
+							${question.openResponseCount} (${question.openResponseFormatStr}%)
+						</td>
+					</tr>
+				</c:if>
+				<c:if test="${question.type == 3}">
+					<tr>
+						<td colspan="2">
+							<table class="tablesorter" data-question-uid="${question.uid}">
+								<thead>
+									<tr>
+										<th title="<fmt:message key='label.sort.by.answer'/>" style="background-color: #D2E1DF; color: black; height: 15px;">
+											<fmt:message key="label.answer" />
+										</th>
+									</tr>
+								</thead>
+								<tbody>
+			
+								</tbody>
+							</table>
+								
+							<!-- pager -->
+							<div class="pager">
+								<form>
+								   	<img class="tablesorter-first"/>
+							    	<img class="tablesorter-prev"/>
+							    	<span class="pagedisplay"></span> 
+							   		<img class="tablesorter-next"/>
+							    	<img class="tablesorter-last"/>
+							    	<select class="pagesize">
+							      		<option selected="selected" value="10">10</option>
+							      		<option value="20">20</option>
+							      		<option value="30">30</option>
+							      		<option value="40">40</option>
+							      		<option value="50">50</option>
+							      		<option value="100">100</option>
+							    	</select>
+							  </form>
+							</div>	
+						</td>
+					</tr>
+				</c:if>
+				
+				</table>
+	
+			</c:forEach>
+		</c:if>
+
+		<c:if test="${sessionMap.mode != 'teacher'}">
+			<table>
+				<tr>
+					<td>
+						<span class="right-buttons">
+							<c:choose>
+							
+								<c:when test="${sessionMap.reflectOn}">
+									<html:button property="ContinueButton" onclick="return continueReflect()" styleClass="button">
+										<fmt:message key="label.continue" />
+									</html:button>
+								</c:when>
+								
+								<c:otherwise>
+									<html:link href="#nogo" property="FinishButton" styleId="finishButton"
+										onclick="return finishSession()" styleClass="button">
+										<span class="nextActivity">
+											<c:choose>
+							 					<c:when test="${sessionMap.activityPosition.last}">
+							 						<fmt:message key="label.submit" />
+							 					</c:when>
+							 					
+							 					<c:otherwise>
+							 		 				<fmt:message key="label.finished" />
+							 					</c:otherwise>
+							 				</c:choose>
+							 			</span>
+									</html:link>
+								</c:otherwise>
+							</c:choose> 
+						</span>
+					</td>
+				</tr>
+			</table>
+		</c:if>
+	</div>
+	<!--closes content-->
+
+	<div id="footer"></div>
+	<!--closes footer-->
+
+</body>
+</lams:html>
+
+
Index: lams_tool_survey/web/pages/learning/submissionDeadline.jsp
===================================================================
diff -u -r77bcef245dda8b166cdf40c12c62a7a5ab8d6fde -r098d686653002409f33aff0f69b1183e07c1eab3
--- lams_tool_survey/web/pages/learning/submissionDeadline.jsp	(.../submissionDeadline.jsp)	(revision 77bcef245dda8b166cdf40c12c62a7a5ab8d6fde)
+++ lams_tool_survey/web/pages/learning/submissionDeadline.jsp	(.../submissionDeadline.jsp)	(revision 098d686653002409f33aff0f69b1183e07c1eab3)
@@ -8,17 +8,14 @@
 	<%@ include file="/common/header.jsp"%>
 
 	<script type="text/javascript">
-	<!--
 		function finishSession(){
 			document.getElementById("finishButton").disabled = true;
 			document.location.href ='<c:url value="/learning/finish.do?sessionMapID=${sessionMapID}"/>';
 			return false;
 		}
 		function continueReflect(){
 			document.location.href='<c:url value="/learning/newReflection.do?sessionMapID=${sessionMapID}"/>';
-		}
-		
-	-->        
+		} 
     </script>
 </lams:head>
 
Index: lams_tool_survey/web/pages/monitoring/advanceoptions.jsp
===================================================================
diff -u
--- lams_tool_survey/web/pages/monitoring/advanceoptions.jsp	(revision 0)
+++ lams_tool_survey/web/pages/monitoring/advanceoptions.jsp	(revision 098d686653002409f33aff0f69b1183e07c1eab3)
@@ -0,0 +1,111 @@
+<h1 style="padding-bottom: 10px;">
+	<img src="<lams:LAMSURL/>/images/tree_closed.gif" id="treeIcon" onclick="javascript:toggleAdvancedOptionsVisibility(document.getElementById('advancedDiv'), document.getElementById('treeIcon'), '<lams:LAMSURL/>');" />
+
+	<a href="javascript:toggleAdvancedOptionsVisibility(document.getElementById('advancedDiv'), document.getElementById('treeIcon'),'<lams:LAMSURL/>');" >
+		<fmt:message key="monitor.summary.th.advancedSettings" />
+	</a>
+</h1>
+<br />
+
+<div class="monitoring-advanced" id="advancedDiv" style="display:none">
+	<table class="alternative-color">
+		<tr>
+			<td>
+				<fmt:message key="label.authoring.advance.lock.on.finished" />
+			</td>
+			
+			<td>
+				<c:choose>
+					<c:when test="${sessionMap.survey.lockWhenFinished}">
+						<fmt:message key="label.on" />
+					</c:when>
+					<c:otherwise>
+						<fmt:message key="label.off" />
+					</c:otherwise>
+				</c:choose>	
+			</td>
+		</tr>
+		
+		<tr>
+			<td>
+				<fmt:message key="label.authoring.advance.show.on.one.page" />
+			</td>
+			
+			<td>
+				<c:choose>
+					<c:when test="${sessionMap.survey.showOnePage}">
+						<fmt:message key="label.on" />
+					</c:when>
+					<c:otherwise>
+						<fmt:message key="label.off" />
+					</c:otherwise>
+				</c:choose>	
+			</td>
+		</tr>
+		
+		<tr>
+			<td>
+				<fmt:message key="label.show.answers.from.other.users" />
+			</td>
+			
+			<td>
+				<c:choose>
+					<c:when test="${sessionMap.survey.showOtherUsersAnswers}">
+						<fmt:message key="label.on" />
+					</c:when>
+					<c:otherwise>
+						<fmt:message key="label.off" />
+					</c:otherwise>
+				</c:choose>	
+			</td>
+		</tr>
+		
+		<tr>
+			<td>
+				<fmt:message key="label.authoring.advanced.notify.onanswersubmit" />
+			</td>
+			
+			<td>
+				<c:choose>
+					<c:when test="${sessionMap.survey.notifyTeachersOnAnswerSumbit}">
+						<fmt:message key="label.on" />
+					</c:when>
+					<c:otherwise>
+						<fmt:message key="label.off" />
+					</c:otherwise>
+				</c:choose>	
+			</td>
+			
+		</tr>
+		
+		<tr>
+			<td>
+				<fmt:message key="monitor.summary.td.addNotebook" />
+			</td>
+			
+			<td>
+				<c:choose>
+					<c:when test="${sessionMap.survey.reflectOnActivity}">
+						<fmt:message key="label.on" />
+					</c:when>
+					<c:otherwise>
+						<fmt:message key="label.off" />
+					</c:otherwise>
+				</c:choose>	
+			</td>
+		</tr>
+		
+		<c:choose>
+			<c:when test="${sessionMap.survey.reflectOnActivity}">
+				<tr>
+					<td>
+						<fmt:message key="monitor.summary.td.notebookInstructions" />
+					</td>
+					<td>
+						<lams:out value="${sessionMap.survey.reflectInstructions}" escapeHtml="true"/>
+					</td>
+				</tr>
+			</c:when>
+		</c:choose>
+	</table>
+</div>
\ No newline at end of file
Index: lams_tool_survey/web/pages/monitoring/listanswers.jsp
===================================================================
diff -u -r40c565b9307a4b3fcc2f724b1baf104c6c8a2dcf -r098d686653002409f33aff0f69b1183e07c1eab3
--- lams_tool_survey/web/pages/monitoring/listanswers.jsp	(.../listanswers.jsp)	(revision 40c565b9307a4b3fcc2f724b1baf104c6c8a2dcf)
+++ lams_tool_survey/web/pages/monitoring/listanswers.jsp	(.../listanswers.jsp)	(revision 098d686653002409f33aff0f69b1183e07c1eab3)
@@ -1,7 +1,6 @@
 <!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="tool"><lams:WebAppURL/></c:set>
 <%@ page import="org.lamsfoundation.lams.tool.survey.SurveyConstants"%>
Index: lams_tool_survey/web/pages/monitoring/notebook.jsp
===================================================================
diff -u -r40c565b9307a4b3fcc2f724b1baf104c6c8a2dcf -r098d686653002409f33aff0f69b1183e07c1eab3
--- lams_tool_survey/web/pages/monitoring/notebook.jsp	(.../notebook.jsp)	(revision 40c565b9307a4b3fcc2f724b1baf104c6c8a2dcf)
+++ lams_tool_survey/web/pages/monitoring/notebook.jsp	(.../notebook.jsp)	(revision 098d686653002409f33aff0f69b1183e07c1eab3)
@@ -1,6 +1,6 @@
-<%@ 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"%>
 
 <lams:html>
 <lams:head>
Index: lams_tool_survey/web/pages/monitoring/summary.jsp
===================================================================
diff -u -r77bcef245dda8b166cdf40c12c62a7a5ab8d6fde -r098d686653002409f33aff0f69b1183e07c1eab3
--- lams_tool_survey/web/pages/monitoring/summary.jsp	(.../summary.jsp)	(revision 77bcef245dda8b166cdf40c12c62a7a5ab8d6fde)
+++ lams_tool_survey/web/pages/monitoring/summary.jsp	(.../summary.jsp)	(revision 098d686653002409f33aff0f69b1183e07c1eab3)
@@ -214,98 +214,7 @@
 	
 <br />	
 	
-<h1 style="padding-bottom: 10px;">
-	<img src="<lams:LAMSURL/>/images/tree_closed.gif" id="treeIcon" onclick="javascript:toggleAdvancedOptionsVisibility(document.getElementById('advancedDiv'), document.getElementById('treeIcon'), '<lams:LAMSURL/>');" />
+<%@include file="advanceoptions.jsp"%>
 
-	<a href="javascript:toggleAdvancedOptionsVisibility(document.getElementById('advancedDiv'), document.getElementById('treeIcon'),'<lams:LAMSURL/>');" >
-		<fmt:message key="monitor.summary.th.advancedSettings" />
-	</a>
-</h1>
-<br />
-
-<div class="monitoring-advanced" id="advancedDiv" style="display:none">
-<table class="alternative-color">
-	<tr>
-		<td>
-			<fmt:message key="label.authoring.advance.lock.on.finished" />
-		</td>
-		
-		<td>
-			<c:choose>
-				<c:when test="${sessionMap.survey.lockWhenFinished}">
-					<fmt:message key="label.on" />
-				</c:when>
-				<c:otherwise>
-					<fmt:message key="label.off" />
-				</c:otherwise>
-			</c:choose>	
-		</td>
-	</tr>
-	
-	<tr>
-		<td>
-			<fmt:message key="label.authoring.advance.show.on.one.page" />
-		</td>
-		
-		<td>
-			<c:choose>
-				<c:when test="${sessionMap.survey.showOnePage}">
-					<fmt:message key="label.on" />
-				</c:when>
-				<c:otherwise>
-					<fmt:message key="label.off" />
-				</c:otherwise>
-			</c:choose>	
-		</td>
-	</tr>
-	<tr>
-		<td>
-			<fmt:message key="label.authoring.advanced.notify.onanswersubmit" />
-		</td>
-		
-		<td>
-			<c:choose>
-				<c:when test="${sessionMap.survey.notifyTeachersOnAnswerSumbit}">
-					<fmt:message key="label.on" />
-				</c:when>
-				<c:otherwise>
-					<fmt:message key="label.off" />
-				</c:otherwise>
-			</c:choose>	
-		</td>
-		
-	</tr>
-	<tr>
-		<td>
-			<fmt:message key="monitor.summary.td.addNotebook" />
-		</td>
-		
-		<td>
-			<c:choose>
-				<c:when test="${sessionMap.survey.reflectOnActivity}">
-					<fmt:message key="label.on" />
-				</c:when>
-				<c:otherwise>
-					<fmt:message key="label.off" />
-				</c:otherwise>
-			</c:choose>	
-		</td>
-	</tr>
-	
-	<c:choose>
-		<c:when test="${sessionMap.survey.reflectOnActivity}">
-			<tr>
-				<td>
-					<fmt:message key="monitor.summary.td.notebookInstructions" />
-				</td>
-				<td>
-					<lams:out value="${sessionMap.survey.reflectInstructions}" escapeHtml="true"/>
-				</td>
-			</tr>
-		</c:when>
-	</c:choose>
-</table>
-</div>
-
 <%@include file="daterestriction.jsp"%>
 	
\ No newline at end of file