Index: lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/dto/ReflectDTO.java =================================================================== diff -u -r65e3365a66b2977b268fafdd0b9fe2e5eae732a4 -r405cf90d2646e8f7dc1aa032b28c820279e50930 --- lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/dto/ReflectDTO.java (.../ReflectDTO.java) (revision 65e3365a66b2977b268fafdd0b9fe2e5eae732a4) +++ lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/dto/ReflectDTO.java (.../ReflectDTO.java) (revision 405cf90d2646e8f7dc1aa032b28c820279e50930) @@ -23,31 +23,23 @@ /* $Id$ */ package org.lamsfoundation.lams.tool.assessment.dto; +import java.util.Date; + import org.lamsfoundation.lams.tool.assessment.model.AssessmentUser; public class ReflectDTO { private Long userUid; private String fullName; private String loginName; - private boolean hasRefection; - private String reflectInstrctions; - private boolean finishReflection; private String reflect; + private Date date; public ReflectDTO(AssessmentUser user) { this.setLoginName(user.getLoginName()); this.setFullName(user.getFirstName() + " " + user.getLastName()); this.setUserUid(user.getUid()); } - public boolean isFinishReflection() { - return finishReflection; - } - - public void setFinishReflection(boolean finishReflection) { - this.finishReflection = finishReflection; - } - public String getFullName() { return fullName; } @@ -56,14 +48,6 @@ this.fullName = fullName; } - public boolean isHasRefection() { - return hasRefection; - } - - public void setHasRefection(boolean hasRefection) { - this.hasRefection = hasRefection; - } - public String getLoginName() { return loginName; } @@ -80,19 +64,19 @@ this.reflect = reflect; } - public String getReflectInstrctions() { - return reflectInstrctions; - } - - public void setReflectInstrctions(String reflectInstrctions) { - this.reflectInstrctions = reflectInstrctions; - } - public Long getUserUid() { return userUid; } public void setUserUid(Long userUid) { this.userUid = userUid; } + + public Date getDate() { + return date; + } + + public void setDate(Date date) { + this.date = date; + } } Index: lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/service/AssessmentServiceImpl.java =================================================================== diff -u -rbb597b8155375e6ac4dfe280f630d323b6e5e575 -r405cf90d2646e8f7dc1aa032b28c820279e50930 --- lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/service/AssessmentServiceImpl.java (.../AssessmentServiceImpl.java) (revision bb597b8155375e6ac4dfe280f630d323b6e5e575) +++ lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/service/AssessmentServiceImpl.java (.../AssessmentServiceImpl.java) (revision 405cf90d2646e8f7dc1aa032b28c820279e50930) @@ -604,31 +604,30 @@ } @Override - public Map> getReflectList(Long contentId) { - Map> map = new HashMap>(); + public List getReflectList(Long contentId) { + List reflectList = new LinkedList(); List sessionList = assessmentSessionDao.getByContentId(contentId); for (AssessmentSession session : sessionList) { Long sessionId = session.getSessionId(); - boolean hasRefection = session.getAssessment().isReflectOnActivity(); - Set list = new TreeSet(new ReflectDTOComparator()); // get all users in this session List users = assessmentUserDao.getBySessionID(sessionId); for (AssessmentUser user : users) { - ReflectDTO ref = new ReflectDTO(user); NotebookEntry entry = getEntry(sessionId, user.getUserId().intValue()); if (entry != null) { + ReflectDTO ref = new ReflectDTO(user); ref.setReflect(entry.getEntry()); + Date postedDate = (entry.getLastModified() != null) ? entry.getLastModified() : entry + .getCreateDate(); + ref.setDate(postedDate); + reflectList.add(ref); } - - ref.setHasRefection(hasRefection); - list.add(ref); + } - map.put(sessionId, list); } - return map; + return reflectList; } Index: lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/service/IAssessmentService.java =================================================================== diff -u -rbb597b8155375e6ac4dfe280f630d323b6e5e575 -r405cf90d2646e8f7dc1aa032b28c820279e50930 --- lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/service/IAssessmentService.java (.../IAssessmentService.java) (revision bb597b8155375e6ac4dfe280f630d323b6e5e575) +++ lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/service/IAssessmentService.java (.../IAssessmentService.java) (revision 405cf90d2646e8f7dc1aa032b28c820279e50930) @@ -235,7 +235,7 @@ void updateEntry(NotebookEntry notebookEntry); - Map> getReflectList(Long contentId); + List getReflectList(Long contentId); /** * If success return next activity's url, otherwise return null. Index: lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/web/action/MonitoringAction.java =================================================================== diff -u -ref851974c4dc3f5468b188fab9997a37b73c0e50 -r405cf90d2646e8f7dc1aa032b28c820279e50930 --- lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/web/action/MonitoringAction.java (.../MonitoringAction.java) (revision ef851974c4dc3f5468b188fab9997a37b73c0e50) +++ lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/web/action/MonitoringAction.java (.../MonitoringAction.java) (revision 405cf90d2646e8f7dc1aa032b28c820279e50930) @@ -25,7 +25,6 @@ package org.lamsfoundation.lams.tool.assessment.web.action; import java.io.IOException; -import java.io.PrintWriter; import java.util.ArrayList; import java.util.Date; import java.util.LinkedHashMap; @@ -46,9 +45,9 @@ import org.apache.struts.action.ActionForm; import org.apache.struts.action.ActionForward; import org.apache.struts.action.ActionMapping; -import org.apache.struts.action.ActionMessage; import org.lamsfoundation.lams.tool.assessment.AssessmentConstants; import org.lamsfoundation.lams.tool.assessment.dto.QuestionSummary; +import org.lamsfoundation.lams.tool.assessment.dto.ReflectDTO; import org.lamsfoundation.lams.tool.assessment.dto.Summary; import org.lamsfoundation.lams.tool.assessment.dto.UserSummary; import org.lamsfoundation.lams.tool.assessment.model.Assessment; @@ -133,6 +132,13 @@ Date tzSubmissionDeadline = DateUtil.convertToTimeZoneFromDefault(teacherTimeZone, submissionDeadline); request.setAttribute(AssessmentConstants.ATTR_SUBMISSION_DEADLINE, tzSubmissionDeadline.getTime()); } + + // Create reflectList if reflection is enabled. + if (assessment.isReflectOnActivity()) { + List reflectList = service.getReflectList(assessment.getContentId()); + // Add reflectList to sessionMap + sessionMap.put(AssessmentConstants.ATTR_REFLECT_LIST, reflectList); + } // cache into sessionMap boolean isGroupedActivity = service.isGroupedActivity(contentId); Index: lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/web/servlet/ExportServlet.java =================================================================== diff -u -ref851974c4dc3f5468b188fab9997a37b73c0e50 -r405cf90d2646e8f7dc1aa032b28c820279e50930 --- lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/web/servlet/ExportServlet.java (.../ExportServlet.java) (revision ef851974c4dc3f5468b188fab9997a37b73c0e50) +++ lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/web/servlet/ExportServlet.java (.../ExportServlet.java) (revision 405cf90d2646e8f7dc1aa032b28c820279e50930) @@ -197,7 +197,7 @@ // Create reflectList if reflection is enabled. if (content.isReflectOnActivity()) { - Map> reflectList = service.getReflectList(content.getContentId()); + List reflectList = service.getReflectList(content.getContentId()); // Add reflectList to sessionMap sessionMap.put(AssessmentConstants.ATTR_REFLECT_LIST, reflectList); } Index: lams_tool_assessment/web/pages/export/parts/exportteacher.jsp =================================================================== diff -u -r20946003bc95595d01d3d330f582e81e178af97d -r405cf90d2646e8f7dc1aa032b28c820279e50930 --- lams_tool_assessment/web/pages/export/parts/exportteacher.jsp (.../exportteacher.jsp) (revision 20946003bc95595d01d3d330f582e81e178af97d) +++ lams_tool_assessment/web/pages/export/parts/exportteacher.jsp (.../exportteacher.jsp) (revision 405cf90d2646e8f7dc1aa032b28c820279e50930) @@ -227,20 +227,16 @@ - -
- ${summary.sessionName} -
+ - - -

- -

-

- -

-
+

+ +

+ +

+ +

+
Index: lams_tool_assessment/web/pages/monitoring/parts/reflections.jsp =================================================================== diff -u --- lams_tool_assessment/web/pages/monitoring/parts/reflections.jsp (revision 0) +++ lams_tool_assessment/web/pages/monitoring/parts/reflections.jsp (revision 405cf90d2646e8f7dc1aa032b28c820279e50930) @@ -0,0 +1,23 @@ +<%@ include file="/common/taglibs.jsp"%> + +
+ + + + + + + + + + + + +
+ +
+ +
+ + +
Index: lams_tool_assessment/web/pages/monitoring/summary.jsp =================================================================== diff -u -r3d7bcdb3b84b2fda69b51b5b3c6d8aebd3fbe394 -r405cf90d2646e8f7dc1aa032b28c820279e50930 --- lams_tool_assessment/web/pages/monitoring/summary.jsp (.../summary.jsp) (revision 3d7bcdb3b84b2fda69b51b5b3c6d8aebd3fbe394) +++ lams_tool_assessment/web/pages/monitoring/summary.jsp (.../summary.jsp) (revision 405cf90d2646e8f7dc1aa032b28c820279e50930) @@ -268,6 +268,10 @@
+ + <%@ include file="parts/reflections.jsp"%> + + <%@ include file="parts/advanceoptions.jsp"%> <%@ include file="parts/dateRestriction.jsp"%>