Index: lams_tool_zoom/src/java/org/lamsfoundation/lams/tool/zoom/dto/ContentDTO.java =================================================================== diff -u -r2cf1e086b82a29fae22136351fc7a44821181323 -rfba5bdf4505b4dbe6d95809740ac8f3c14b07ef1 --- lams_tool_zoom/src/java/org/lamsfoundation/lams/tool/zoom/dto/ContentDTO.java (.../ContentDTO.java) (revision 2cf1e086b82a29fae22136351fc7a44821181323) +++ lams_tool_zoom/src/java/org/lamsfoundation/lams/tool/zoom/dto/ContentDTO.java (.../ContentDTO.java) (revision fba5bdf4505b4dbe6d95809740ac8f3c14b07ef1) @@ -23,7 +23,11 @@ package org.lamsfoundation.lams.tool.zoom.dto; +import java.util.Set; +import java.util.TreeSet; + import org.lamsfoundation.lams.tool.zoom.model.Zoom; +import org.lamsfoundation.lams.tool.zoom.model.ZoomSession; public class ContentDTO { @@ -39,8 +43,12 @@ Long currentTab; + boolean contentInUse; + boolean isGroupedActivity; // set manually in MonitoringAction + Set sessionDTOs = new TreeSet(); + /* Constructors */ public ContentDTO() { } @@ -49,8 +57,14 @@ this.toolContentId = zoom.getToolContentId(); this.title = zoom.getTitle(); this.instructions = zoom.getInstructions(); + this.contentInUse = zoom.isContentInUse(); this.reflectInstructions = zoom.getReflectInstructions(); this.reflectOnActivity = zoom.isReflectOnActivity(); + for (ZoomSession session : zoom.getZoomSessions()) { + sessionDTOs.add(new SessionDTO(session)); + } + + isGroupedActivity = false; } public String getInstructions() { @@ -93,6 +107,14 @@ this.toolContentId = toolContentID; } + public Boolean getContentInUse() { + return contentInUse; + } + + public void setContentInUse(Boolean contentInUse) { + this.contentInUse = contentInUse; + } + public Long getCurrentTab() { return currentTab; } @@ -108,4 +130,12 @@ public void setGroupedActivity(boolean isGroupedActivity) { this.isGroupedActivity = isGroupedActivity; } + + public Set getSessionDTOs() { + return sessionDTOs; + } + + public void setSessionDTOs(Set sessionDTOs) { + this.sessionDTOs = sessionDTOs; + } } \ No newline at end of file Index: lams_tool_zoom/src/java/org/lamsfoundation/lams/tool/zoom/dto/SessionDTO.java =================================================================== diff -u -r2cf1e086b82a29fae22136351fc7a44821181323 -rfba5bdf4505b4dbe6d95809740ac8f3c14b07ef1 --- lams_tool_zoom/src/java/org/lamsfoundation/lams/tool/zoom/dto/SessionDTO.java (.../SessionDTO.java) (revision 2cf1e086b82a29fae22136351fc7a44821181323) +++ lams_tool_zoom/src/java/org/lamsfoundation/lams/tool/zoom/dto/SessionDTO.java (.../SessionDTO.java) (revision fba5bdf4505b4dbe6d95809740ac8f3c14b07ef1) @@ -27,18 +27,30 @@ import java.util.TreeSet; import org.lamsfoundation.lams.tool.zoom.model.ZoomSession; +import org.lamsfoundation.lams.tool.zoom.model.ZoomUser; +import org.lamsfoundation.lams.tool.zoom.service.IZoomService; public class SessionDTO implements Comparable { + private static IZoomService zoomService; Long sessionID; String sessionName; + int numberOfLearners; + Set userDTOs = new TreeSet(); public SessionDTO(ZoomSession session) { this.sessionID = session.getSessionId(); this.sessionName = session.getSessionName(); + + for (ZoomUser zoomUser : session.getZoomUsers()) { + ZoomUserDTO userDTO = zoomService.createUserDTO(zoomUser); + userDTOs.add(userDTO); + } + + numberOfLearners = userDTOs.size(); } @Override @@ -76,4 +88,16 @@ public void setUserDTOs(Set zoomUsers) { this.userDTOs = zoomUsers; } + + public int getNumberOfLearners() { + return numberOfLearners; + } + + public void setNumberOfLearners(int numberOfLearners) { + this.numberOfLearners = numberOfLearners; + } + + public static void setZoomService(IZoomService zoomService) { + SessionDTO.zoomService = zoomService; + } } \ No newline at end of file Index: lams_tool_zoom/src/java/org/lamsfoundation/lams/tool/zoom/service/ZoomService.java =================================================================== diff -u -reeaae70bac24a36a5f05a5d56638df29c1a495c5 -rfba5bdf4505b4dbe6d95809740ac8f3c14b07ef1 --- lams_tool_zoom/src/java/org/lamsfoundation/lams/tool/zoom/service/ZoomService.java (.../ZoomService.java) (revision eeaae70bac24a36a5f05a5d56638df29c1a495c5) +++ lams_tool_zoom/src/java/org/lamsfoundation/lams/tool/zoom/service/ZoomService.java (.../ZoomService.java) (revision fba5bdf4505b4dbe6d95809740ac8f3c14b07ef1) @@ -706,7 +706,7 @@ JSONObject bodyJSON = new JSONObject(); String lastName = userDTO.getLastName(); - if (!sessionName.endsWith(" learners")) { + if (isGroupedActivity(zoom.getToolContentId())) { lastName += " (" + sessionName + ")"; } bodyJSON.put("email", userDTO.getEmail()).put("first_name", userDTO.getFirstName()).put("last_name", lastName); Index: lams_tool_zoom/src/java/org/lamsfoundation/lams/tool/zoom/web/actions/MonitoringAction.java =================================================================== diff -u -reeaae70bac24a36a5f05a5d56638df29c1a495c5 -rfba5bdf4505b4dbe6d95809740ac8f3c14b07ef1 --- lams_tool_zoom/src/java/org/lamsfoundation/lams/tool/zoom/web/actions/MonitoringAction.java (.../MonitoringAction.java) (revision eeaae70bac24a36a5f05a5d56638df29c1a495c5) +++ lams_tool_zoom/src/java/org/lamsfoundation/lams/tool/zoom/web/actions/MonitoringAction.java (.../MonitoringAction.java) (revision fba5bdf4505b4dbe6d95809740ac8f3c14b07ef1) @@ -34,16 +34,14 @@ import org.lamsfoundation.lams.notebook.model.NotebookEntry; import org.lamsfoundation.lams.tool.zoom.dto.ContentDTO; import org.lamsfoundation.lams.tool.zoom.dto.NotebookEntryDTO;//import org.lamsfoundation.lams.tool.zoom.dto.UserDTO; +import org.lamsfoundation.lams.tool.zoom.dto.SessionDTO; import org.lamsfoundation.lams.tool.zoom.dto.ZoomUserDTO; import org.lamsfoundation.lams.tool.zoom.model.Zoom; -import org.lamsfoundation.lams.tool.zoom.model.ZoomSession; import org.lamsfoundation.lams.tool.zoom.model.ZoomUser; import org.lamsfoundation.lams.tool.zoom.service.IZoomService; import org.lamsfoundation.lams.tool.zoom.service.ZoomServiceProxy; import org.lamsfoundation.lams.tool.zoom.util.ZoomConstants; -import org.lamsfoundation.lams.tool.zoom.web.forms.MonitoringForm; import org.lamsfoundation.lams.util.WebUtil; -import org.lamsfoundation.lams.web.session.SessionManager; import org.lamsfoundation.lams.web.util.AttributeNames; public class MonitoringAction extends DispatchAction { @@ -76,12 +74,14 @@ logger.error("Unable to find tool content with id :" + toolContentID); } - ContentDTO contentDT0 = new ContentDTO(zoom); + SessionDTO.setZoomService(zoomService); + ContentDTO contentDTO = new ContentDTO(zoom); Long currentTab = WebUtil.readLongParam(request, AttributeNames.PARAM_CURRENT_TAB, true); - contentDT0.setCurrentTab(currentTab); + contentDTO.setCurrentTab(currentTab); + contentDTO.setGroupedActivity(zoomService.isGroupedActivity(toolContentID)); - request.setAttribute(ZoomConstants.ATTR_CONTENT_DTO, contentDT0); + request.setAttribute(ZoomConstants.ATTR_CONTENT_DTO, contentDTO); request.setAttribute(ZoomConstants.ATTR_CONTENT_FOLDER_ID, contentFolderID); return mapping.findForward("success"); Index: lams_tool_zoom/web/pages/monitoring/statistics.jsp =================================================================== diff -u -r2cf1e086b82a29fae22136351fc7a44821181323 -rfba5bdf4505b4dbe6d95809740ac8f3c14b07ef1 --- lams_tool_zoom/web/pages/monitoring/statistics.jsp (.../statistics.jsp) (revision 2cf1e086b82a29fae22136351fc7a44821181323) +++ lams_tool_zoom/web/pages/monitoring/statistics.jsp (.../statistics.jsp) (revision fba5bdf4505b4dbe6d95809740ac8f3c14b07ef1) @@ -22,14 +22,6 @@ ${session.numberOfLearners} - - - heading.totalFinishedLearnersInGroup - - - ${session.numberOfFinishedLearners} - - Index: lams_tool_zoom/web/pages/monitoring/summary.jsp =================================================================== diff -u -r2cf1e086b82a29fae22136351fc7a44821181323 -rfba5bdf4505b4dbe6d95809740ac8f3c14b07ef1 --- lams_tool_zoom/web/pages/monitoring/summary.jsp (.../summary.jsp) (revision 2cf1e086b82a29fae22136351fc7a44821181323) +++ lams_tool_zoom/web/pages/monitoring/summary.jsp (.../summary.jsp) (revision fba5bdf4505b4dbe6d95809740ac8f3c14b07ef1) @@ -18,100 +18,57 @@ - - -
- + + + + + - +
 
+ + + + + + - - - - - - -
- -
- - - -

 ${session.numberOfLearners}

- -
 
-
- - - -
+ + + +
- + + - - + + - - - - - - - - -
- - - - + + + + + + + + + + + + + +
- - - - - - - - - - - - - -
-
- - - - -
- -
- ${ !usePanel || ! status.last ? '
 
' : ''} - - - - - + + + + +
 
+
-
- - - + \ No newline at end of file