Index: lams_tool_images/src/java/org/lamsfoundation/lams/tool/imageGallery/dto/ReflectDTO.java =================================================================== diff -u -r5773f84ed608838de3521ecde87c52f3c72d478c -rf851d4f476f3856ffb55d5004602a57f0dbe02c0 --- lams_tool_images/src/java/org/lamsfoundation/lams/tool/imageGallery/dto/ReflectDTO.java (.../ReflectDTO.java) (revision 5773f84ed608838de3521ecde87c52f3c72d478c) +++ lams_tool_images/src/java/org/lamsfoundation/lams/tool/imageGallery/dto/ReflectDTO.java (.../ReflectDTO.java) (revision f851d4f476f3856ffb55d5004602a57f0dbe02c0) @@ -9,6 +9,7 @@ */ public class ReflectDTO { private Long userUid; + private Long userId; private String fullName; private String loginName; private boolean hasRefection; @@ -20,6 +21,7 @@ this.setLoginName(user.getLoginName()); this.setFullName(user.getFirstName() + " " + user.getLastName()); this.setUserUid(user.getUid()); + this.setUserId(user.getUserId()); } public boolean isFinishReflection() { @@ -77,4 +79,12 @@ public void setUserUid(Long userUid) { this.userUid = userUid; } + + public Long getUserId() { + return userId; + } + + public void setUserId(Long userId) { + this.userId = userId; + } } Index: lams_tool_images/src/java/org/lamsfoundation/lams/tool/imageGallery/dto/Summary.java =================================================================== diff -u -r3399163940c61c9132223c758d274486e57ff9b7 -rf851d4f476f3856ffb55d5004602a57f0dbe02c0 --- lams_tool_images/src/java/org/lamsfoundation/lams/tool/imageGallery/dto/Summary.java (.../Summary.java) (revision 3399163940c61c9132223c758d274486e57ff9b7) +++ lams_tool_images/src/java/org/lamsfoundation/lams/tool/imageGallery/dto/Summary.java (.../Summary.java) (revision f851d4f476f3856ffb55d5004602a57f0dbe02c0) @@ -54,6 +54,7 @@ private boolean itemHide; private String itemTitle; private String username; + private Long userId; // LAMS user id private int numberOfVotes; private ItemRatingDTO itemRatingDto; @@ -83,6 +84,7 @@ this.itemTitle = item.getTitle(); this.username = item.getCreateBy() == null ? "" : item.getCreateBy().getFirstName() + " " + item.getCreateBy().getLastName(); + this.userId = item.getCreateBy() == null ? null : item.getCreateBy().getUserId(); } else { this.itemUid = new Long(-1); } @@ -181,4 +183,12 @@ public void setInitGroup(boolean isInitGroup) { this.isInitGroup = isInitGroup; } + + public Long getUserId() { + return userId; + } + + public void setUserId(Long userId) { + this.userId = userId; + } } \ No newline at end of file Index: lams_tool_images/src/java/org/lamsfoundation/lams/tool/imageGallery/web/action/MonitoringAction.java =================================================================== diff -u -r942b2a5bf6dac160bf9e00bac22e6232ae65ef98 -rf851d4f476f3856ffb55d5004602a57f0dbe02c0 --- lams_tool_images/src/java/org/lamsfoundation/lams/tool/imageGallery/web/action/MonitoringAction.java (.../MonitoringAction.java) (revision 942b2a5bf6dac160bf9e00bac22e6232ae65ef98) +++ lams_tool_images/src/java/org/lamsfoundation/lams/tool/imageGallery/web/action/MonitoringAction.java (.../MonitoringAction.java) (revision f851d4f476f3856ffb55d5004602a57f0dbe02c0) @@ -144,7 +144,7 @@ ImageGallery imageGallery = (ImageGallery) sessionMap.get(ImageGalleryConstants.ATTR_IMAGE_GALLERY); Long imageUid = new Long(request.getParameter(ImageGalleryConstants.PARAM_IMAGE_UID)); ImageGalleryItem image = getImageGalleryService().getImageGalleryItemByUid(imageUid); - Long toolSessionId = WebUtil.readLongParam(request, ImageGalleryConstants.ATTR_SESSION_MAP_ID); + Long toolSessionId = WebUtil.readLongParam(request, ImageGalleryConstants.PARAM_TOOL_SESSION_ID); if (imageGallery.isAllowVote()) { List> imageSummary = getImageGalleryService().getImageSummary(contentId, Index: lams_tool_images/web/pages/monitoring/summary.jsp =================================================================== diff -u -r4583983b64efe1d91fbb47cdde6a759a6a30e859 -rf851d4f476f3856ffb55d5004602a57f0dbe02c0 --- lams_tool_images/web/pages/monitoring/summary.jsp (.../summary.jsp) (revision 4583983b64efe1d91fbb47cdde6a759a6a30e859) +++ lams_tool_images/web/pages/monitoring/summary.jsp (.../summary.jsp) (revision f851d4f476f3856ffb55d5004602a57f0dbe02c0) @@ -1,7 +1,15 @@ <%@ include file="/common/taglibs.jsp"%> + + + +

@@ -94,7 +102,7 @@ ${image.thumbnailFileUuid}&preferDownload=false - ?sessionMapID=${sessionMapID}&imageUid=${image.uid}&KeepThis=true&TB_iframe=true&modal=true&toolSessionID=${sessionId} + ?sessionMapID=${sessionMapID}&imageUid=${image.uid}&toolSessionID=${sessionId}&KeepThis=true&TB_iframe=true&modal=true @@ -107,15 +115,24 @@ - - [ ] - - - - + + + + + + + ${fn:replace(portraitURL, "class=\"", "href=\"REPLACEMEURL\" class=\"thickbox ")} + ${fn:replace(authorref, "REPLACEMEURL", url)} + + + + + + [  ${authorref} ] + - + @@ -195,7 +212,7 @@ - + Index: lams_tool_imscc/src/java/org/lamsfoundation/lams/tool/commonCartridge/dto/ReflectDTO.java =================================================================== diff -u -r5773f84ed608838de3521ecde87c52f3c72d478c -rf851d4f476f3856ffb55d5004602a57f0dbe02c0 --- lams_tool_imscc/src/java/org/lamsfoundation/lams/tool/commonCartridge/dto/ReflectDTO.java (.../ReflectDTO.java) (revision 5773f84ed608838de3521ecde87c52f3c72d478c) +++ lams_tool_imscc/src/java/org/lamsfoundation/lams/tool/commonCartridge/dto/ReflectDTO.java (.../ReflectDTO.java) (revision f851d4f476f3856ffb55d5004602a57f0dbe02c0) @@ -9,6 +9,7 @@ */ public class ReflectDTO { private Long userUid; + private Long userId; private String fullName; private String loginName; private boolean hasRefection; @@ -20,6 +21,7 @@ this.setLoginName(user.getLoginName()); this.setFullName(user.getFirstName() + " " + user.getLastName()); this.setUserUid(user.getUid()); + this.setUserId(user.getUserId()); } public boolean isFinishReflection() { @@ -77,4 +79,12 @@ public void setUserUid(Long userUid) { this.userUid = userUid; } + + public Long getUserId() { + return userId; + } + + public void setUserId(Long userId) { + this.userId = userId; + } } Index: lams_tool_imscc/web/pages/monitoring/summary.jsp =================================================================== diff -u -rd703ba41fe0d49d4dee425f8aed091eeb0c958b3 -rf851d4f476f3856ffb55d5004602a57f0dbe02c0 --- lams_tool_imscc/web/pages/monitoring/summary.jsp (.../summary.jsp) (revision d703ba41fe0d49d4dee425f8aed091eeb0c958b3) +++ lams_tool_imscc/web/pages/monitoring/summary.jsp (.../summary.jsp) (revision f851d4f476f3856ffb55d5004602a57f0dbe02c0) @@ -2,6 +2,13 @@ + + +

@@ -143,7 +150,7 @@ - + Index: lams_tool_imscc/web/pages/monitoring/userlist.jsp =================================================================== diff -u -rd703ba41fe0d49d4dee425f8aed091eeb0c958b3 -rf851d4f476f3856ffb55d5004602a57f0dbe02c0 --- lams_tool_imscc/web/pages/monitoring/userlist.jsp (.../userlist.jsp) (revision d703ba41fe0d49d4dee425f8aed091eeb0c958b3) +++ lams_tool_imscc/web/pages/monitoring/userlist.jsp (.../userlist.jsp) (revision f851d4f476f3856ffb55d5004602a57f0dbe02c0) @@ -4,8 +4,14 @@ <%@ include file="/common/header.jsp" %> + + - + @@ -27,7 +33,8 @@ - + + Index: lams_tool_kaltura/src/java/org/lamsfoundation/lams/tool/kaltura/dto/NotebookEntryDTO.java =================================================================== diff -u -r5773f84ed608838de3521ecde87c52f3c72d478c -rf851d4f476f3856ffb55d5004602a57f0dbe02c0 --- lams_tool_kaltura/src/java/org/lamsfoundation/lams/tool/kaltura/dto/NotebookEntryDTO.java (.../NotebookEntryDTO.java) (revision 5773f84ed608838de3521ecde87c52f3c72d478c) +++ lams_tool_kaltura/src/java/org/lamsfoundation/lams/tool/kaltura/dto/NotebookEntryDTO.java (.../NotebookEntryDTO.java) (revision f851d4f476f3856ffb55d5004602a57f0dbe02c0) @@ -7,6 +7,8 @@ public class NotebookEntryDTO { private Long uid; + + private Long userId; private String entry; @@ -63,4 +65,12 @@ this.fullName = fullName; } + public Long getUserId() { + return userId; + } + + public void setUserId(Long userId) { + this.userId = userId; + } + } Index: lams_tool_kaltura/src/java/org/lamsfoundation/lams/tool/kaltura/service/KalturaService.java =================================================================== diff -u -r2abc3485dc2d24ea02044a64271f3ee0d3b8c11b -rf851d4f476f3856ffb55d5004602a57f0dbe02c0 --- lams_tool_kaltura/src/java/org/lamsfoundation/lams/tool/kaltura/service/KalturaService.java (.../KalturaService.java) (revision 2abc3485dc2d24ea02044a64271f3ee0d3b8c11b) +++ lams_tool_kaltura/src/java/org/lamsfoundation/lams/tool/kaltura/service/KalturaService.java (.../KalturaService.java) (revision f851d4f476f3856ffb55d5004602a57f0dbe02c0) @@ -400,6 +400,7 @@ if (entry != null) { NotebookEntryDTO notebookEntryDTO = new NotebookEntryDTO(entry); notebookEntryDTO.setFullName(user.getFirstName() + " " + user.getLastName()); + notebookEntryDTO.setUserId(user.getUserId()); Date postedDate = (entry.getLastModified() != null) ? entry.getLastModified() : entry.getCreateDate(); notebookEntryDTO.setLastModified(postedDate); Index: lams_tool_kaltura/web/pages/monitoring/reflections.jsp =================================================================== diff -u -re535a51e19cbc72d07bda00b019c947847d869b0 -rf851d4f476f3856ffb55d5004602a57f0dbe02c0 --- lams_tool_kaltura/web/pages/monitoring/reflections.jsp (.../reflections.jsp) (revision e535a51e19cbc72d07bda00b019c947847d869b0) +++ lams_tool_kaltura/web/pages/monitoring/reflections.jsp (.../reflections.jsp) (revision f851d4f476f3856ffb55d5004602a57f0dbe02c0) @@ -1,5 +1,12 @@ <%@ include file="/common/taglibs.jsp"%> + + +
@@ -17,7 +24,7 @@ - - + -
Index: lams_tool_lamc/conf/language/lams/ApplicationResources.properties =================================================================== diff -u -r6c575f01c954df59ea67ff2bc26b0a0af4a2ade8 -rf851d4f476f3856ffb55d5004602a57f0dbe02c0 --- lams_tool_lamc/conf/language/lams/ApplicationResources.properties (.../ApplicationResources.properties) (revision 6c575f01c954df59ea67ff2bc26b0a0af4a2ade8) +++ lams_tool_lamc/conf/language/lams/ApplicationResources.properties (.../ApplicationResources.properties) (revision f851d4f476f3856ffb55d5004602a57f0dbe02c0) @@ -210,6 +210,8 @@ label.number.learners.in.mark.range =Number of learners in mark range label.number.groups.in.mark.range =Number of groups in mark range label.average.mark =Average mark +label.error =Error +error.loaderror =There was an error loading the grid. If this problem persists, please contact your system administrator. +label.ok =OK - #======= End labels: Exported 204 labels for en AU ===== Index: lams_tool_lamc/src/java/org/lamsfoundation/lams/tool/mc/dao/IMcUserDAO.java =================================================================== diff -u -rec381d32c228f460e0fd3ce3857aab14d4f6fd87 -rf851d4f476f3856ffb55d5004602a57f0dbe02c0 --- lams_tool_lamc/src/java/org/lamsfoundation/lams/tool/mc/dao/IMcUserDAO.java (.../IMcUserDAO.java) (revision ec381d32c228f460e0fd3ce3857aab14d4f6fd87) +++ lams_tool_lamc/src/java/org/lamsfoundation/lams/tool/mc/dao/IMcUserDAO.java (.../IMcUserDAO.java) (revision f851d4f476f3856ffb55d5004602a57f0dbe02c0) @@ -26,6 +26,7 @@ import org.lamsfoundation.lams.tool.mc.dto.McUserMarkDTO; import org.lamsfoundation.lams.tool.mc.pojos.McQueUsr; +import org.lamsfoundation.lams.usermanagement.service.IUserManagementService; /** * @author Ozgur Demirtas @@ -72,7 +73,7 @@ void removeMcUser(McQueUsr mcUser); List getPagedUsersBySession(Long sessionId, int page, int size, String sortBy, String sortOrder, - String searchString); + String searchString, IUserManagementService userManagementService); int getCountPagedUsersBySession(Long sessionId, String searchString); Index: lams_tool_lamc/src/java/org/lamsfoundation/lams/tool/mc/dao/hibernate/McUserDAO.java =================================================================== diff -u -rec381d32c228f460e0fd3ce3857aab14d4f6fd87 -rf851d4f476f3856ffb55d5004602a57f0dbe02c0 --- lams_tool_lamc/src/java/org/lamsfoundation/lams/tool/mc/dao/hibernate/McUserDAO.java (.../McUserDAO.java) (revision ec381d32c228f460e0fd3ce3857aab14d4f6fd87) +++ lams_tool_lamc/src/java/org/lamsfoundation/lams/tool/mc/dao/hibernate/McUserDAO.java (.../McUserDAO.java) (revision f851d4f476f3856ffb55d5004602a57f0dbe02c0) @@ -33,6 +33,7 @@ import org.lamsfoundation.lams.tool.mc.dao.IMcUserDAO; import org.lamsfoundation.lams.tool.mc.dto.McUserMarkDTO; import org.lamsfoundation.lams.tool.mc.pojos.McQueUsr; +import org.lamsfoundation.lams.usermanagement.service.IUserManagementService; import org.springframework.stereotype.Repository; /** @@ -100,19 +101,30 @@ public void removeMcUser(McQueUsr mcUser) { this.getSession().delete(mcUser); } + + final String LOAD_USERS_SELECT = "SELECT user.uid, user.que_usr_id, user.fullname, user.last_attempt_total_mark "; + final String LOAD_USERS_FROM = " FROM tl_lamc11_que_usr user "; + final String LOAD_USERS_JOINWHERE = " JOIN tl_lamc11_session session on user.mc_session_id = session.uid " + + " WHERE session.mc_session_id = :sessionId " + + " AND (user.fullname LIKE CONCAT('%', :searchString, '%')) " + + " ORDER BY CASE WHEN :sortBy='userName' THEN user.fullname " + + " WHEN :sortBy='total' THEN user.last_attempt_total_mark END "; @SuppressWarnings("unchecked") @Override public List getPagedUsersBySession(Long sessionId, int page, int size, String sortBy, - String sortOrder, String searchString) { + String sortOrder, String searchString, IUserManagementService userManagementService) { - final String LOAD_USERS = "SELECT DISTINCT user.uid, user.fullname, user.lastAttemptTotalMark " + "FROM " - + McQueUsr.class.getName() + " user " + "WHERE user.mcSession.mcSessionId = :sessionId " - + " AND (user.fullname LIKE CONCAT('%', :searchString, '%')) " + " ORDER BY " + " CASE " - + " WHEN :sortBy='userName' THEN user.fullname " - + " WHEN :sortBy='total' THEN user.lastAttemptTotalMark " + " END " + sortOrder; + String[] portraitStrings = userManagementService.getPortraitSQL("user.que_usr_id"); - Query query = getSession().createQuery(LOAD_USERS); + StringBuilder bldr = new StringBuilder(LOAD_USERS_SELECT) + .append(portraitStrings[0]) + .append(LOAD_USERS_FROM) + .append(portraitStrings[1]) + .append(LOAD_USERS_JOINWHERE) + .append(sortOrder); + + SQLQuery query = getSession().createSQLQuery(bldr.toString()); query.setLong("sessionId", sessionId); // support for custom search from a toolbar searchString = searchString == null ? "" : searchString; @@ -126,14 +138,18 @@ if (list != null && list.size() > 0) { for (Object[] element : list) { - Long userId = ((Number) element[0]).longValue(); - String fullName = (String) element[1]; - Integer totalMark = element[2] == null ? 0 : ((Number) element[2]).intValue(); + Long userUid = ((Number) element[0]).longValue(); + Long userId = ((Number) element[1]).longValue(); + String fullName = (String) element[2]; + Integer totalMark = element[3] == null ? 0 : ((Number) element[3]).intValue(); + Long portraitId = element[4] == null ? null : ((Number) element[4]).longValue(); McUserMarkDTO userDto = new McUserMarkDTO(); - userDto.setQueUsrId(userId.toString()); + userDto.setQueUsrId(userUid.toString()); + userDto.setUserId(userId.toString()); userDto.setFullName(fullName); userDto.setTotalMark(new Long(totalMark)); + userDto.setPortraitId(portraitId==null ? null : portraitId.toString()); userDtos.add(userDto); } Index: lams_tool_lamc/src/java/org/lamsfoundation/lams/tool/mc/dto/McUserMarkDTO.java =================================================================== diff -u -r289926a27bdbc9bd2519e3064a85f489fc1845ec -rf851d4f476f3856ffb55d5004602a57f0dbe02c0 --- lams_tool_lamc/src/java/org/lamsfoundation/lams/tool/mc/dto/McUserMarkDTO.java (.../McUserMarkDTO.java) (revision 289926a27bdbc9bd2519e3064a85f489fc1845ec) +++ lams_tool_lamc/src/java/org/lamsfoundation/lams/tool/mc/dto/McUserMarkDTO.java (.../McUserMarkDTO.java) (revision f851d4f476f3856ffb55d5004602a57f0dbe02c0) @@ -36,7 +36,9 @@ public class McUserMarkDTO implements Comparable { private String sessionId; private String sessionName; - private String queUsrId; + private String queUsrId; // mc user uid + private String userId; // LAMS userId + private String portraitId; private String userName; private String fullName; private boolean isUserLeader; @@ -205,4 +207,20 @@ public void setAttemptTime(Date attemptTime) { this.attemptTime = attemptTime; } + + public String getUserId() { + return userId; + } + + public void setUserId(String userId) { + this.userId = userId; + } + + public String getPortraitId() { + return portraitId; + } + + public void setPortraitId(String portraitId) { + this.portraitId = portraitId; + } } Index: lams_tool_lamc/src/java/org/lamsfoundation/lams/tool/mc/dto/ReflectionDTO.java =================================================================== diff -u -r289926a27bdbc9bd2519e3064a85f489fc1845ec -rf851d4f476f3856ffb55d5004602a57f0dbe02c0 --- lams_tool_lamc/src/java/org/lamsfoundation/lams/tool/mc/dto/ReflectionDTO.java (.../ReflectionDTO.java) (revision 289926a27bdbc9bd2519e3064a85f489fc1845ec) +++ lams_tool_lamc/src/java/org/lamsfoundation/lams/tool/mc/dto/ReflectionDTO.java (.../ReflectionDTO.java) (revision f851d4f476f3856ffb55d5004602a57f0dbe02c0) @@ -23,6 +23,8 @@ package org.lamsfoundation.lams.tool.mc.dto; import org.apache.commons.lang.builder.ToStringBuilder; +import org.lamsfoundation.lams.notebook.model.NotebookEntry; +import org.lamsfoundation.lams.tool.mc.pojos.McQueUsr; /** *

@@ -42,6 +44,15 @@ protected String entry; + public ReflectionDTO(McQueUsr user, String sessionId, NotebookEntry notebookEntry) { + this.setUserId(user.getQueUsrId().toString()); + this.setSessionId(sessionId); + this.setUserName(user.getFullname()); + this.setReflectionUid(notebookEntry.getUid().toString()); + // String notebookEntryPresentable = McUtils.replaceNewLines(notebookEntry.getEntry()); + this.setEntry(notebookEntry.getEntry()); + } + @Override public int compareTo(Object o) { ReflectionDTO reflectionDTO = (ReflectionDTO) o; Index: lams_tool_lamc/src/java/org/lamsfoundation/lams/tool/mc/service/McService.java =================================================================== diff -u -r85ef8aca5722a36582117db1e8b988e2c16b6369 -rf851d4f476f3856ffb55d5004602a57f0dbe02c0 --- lams_tool_lamc/src/java/org/lamsfoundation/lams/tool/mc/service/McService.java (.../McService.java) (revision 85ef8aca5722a36582117db1e8b988e2c16b6369) +++ lams_tool_lamc/src/java/org/lamsfoundation/lams/tool/mc/service/McService.java (.../McService.java) (revision f851d4f476f3856ffb55d5004602a57f0dbe02c0) @@ -418,7 +418,8 @@ @Override public List getPagedUsersBySession(Long sessionId, int page, int size, String sortBy, String sortOrder, String searchString) { - return mcUserDAO.getPagedUsersBySession(sessionId, page, size, sortBy, sortOrder, searchString); + return mcUserDAO.getPagedUsersBySession(sessionId, page, size, sortBy, sortOrder, searchString, + userManagementService); } @Override @@ -572,6 +573,7 @@ mcUserMarkDTO.setUserGroupLeader(session.isUserGroupLeader(user)); mcUserMarkDTO.setUserName(user.getUsername()); mcUserMarkDTO.setQueUsrId(user.getUid().toString()); + mcUserMarkDTO.setUserId(user.getQueUsrId().toString()); if (isFullAttemptDetailsRequired) { @@ -1689,54 +1691,22 @@ @Override public List getReflectionList(McContent mcContent, Long userID) { List reflectionsContainerDTO = new LinkedList(); - if (userID == null) { - // all users mode - for (McSession mcSession : (Set) mcContent.getMcSessions()) { - - for (McQueUsr user : (Set) mcSession.getMcQueUsers()) { - + for (McSession mcSession : (Set) mcContent.getMcSessions()) { + for (McQueUsr user : (Set) mcSession.getMcQueUsers()) { + // if all users mode or single user mode and found right user + if (userID == null || user.getQueUsrId().equals(userID)) { NotebookEntry notebookEntry = this.getEntry(mcSession.getMcSessionId(), CoreNotebookConstants.NOTEBOOK_TOOL, McAppConstants.TOOL_SIGNATURE, new Integer(user.getQueUsrId().toString())); if (notebookEntry != null) { - ReflectionDTO reflectionDTO = new ReflectionDTO(); - reflectionDTO.setUserId(user.getQueUsrId().toString()); - reflectionDTO.setSessionId(mcSession.getMcSessionId().toString()); - reflectionDTO.setUserName(user.getFullname()); - reflectionDTO.setReflectionUid(notebookEntry.getUid().toString()); - // String notebookEntryPresentable = McUtils.replaceNewLines(notebookEntry.getEntry()); - reflectionDTO.setEntry(notebookEntry.getEntry()); + ReflectionDTO reflectionDTO = new ReflectionDTO(user, + mcSession.getMcSessionId().toString(), notebookEntry); reflectionsContainerDTO.add(reflectionDTO); } } } - } else { - // single user mode - for (Iterator sessionIter = mcContent.getMcSessions().iterator(); sessionIter.hasNext();) { - McSession mcSession = (McSession) sessionIter.next(); - for (Iterator userIter = mcSession.getMcQueUsers().iterator(); userIter.hasNext();) { - McQueUsr user = (McQueUsr) userIter.next(); - if (user.getQueUsrId().equals(userID)) { - NotebookEntry notebookEntry = this.getEntry(mcSession.getMcSessionId(), - CoreNotebookConstants.NOTEBOOK_TOOL, McAppConstants.TOOL_SIGNATURE, - new Integer(user.getQueUsrId().toString())); - - if (notebookEntry != null) { - ReflectionDTO reflectionDTO = new ReflectionDTO(); - reflectionDTO.setUserId(user.getQueUsrId().toString()); - reflectionDTO.setSessionId(mcSession.getMcSessionId().toString()); - reflectionDTO.setUserName(user.getFullname()); - reflectionDTO.setReflectionUid(notebookEntry.getUid().toString()); - // String notebookEntryPresentable = McUtils.replaceNewLines(notebookEntry.getEntry()); - reflectionDTO.setEntry(notebookEntry.getEntry()); - reflectionsContainerDTO.add(reflectionDTO); - } - } - } - } } - return reflectionsContainerDTO; } Index: lams_tool_lamc/src/java/org/lamsfoundation/lams/tool/mc/web/action/McMonitoringAction.java =================================================================== diff -u -r4170df8bc66e658ef4dcd47e99eceddec3c2673a -rf851d4f476f3856ffb55d5004602a57f0dbe02c0 --- lams_tool_lamc/src/java/org/lamsfoundation/lams/tool/mc/web/action/McMonitoringAction.java (.../McMonitoringAction.java) (revision 4170df8bc66e658ef4dcd47e99eceddec3c2673a) +++ lams_tool_lamc/src/java/org/lamsfoundation/lams/tool/mc/web/action/McMonitoringAction.java (.../McMonitoringAction.java) (revision f851d4f476f3856ffb55d5004602a57f0dbe02c0) @@ -302,6 +302,8 @@ JSONArray visitLogData = new JSONArray(); Long userUid = Long.parseLong(userDto.getQueUsrId()); visitLogData.put(userUid); + visitLogData.put(userDto.getUserId()); + String fullName = StringEscapeUtils.escapeHtml(userDto.getFullName()); if (groupLeader != null && groupLeader.getUid().equals(userUid)) { fullName += " (" + mcService.getLocalizedMessage("label.monitoring.group.leader") + ")"; @@ -311,6 +313,8 @@ Long totalMark = (userDto.getTotalMark() == null) ? 0 : userDto.getTotalMark(); visitLogData.put(totalMark); + visitLogData.put(userDto.getPortraitId()); + JSONObject userRow = new JSONObject(); userRow.put("id", i++); userRow.put("cell", visitLogData); Index: lams_tool_lamc/web/monitoring/MonitoringMaincontent.jsp =================================================================== diff -u -rec381d32c228f460e0fd3ce3857aab14d4f6fd87 -rf851d4f476f3856ffb55d5004602a57f0dbe02c0 --- lams_tool_lamc/web/monitoring/MonitoringMaincontent.jsp (.../MonitoringMaincontent.jsp) (revision ec381d32c228f460e0fd3ce3857aab14d4f6fd87) +++ lams_tool_lamc/web/monitoring/MonitoringMaincontent.jsp (.../MonitoringMaincontent.jsp) (revision f851d4f476f3856ffb55d5004602a57f0dbe02c0) @@ -8,7 +8,7 @@ <fmt:message key="label.monitoring"/> <%@ include file="/common/monitoringheader.jsp"%> - + +