Index: lams_tool_task/src/java/org/lamsfoundation/lams/tool/taskList/dto/Summary.java =================================================================== diff -u -rbe42ba5063335619d4e04832ea235f96b82f0e53 -r9c5284945798a4bfc0048ad1bbbd7f6c104fc179 --- lams_tool_task/src/java/org/lamsfoundation/lams/tool/taskList/dto/Summary.java (.../Summary.java) (revision be42ba5063335619d4e04832ea235f96b82f0e53) +++ lams_tool_task/src/java/org/lamsfoundation/lams/tool/taskList/dto/Summary.java (.../Summary.java) (revision 9c5284945798a4bfc0048ad1bbbd7f6c104fc179) @@ -43,6 +43,9 @@ */ public class Summary { + private Long sessionId; + private String sessionName; + private boolean isMonitorVerificationRequired; private List userNames; @@ -56,14 +59,33 @@ /** * Contruction method for monitoring summary function. */ - public Summary(List taskListItems, List userNames, boolean[][] completeMap, int[] visitNumbers, boolean isMonitorVerificationRequired){ + public Summary(Long sessionId, String sessionName, + List taskListItems, List userNames, + boolean[][] completeMap, int[] visitNumbers, + boolean isMonitorVerificationRequired) { + this.sessionId = sessionId; + this.sessionName = sessionName; this.userNames = userNames; this.taskListItems = taskListItems; this.completeMap = completeMap; this.visitNumbers = visitNumbers; this.isMonitorVerificationRequired = isMonitorVerificationRequired; } + public Long getSessionId() { + return sessionId; + } + public void setSessionId(Long sessionId) { + this.sessionId = sessionId; + } + + public String getSessionName() { + return sessionName; + } + public void setSessionName(String sessionName) { + this.sessionName = sessionName; + } + public List getUserNames() { return userNames; } Index: lams_tool_task/src/java/org/lamsfoundation/lams/tool/taskList/service/ITaskListService.java =================================================================== diff -u -rbe42ba5063335619d4e04832ea235f96b82f0e53 -r9c5284945798a4bfc0048ad1bbbd7f6c104fc179 --- lams_tool_task/src/java/org/lamsfoundation/lams/tool/taskList/service/ITaskListService.java (.../ITaskListService.java) (revision be42ba5063335619d4e04832ea235f96b82f0e53) +++ lams_tool_task/src/java/org/lamsfoundation/lams/tool/taskList/service/ITaskListService.java (.../ITaskListService.java) (revision 9c5284945798a4bfc0048ad1bbbd7f6c104fc179) @@ -334,12 +334,12 @@ public List exportForTeacher(Long contentUid); /** - * Return summary for the specified TaskList. Used in monitoring. + * Return summary list for the specified TaskList. Used in monitoring. * * @param contentId specified TaskList uid * @return */ - Summary getSummary(Long contentId); + List getSummary(Long contentId); /** * Return task summary for the specified TaskListItem. Used in monitoring. Index: lams_tool_task/src/java/org/lamsfoundation/lams/tool/taskList/service/TaskListServiceImpl.java =================================================================== diff -u -rbe42ba5063335619d4e04832ea235f96b82f0e53 -r9c5284945798a4bfc0048ad1bbbd7f6c104fc179 --- lams_tool_task/src/java/org/lamsfoundation/lams/tool/taskList/service/TaskListServiceImpl.java (.../TaskListServiceImpl.java) (revision be42ba5063335619d4e04832ea235f96b82f0e53) +++ lams_tool_task/src/java/org/lamsfoundation/lams/tool/taskList/service/TaskListServiceImpl.java (.../TaskListServiceImpl.java) (revision 9c5284945798a4bfc0048ad1bbbd7f6c104fc179) @@ -431,54 +431,57 @@ /** * {@inheritDoc} */ - public Summary getSummary(Long contentId) { + public List getSummary(Long contentId) { - TaskList taskList = taskListDao.getByContentId(contentId); - ArrayList itemList = new ArrayList(); - itemList.addAll(taskList.getTaskListItems()); + List summaryList = new ArrayList(); - ArrayList userList = new ArrayList(); - //retrieve all the sessions associated with this taskList List sessionList = taskListSessionDao.getByContentId(contentId); + + TaskList taskList = taskListDao.getByContentId(contentId); + ArrayList itemList = new ArrayList(); + itemList.addAll(taskList.getTaskListItems()); //create the list containing all taskListItems - //create the user list of all whom were started this task for(TaskListSession session:sessionList) { Set newItems = session.getTaskListItems(); for(TaskListItem item : newItems) { if (!itemList.contains(item)) itemList.add(item); } - - List newUsers = taskListUserDao.getBySessionID(session.getSessionId()); - for(TaskListUser user : newUsers) { - if (!userList.contains(user)) userList.add(user); - } } - //Fill up the copmletion table - boolean[][] complete = new boolean[userList.size()][itemList.size()]; - //Fill up the array of visitNumbers - int[] visitNumbers = new int[itemList.size()]; - for (int i = 0; i < userList.size(); i++) { - TaskListUser user = userList.get(i); + + //create the user list of all whom were started this task + for(TaskListSession session:sessionList) { - for (int j = 0; j < itemList.size(); j++) { - TaskListItem item = itemList.get(j); + List userList = taskListUserDao.getBySessionID(session.getSessionId()); + + //Fill up the copmletion table + boolean[][] complete = new boolean[userList.size()][itemList.size()]; + //Fill up the array of visitNumbers + int[] visitNumbers = new int[itemList.size()]; + for (int i = 0; i < userList.size(); i++) { + TaskListUser user = userList.get(i); - //retreiving TaskListItemVisitLog for current taskList and user - TaskListItemVisitLog visitLog = taskListItemVisitDao.getTaskListItemLog(item.getUid(), user.getUserId()); - if (visitLog !=null) { - complete[i][j] = visitLog.isComplete(); - if (visitLog.isComplete()) visitNumbers[j]++; - } else { - complete[i][j] = false; + for (int j = 0; j < itemList.size(); j++) { + TaskListItem item = itemList.get(j); + + //retreiving TaskListItemVisitLog for current taskList and user + TaskListItemVisitLog visitLog = taskListItemVisitDao.getTaskListItemLog(item.getUid(), user.getUserId()); + if (visitLog !=null) { + complete[i][j] = visitLog.isComplete(); + if (visitLog.isComplete()) visitNumbers[j]++; + } else { + complete[i][j] = false; + } } } + + Summary summary = new Summary(session.getSessionId(), session.getSessionName(), itemList, userList, complete, visitNumbers, taskList.isMonitorVerificationRequired()); + summaryList.add(summary); } - - Summary summary = new Summary(itemList, userList, complete, visitNumbers, taskList.isMonitorVerificationRequired()); - return summary; + + return summaryList; } /** @@ -640,7 +643,7 @@ taskSummary.setTaskSummaryItems(newTaskSummaryItems); //get rid of TaskListItemComments and TaskListItemAttachments belong to another users - if (taskSummary.getTaskListItem().isCommentsAllowed() && !taskSummary.getTaskListItem().getShowCommentsToAll()) { + if (taskSummary.getTaskListItem().isCommentsFilesAllowed() && !taskSummary.getTaskListItem().getShowCommentsToAll()) { TaskSummaryItem taskSummaryItem = taskSummary.getTaskSummaryItems().get(0); List newComments = new ArrayList(); Index: lams_tool_task/src/java/org/lamsfoundation/lams/tool/taskList/web/action/MonitoringAction.java =================================================================== diff -u -rbe42ba5063335619d4e04832ea235f96b82f0e53 -r9c5284945798a4bfc0048ad1bbbd7f6c104fc179 --- lams_tool_task/src/java/org/lamsfoundation/lams/tool/taskList/web/action/MonitoringAction.java (.../MonitoringAction.java) (revision be42ba5063335619d4e04832ea235f96b82f0e53) +++ lams_tool_task/src/java/org/lamsfoundation/lams/tool/taskList/web/action/MonitoringAction.java (.../MonitoringAction.java) (revision 9c5284945798a4bfc0048ad1bbbd7f6c104fc179) @@ -103,10 +103,10 @@ TaskList taskList = service.getTaskListByContentId(contentId); taskList.toDTO(); - Summary summary = service.getSummary(contentId); + List summaryList = service.getSummary(contentId); //cache into sessionMap - sessionMap.put(TaskListConstants.ATTR_SUMMARY, summary); + sessionMap.put(TaskListConstants.ATTR_TASK_SUMMARY_LIST, summaryList); sessionMap.put(TaskListConstants.PAGE_EDITABLE, taskList.isContentInUse()); sessionMap.put(TaskListConstants.ATTR_RESOURCE, taskList); sessionMap.put(TaskListConstants.ATTR_TOOL_CONTENT_ID, contentId); Index: lams_tool_task/web/pages/export/summaryTask.jsp =================================================================== diff -u -rd7b9065fa1aec9a60feeb4d57c6e347e7077b371 -r9c5284945798a4bfc0048ad1bbbd7f6c104fc179 --- lams_tool_task/web/pages/export/summaryTask.jsp (.../summaryTask.jsp) (revision d7b9065fa1aec9a60feeb4d57c6e347e7077b371) +++ lams_tool_task/web/pages/export/summaryTask.jsp (.../summaryTask.jsp) (revision 9c5284945798a4bfc0048ad1bbbd7f6c104fc179) @@ -11,15 +11,15 @@
- +
  • (: ${item.parentTaskName})
  • - +
  • @@ -40,7 +40,7 @@ - + @@ -71,7 +71,7 @@ - +
      Index: lams_tool_task/web/pages/monitoring/summary.jsp =================================================================== diff -u -rbe42ba5063335619d4e04832ea235f96b82f0e53 -r9c5284945798a4bfc0048ad1bbbd7f6c104fc179 --- lams_tool_task/web/pages/monitoring/summary.jsp (.../summary.jsp) (revision be42ba5063335619d4e04832ea235f96b82f0e53) +++ lams_tool_task/web/pages/monitoring/summary.jsp (.../summary.jsp) (revision 9c5284945798a4bfc0048ad1bbbd7f6c104fc179) @@ -1,6 +1,6 @@ <%@ include file="/common/taglibs.jsp"%> - + - - - - - - - - - - - - - - + +
      + +
      +
      - - - - - - - - - - - - - - - - -
      -

      -
      - - - - ${item.title} - - - -
      - ${user.loginName} - - - - - - - - - - - - - - - - - - [ for ${user.loginName}] - - - -
      + +

      ${summary.sessionName}

      +
      -<%-- Reflection list --%> - - +

      - + - - - - - - - - + + + + + + + + + + + - - + + + + + + + + + + -
      -

      -
      - + + - -
      + + ${item.title} + + + +
      + ${user.loginName} - - - - - - - + + + + + + + + + + + + + + + + + [ for ${user.loginName}] + + + +
      -
      + + <%-- Reflection list --%> + + + +

      + + + + + + + + + + + + + + + +
      + + + +
      + ${user.loginName} + + + + + + + +
      +
      +
      + +
      + + Index: lams_tool_task/web/pages/monitoring/summaryTask.jsp =================================================================== diff -u -ra370ee2f7fe049337ac7b342e0218bdde3f3ee68 -r9c5284945798a4bfc0048ad1bbbd7f6c104fc179 --- lams_tool_task/web/pages/monitoring/summaryTask.jsp (.../summaryTask.jsp) (revision a370ee2f7fe049337ac7b342e0218bdde3f3ee68) +++ lams_tool_task/web/pages/monitoring/summaryTask.jsp (.../summaryTask.jsp) (revision 9c5284945798a4bfc0048ad1bbbd7f6c104fc179) @@ -28,15 +28,15 @@
      - +
      • (: ${item.parentTaskName})
      • - +
      • @@ -57,7 +57,7 @@ - + @@ -88,7 +88,7 @@ - +