Index: lams_tool_images/src/java/org/lamsfoundation/lams/tool/imageGallery/service/IImageGalleryService.java =================================================================== diff -u -r2abc3485dc2d24ea02044a64271f3ee0d3b8c11b -r942b2a5bf6dac160bf9e00bac22e6232ae65ef98 --- lams_tool_images/src/java/org/lamsfoundation/lams/tool/imageGallery/service/IImageGalleryService.java (.../IImageGalleryService.java) (revision 2abc3485dc2d24ea02044a64271f3ee0d3b8c11b) +++ lams_tool_images/src/java/org/lamsfoundation/lams/tool/imageGallery/service/IImageGalleryService.java (.../IImageGalleryService.java) (revision 942b2a5bf6dac160bf9e00bac22e6232ae65ef98) @@ -122,7 +122,7 @@ */ Set getImagesForGroup(ImageGallery imageGallery, Long sessionId); - ItemRatingDTO getRatingCriteriaDtos(Long contentId, Long imageUid, Long userId); + ItemRatingDTO getRatingCriteriaDtos(Long contentId, Long toolSessionId, Long imageUid, Long userId); /** * Save/update current ImageGalleryItem. Index: lams_tool_images/src/java/org/lamsfoundation/lams/tool/imageGallery/service/ImageGalleryOutputFactory.java =================================================================== diff -u -r3399163940c61c9132223c758d274486e57ff9b7 -r942b2a5bf6dac160bf9e00bac22e6232ae65ef98 --- lams_tool_images/src/java/org/lamsfoundation/lams/tool/imageGallery/service/ImageGalleryOutputFactory.java (.../ImageGalleryOutputFactory.java) (revision 3399163940c61c9132223c758d274486e57ff9b7) +++ lams_tool_images/src/java/org/lamsfoundation/lams/tool/imageGallery/service/ImageGalleryOutputFactory.java (.../ImageGalleryOutputFactory.java) (revision 942b2a5bf6dac160bf9e00bac22e6232ae65ef98) @@ -211,7 +211,7 @@ boolean isCommentsByOtherUsersRequired = user == null; Long userId = user == null ? -1L : user.getUserId(); - List ratingCriteriaDtos = imageGalleryService.getRatingCriteriaDtos(contentId, itemIds, + List ratingCriteriaDtos = imageGalleryService.getRatingCriteriaDtos(contentId, session.getSessionId(), itemIds, isCommentsByOtherUsersRequired, userId); if (user != null) { Index: lams_tool_images/src/java/org/lamsfoundation/lams/tool/imageGallery/service/ImageGalleryServiceImpl.java =================================================================== diff -u -r2abc3485dc2d24ea02044a64271f3ee0d3b8c11b -r942b2a5bf6dac160bf9e00bac22e6232ae65ef98 --- lams_tool_images/src/java/org/lamsfoundation/lams/tool/imageGallery/service/ImageGalleryServiceImpl.java (.../ImageGalleryServiceImpl.java) (revision 2abc3485dc2d24ea02044a64271f3ee0d3b8c11b) +++ lams_tool_images/src/java/org/lamsfoundation/lams/tool/imageGallery/service/ImageGalleryServiceImpl.java (.../ImageGalleryServiceImpl.java) (revision 942b2a5bf6dac160bf9e00bac22e6232ae65ef98) @@ -344,19 +344,19 @@ } @Override - public ItemRatingDTO getRatingCriteriaDtos(Long contentId, Long imageUid, Long userId) { + public ItemRatingDTO getRatingCriteriaDtos(Long contentId, Long toolSessionId, Long imageUid, Long userId) { LinkedList itemIds = new LinkedList(); itemIds.add(imageUid); - ItemRatingDTO ratingCriteria = getRatingCriteriaDtos(contentId, itemIds, true, userId).get(0); + ItemRatingDTO ratingCriteria = getRatingCriteriaDtos(contentId, toolSessionId, itemIds, true, userId).get(0); return ratingCriteria; } @Override - public List getRatingCriteriaDtos(Long contentId, Collection itemIds, + public List getRatingCriteriaDtos(Long contentId, Long toolSessionId, Collection itemIds, boolean isCommentsByOtherUsersRequired, Long userId) { - return ratingService.getRatingCriteriaDtos(contentId, itemIds, isCommentsByOtherUsersRequired, userId); + return ratingService.getRatingCriteriaDtos(contentId, toolSessionId, itemIds, isCommentsByOtherUsersRequired, userId); } @Override @@ -411,7 +411,7 @@ final boolean IS_COMMENTS_BY_OTHER_USERS_REQUIRED = true; // !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!TODO calculate average ratings based on one // session data - itemRatingDtos = getRatingCriteriaDtos(contentId, itemIds, IS_COMMENTS_BY_OTHER_USERS_REQUIRED, + itemRatingDtos = getRatingCriteriaDtos(contentId, session.getSessionId(), itemIds, IS_COMMENTS_BY_OTHER_USERS_REQUIRED, USER_ID); } Index: lams_tool_images/src/java/org/lamsfoundation/lams/tool/imageGallery/web/action/LearningAction.java =================================================================== diff -u -r2228652cfb59ec7c0fcbb1914a94f65d22815911 -r942b2a5bf6dac160bf9e00bac22e6232ae65ef98 --- lams_tool_images/src/java/org/lamsfoundation/lams/tool/imageGallery/web/action/LearningAction.java (.../LearningAction.java) (revision 2228652cfb59ec7c0fcbb1914a94f65d22815911) +++ lams_tool_images/src/java/org/lamsfoundation/lams/tool/imageGallery/web/action/LearningAction.java (.../LearningAction.java) (revision 942b2a5bf6dac160bf9e00bac22e6232ae65ef98) @@ -434,9 +434,11 @@ int commentsMinWordsLimit = 0; boolean isCommentsEnabled = false; int countRatedImages = 0; + Long toolSessionId = (Long) sessionMap.get(ImageGalleryConstants.ATTR_TOOL_SESSION_ID); + if (imageGallery.isAllowRank()) { - ItemRatingDTO itemRatingDto = service.getRatingCriteriaDtos(imageGallery.getContentId(), imageUid, userId); + ItemRatingDTO itemRatingDto = service.getRatingCriteriaDtos(imageGallery.getContentId(), toolSessionId, imageUid, userId); sessionMap.put(AttributeNames.ATTR_ITEM_RATING_DTO, itemRatingDto); if (itemRatingDto != null) { Index: lams_tool_images/src/java/org/lamsfoundation/lams/tool/imageGallery/web/action/MonitoringAction.java =================================================================== diff -u -r2228652cfb59ec7c0fcbb1914a94f65d22815911 -r942b2a5bf6dac160bf9e00bac22e6232ae65ef98 --- lams_tool_images/src/java/org/lamsfoundation/lams/tool/imageGallery/web/action/MonitoringAction.java (.../MonitoringAction.java) (revision 2228652cfb59ec7c0fcbb1914a94f65d22815911) +++ lams_tool_images/src/java/org/lamsfoundation/lams/tool/imageGallery/web/action/MonitoringAction.java (.../MonitoringAction.java) (revision 942b2a5bf6dac160bf9e00bac22e6232ae65ef98) @@ -144,14 +144,15 @@ 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); + if (imageGallery.isAllowVote()) { List> imageSummary = getImageGalleryService().getImageSummary(contentId, imageUid); request.setAttribute(ImageGalleryConstants.ATTR_IMAGE_SUMMARY, imageSummary); } else if (imageGallery.isAllowRank()) { - ItemRatingDTO itemRatingDto = getImageGalleryService().getRatingCriteriaDtos(contentId, imageUid, -1L); + ItemRatingDTO itemRatingDto = getImageGalleryService().getRatingCriteriaDtos(contentId, toolSessionId, imageUid, -1L); request.setAttribute("itemRatingDto", itemRatingDto); } Index: lams_tool_images/web/pages/learning/parts/commentsarea.jsp =================================================================== diff -u -r50551b789065b93152949df172cb9c718962697e -r942b2a5bf6dac160bf9e00bac22e6232ae65ef98 --- lams_tool_images/web/pages/learning/parts/commentsarea.jsp (.../commentsarea.jsp) (revision 50551b789065b93152949df172cb9c718962697e) +++ lams_tool_images/web/pages/learning/parts/commentsarea.jsp (.../commentsarea.jsp) (revision 942b2a5bf6dac160bf9e00bac22e6232ae65ef98) @@ -9,6 +9,7 @@ + @@ -26,7 +27,9 @@ COUNT_RATED_ITEMS = ${sessionMap.countRatedItems}, COMMENT_TEXTAREA_TIP_LABEL = '', WARN_COMMENTS_IS_BLANK_LABEL = '', - WARN_MIN_NUMBER_WORDS_LABEL = ''; + WARN_MIN_NUMBER_WORDS_LABEL = '', + ALLOW_RERATE = false, + SESSION_ID = ${toolSessionID}; Index: lams_tool_images/web/pages/monitoring/summary.jsp =================================================================== diff -u -r2228652cfb59ec7c0fcbb1914a94f65d22815911 -r942b2a5bf6dac160bf9e00bac22e6232ae65ef98 --- lams_tool_images/web/pages/monitoring/summary.jsp (.../summary.jsp) (revision 2228652cfb59ec7c0fcbb1914a94f65d22815911) +++ lams_tool_images/web/pages/monitoring/summary.jsp (.../summary.jsp) (revision 942b2a5bf6dac160bf9e00bac22e6232ae65ef98) @@ -94,7 +94,7 @@ ${image.thumbnailFileUuid}&preferDownload=false - ?sessionMapID=${sessionMapID}&imageUid=${image.uid}&KeepThis=true&TB_iframe=true&modal=true + ?sessionMapID=${sessionMapID}&imageUid=${image.uid}&KeepThis=true&TB_iframe=true&modal=true&toolSessionID=${sessionId} Index: lams_tool_laqa/src/java/org/lamsfoundation/lams/tool/qa/service/QaServicePOJO.java =================================================================== diff -u -r2abc3485dc2d24ea02044a64271f3ee0d3b8c11b -r942b2a5bf6dac160bf9e00bac22e6232ae65ef98 --- lams_tool_laqa/src/java/org/lamsfoundation/lams/tool/qa/service/QaServicePOJO.java (.../QaServicePOJO.java) (revision 2abc3485dc2d24ea02044a64271f3ee0d3b8c11b) +++ lams_tool_laqa/src/java/org/lamsfoundation/lams/tool/qa/service/QaServicePOJO.java (.../QaServicePOJO.java) (revision 942b2a5bf6dac160bf9e00bac22e6232ae65ef98) @@ -863,9 +863,9 @@ } @Override - public List getRatingCriteriaDtos(Long contentId, Collection itemIds, + public List getRatingCriteriaDtos(Long contentId, Long toolSessionId, Collection itemIds, boolean isCommentsByOtherUsersRequired, Long userId) { - return ratingService.getRatingCriteriaDtos(contentId, itemIds, isCommentsByOtherUsersRequired, userId); + return ratingService.getRatingCriteriaDtos(contentId, toolSessionId, itemIds, isCommentsByOtherUsersRequired, userId); } @Override Index: lams_tool_laqa/src/java/org/lamsfoundation/lams/tool/qa/web/QaLearningAction.java =================================================================== diff -u -r8d784c45657b2b667e4b98d28c754e1aeb604395 -r942b2a5bf6dac160bf9e00bac22e6232ae65ef98 --- lams_tool_laqa/src/java/org/lamsfoundation/lams/tool/qa/web/QaLearningAction.java (.../QaLearningAction.java) (revision 8d784c45657b2b667e4b98d28c754e1aeb604395) +++ lams_tool_laqa/src/java/org/lamsfoundation/lams/tool/qa/web/QaLearningAction.java (.../QaLearningAction.java) (revision 942b2a5bf6dac160bf9e00bac22e6232ae65ef98) @@ -374,7 +374,7 @@ generalLearnerFlowDTO.setHttpSessionID(httpSessionID); /** Set up the data for the view all answers screen */ - QaLearningAction.refreshSummaryData(request, qaContent, QaLearningAction.qaService, httpSessionID, user, + QaLearningAction.refreshSummaryData(request, qaContent, qaSession, QaLearningAction.qaService, httpSessionID, user, generalLearnerFlowDTO); generalLearnerFlowDTO.setRequestLearningReport(new Boolean(true).toString()); @@ -456,7 +456,7 @@ qaLearningForm.resetUserActions(); qaLearningForm.setSubmitAnswersContent(null); - QaLearningAction.refreshSummaryData(request, qaContent, QaLearningAction.qaService, httpSessionID, user, + QaLearningAction.refreshSummaryData(request, qaContent, qaSession, QaLearningAction.qaService, httpSessionID, user, generalLearnerFlowDTO); generalLearnerFlowDTO.setRequestLearningReport(new Boolean(true).toString()); @@ -798,7 +798,7 @@ generalLearnerFlowDTO.setNotebookEntry(entryText); generalLearnerFlowDTO.setRequestLearningReportProgress(new Boolean(true).toString()); - QaLearningAction.refreshSummaryData(request, qaContent, QaLearningAction.qaService, httpSessionID, qaQueUsr, + QaLearningAction.refreshSummaryData(request, qaContent, qaSession, QaLearningAction.qaService, httpSessionID, qaQueUsr, generalLearnerFlowDTO); boolean isLearnerFinished = qaQueUsr.isLearnerFinished(); @@ -928,7 +928,7 @@ * User id is needed if isUserNamesVisible is false && learnerRequest is true, as it is required to work out if the * data being analysed is the current user. */ - public static void refreshSummaryData(HttpServletRequest request, QaContent qaContent, IQaService qaService, + public static void refreshSummaryData(HttpServletRequest request, QaContent qaContent, QaSession qaSession, IQaService qaService, String httpSessionID, QaQueUsr user, GeneralLearnerFlowDTO generalLearnerFlowDTO) { SessionMap sessionMap = (SessionMap) request.getSession() @@ -963,7 +963,7 @@ itemIds.add(responseIter.getResponseId()); } - List itemRatingDtos = qaService.getRatingCriteriaDtos(qaContent.getQaContentId(), itemIds, + List itemRatingDtos = qaService.getRatingCriteriaDtos(qaContent.getQaContentId(), qaSession.getQaSessionId(), itemIds, true, userId); sessionMap.put(AttributeNames.ATTR_ITEM_RATING_DTOS, itemRatingDtos); @@ -991,6 +991,9 @@ countRatedQuestions = qaService.getCountItemsRatedByUser(qaContent.getQaContentId(), userId.intValue()); } } + + request.setAttribute(TOOL_SESSION_ID, qaSession.getQaSessionId()); + sessionMap.put("commentsMinWordsLimit", commentsMinWordsLimit); sessionMap.put("isCommentsEnabled", isCommentsEnabled); sessionMap.put(AttributeNames.ATTR_COUNT_RATED_ITEMS, countRatedQuestions); @@ -1010,7 +1013,7 @@ HttpSession ss = SessionManager.getSession(); UserDTO userDto = (UserDTO) ss.getAttribute(AttributeNames.USER); TimeZone userTimeZone = userDto.getTimeZone(); - + boolean isAllowRateAnswers = WebUtil.readBooleanParam(request, "isAllowRateAnswers"); boolean isAllowRichEditor = WebUtil.readBooleanParam(request, "isAllowRichEditor"); boolean isOnlyLeadersIncluded = WebUtil.readBooleanParam(request, "isOnlyLeadersIncluded", false); @@ -1064,7 +1067,7 @@ //all comments required only for monitoring boolean isCommentsByOtherUsersRequired = isMonitoring; - itemRatingDtos = QaLearningAction.qaService.getRatingCriteriaDtos(qaContentId, itemIds, + itemRatingDtos = QaLearningAction.qaService.getRatingCriteriaDtos(qaContentId, qaSessionId, itemIds, isCommentsByOtherUsersRequired, userId); // store how many items are rated Index: lams_tool_laqa/src/java/org/lamsfoundation/lams/tool/qa/web/QaLearningStarterAction.java =================================================================== diff -u -ra1e4f1c8cd378514548ca6efc7b8739910dafe91 -r942b2a5bf6dac160bf9e00bac22e6232ae65ef98 --- lams_tool_laqa/src/java/org/lamsfoundation/lams/tool/qa/web/QaLearningStarterAction.java (.../QaLearningStarterAction.java) (revision a1e4f1c8cd378514548ca6efc7b8739910dafe91) +++ lams_tool_laqa/src/java/org/lamsfoundation/lams/tool/qa/web/QaLearningStarterAction.java (.../QaLearningStarterAction.java) (revision 942b2a5bf6dac160bf9e00bac22e6232ae65ef98) @@ -272,7 +272,7 @@ generalLearnerFlowDTO.setRequestLearningReportProgress(new Boolean(true).toString()); generalLearnerFlowDTO.setTeacherViewOnly(new Boolean(true).toString()); - QaLearningAction.refreshSummaryData(request, qaContent, qaService, sessionMapId, user, + QaLearningAction.refreshSummaryData(request, qaContent, qaSession, qaService, sessionMapId, user, generalLearnerFlowDTO); request.setAttribute(QaAppConstants.GENERAL_LEARNER_FLOW_DTO, generalLearnerFlowDTO); @@ -306,7 +306,7 @@ */ generalLearnerFlowDTO.setRequestLearningReport(new Boolean(true).toString()); - QaLearningAction.refreshSummaryData(request, qaContent, qaService, sessionMapId, user, + QaLearningAction.refreshSummaryData(request, qaContent, qaSession, qaService, sessionMapId, user, generalLearnerFlowDTO); if (user.isLearnerFinished()) { @@ -357,7 +357,7 @@ */ generalLearnerFlowDTO.setRequestLearningReport(new Boolean(true).toString()); - QaLearningAction.refreshSummaryData(request, qaContent, qaService, sessionMapId, user, + QaLearningAction.refreshSummaryData(request, qaContent, qaSession, qaService, sessionMapId, user, generalLearnerFlowDTO); if (user.isLearnerFinished()) { Index: lams_tool_laqa/web/learning/LearnerRep.jsp =================================================================== diff -u -rd2211def89f93b05a695009b1c8be46205f17f70 -r942b2a5bf6dac160bf9e00bac22e6232ae65ef98 --- lams_tool_laqa/web/learning/LearnerRep.jsp (.../LearnerRep.jsp) (revision d2211def89f93b05a695009b1c8be46205f17f70) +++ lams_tool_laqa/web/learning/LearnerRep.jsp (.../LearnerRep.jsp) (revision 942b2a5bf6dac160bf9e00bac22e6232ae65ef98) @@ -35,7 +35,9 @@ COUNT_RATED_ITEMS = ${sessionMap.countRatedItems}, COMMENT_TEXTAREA_TIP_LABEL = '', WARN_COMMENTS_IS_BLANK_LABEL = '', - WARN_MIN_NUMBER_WORDS_LABEL = ""; + WARN_MIN_NUMBER_WORDS_LABEL = "", + ALLOW_RERATE = false, + SESSION_ID = ${toolSessionID}; Index: lams_tool_laqa/web/learning/RevisitedLearnerRep.jsp =================================================================== diff -u -rd2211def89f93b05a695009b1c8be46205f17f70 -r942b2a5bf6dac160bf9e00bac22e6232ae65ef98 --- lams_tool_laqa/web/learning/RevisitedLearnerRep.jsp (.../RevisitedLearnerRep.jsp) (revision d2211def89f93b05a695009b1c8be46205f17f70) +++ lams_tool_laqa/web/learning/RevisitedLearnerRep.jsp (.../RevisitedLearnerRep.jsp) (revision 942b2a5bf6dac160bf9e00bac22e6232ae65ef98) @@ -24,6 +24,7 @@ + - Index: lams_tool_larsrc/conf/hibernate/mappings/org/lamsfoundation/lams/tool/rsrc/model/Resource.hbm.xml =================================================================== diff -u -r8620cb3f29c4710a0ddfd597fdc1b676d4b2b4b0 -r942b2a5bf6dac160bf9e00bac22e6232ae65ef98 --- lams_tool_larsrc/conf/hibernate/mappings/org/lamsfoundation/lams/tool/rsrc/model/Resource.hbm.xml (.../Resource.hbm.xml) (revision 8620cb3f29c4710a0ddfd597fdc1b676d4b2b4b0) +++ lams_tool_larsrc/conf/hibernate/mappings/org/lamsfoundation/lams/tool/rsrc/model/Resource.hbm.xml (.../Resource.hbm.xml) (revision 942b2a5bf6dac160bf9e00bac22e6232ae65ef98) @@ -185,6 +185,25 @@ column="file_upload_notify" /> + + + + + + + Index: lams_tool_larsrc/web/pages/authoring/parts/addlearningobject.jsp =================================================================== diff -u -rafefa4ff09424c4462d405b7b8a6f75c8461e0c0 -r942b2a5bf6dac160bf9e00bac22e6232ae65ef98 --- lams_tool_larsrc/web/pages/authoring/parts/addlearningobject.jsp (.../addlearningobject.jsp) (revision afefa4ff09424c4462d405b7b8a6f75c8461e0c0) +++ lams_tool_larsrc/web/pages/authoring/parts/addlearningobject.jsp (.../addlearningobject.jsp) (revision 942b2a5bf6dac160bf9e00bac22e6232ae65ef98) @@ -93,6 +93,11 @@ +
+ +   +
+ Index: lams_tool_larsrc/web/pages/authoring/parts/addurl.jsp =================================================================== diff -u -rafefa4ff09424c4462d405b7b8a6f75c8461e0c0 -r942b2a5bf6dac160bf9e00bac22e6232ae65ef98 --- lams_tool_larsrc/web/pages/authoring/parts/addurl.jsp (.../addurl.jsp) (revision afefa4ff09424c4462d405b7b8a6f75c8461e0c0) +++ lams_tool_larsrc/web/pages/authoring/parts/addurl.jsp (.../addurl.jsp) (revision 942b2a5bf6dac160bf9e00bac22e6232ae65ef98) @@ -59,8 +59,13 @@
: -
+
+ +   + + +  
Index: lams_tool_larsrc/web/pages/authoring/parts/addwebsite.jsp =================================================================== diff -u -rafefa4ff09424c4462d405b7b8a6f75c8461e0c0 -r942b2a5bf6dac160bf9e00bac22e6232ae65ef98 --- lams_tool_larsrc/web/pages/authoring/parts/addwebsite.jsp (.../addwebsite.jsp) (revision afefa4ff09424c4462d405b7b8a6f75c8461e0c0) +++ lams_tool_larsrc/web/pages/authoring/parts/addwebsite.jsp (.../addwebsite.jsp) (revision 942b2a5bf6dac160bf9e00bac22e6232ae65ef98) @@ -94,6 +94,11 @@ +
+ +   +
+ Index: lams_tool_larsrc/web/pages/learning/learning.jsp =================================================================== diff -u -ra4bc09633e07b301174911591702607f522c7037 -r942b2a5bf6dac160bf9e00bac22e6232ae65ef98 --- lams_tool_larsrc/web/pages/learning/learning.jsp (.../learning.jsp) (revision a4bc09633e07b301174911591702607f522c7037) +++ lams_tool_larsrc/web/pages/learning/learning.jsp (.../learning.jsp) (revision 942b2a5bf6dac160bf9e00bac22e6232ae65ef98) @@ -9,6 +9,7 @@ <fmt:message key="label.learning.title" /> <%@ include file="/common/header.jsp"%> + <%-- param has higher level for request attribute --%> @@ -25,11 +26,31 @@ - + + + + + - + var reqIDVar = new Date(); + document.location.href = '?mode=${mode}&toolSessionID=${toolSessionID}&reqID='+reqIDVar.getTime(); +
- \ No newline at end of file Index: lams_tool_larsrc/web/pages/monitoring/summary.jsp =================================================================== diff -u -ra4bc09633e07b301174911591702607f522c7037 -r942b2a5bf6dac160bf9e00bac22e6232ae65ef98 --- lams_tool_larsrc/web/pages/monitoring/summary.jsp (.../summary.jsp) (revision a4bc09633e07b301174911591702607f522c7037) +++ lams_tool_larsrc/web/pages/monitoring/summary.jsp (.../summary.jsp) (revision 942b2a5bf6dac160bf9e00bac22e6232ae65ef98) @@ -3,7 +3,8 @@ - + + + + + +