Index: lams_tool_images/src/java/org/lamsfoundation/lams/tool/imageGallery/web/action/LearningAction.java =================================================================== diff -u -rbf49caa03b4d5d074f8c7dcfb3b761e66c57d58d -r6c4f0e1c0f459cf029ae028396b5435a12eb7516 --- lams_tool_images/src/java/org/lamsfoundation/lams/tool/imageGallery/web/action/LearningAction.java (.../LearningAction.java) (revision bf49caa03b4d5d074f8c7dcfb3b761e66c57d58d) +++ lams_tool_images/src/java/org/lamsfoundation/lams/tool/imageGallery/web/action/LearningAction.java (.../LearningAction.java) (revision 6c4f0e1c0f459cf029ae028396b5435a12eb7516) @@ -482,7 +482,12 @@ ImageGalleryUser createdBy = image.getCreateBy(); if (createdBy != null) { image.getCreateBy().getLoginName(); - } + } + + ToolAccessMode mode = (ToolAccessMode) sessionMap.get(AttributeNames.ATTR_MODE); + boolean isTeacher = mode != null && mode.isTeacher(); + boolean isAuthor = !isTeacher && !image.isCreateByAuthor() && (createdBy != null) + && (createdBy.getUserId().equals(imageGalleryUser.getUserId())); if (imageGallery.isAllowCommentImages()) { TreeSet comments = new TreeSet(new ImageCommentComparator()); @@ -498,7 +503,7 @@ sessionMap.put(ImageGalleryConstants.PARAM_COMMENTS, comments); } - if (imageGallery.isAllowRank()) { + if (!isTeacher && imageGallery.isAllowRank()) { ImageRating imageRating = service.getImageRatingByImageAndUser(imageUid, imageGalleryUser.getUserId()); int rating = imageRating == null ? 0 : imageRating.getRating(); @@ -508,7 +513,7 @@ sessionMap.put(ImageGalleryConstants.PARAM_CURRENT_RATING, rating); } - if (imageGallery.isAllowVote()) { + if (!isTeacher && imageGallery.isAllowVote()) { boolean isVotedForThisImage = false; ImageVote imageVote = service.getImageVoteByImageAndUser(image.getUid(), imageGalleryUser.getUserId()); if (imageVote != null && imageVote.isVoted()) { @@ -518,11 +523,7 @@ } // set visibility of "Delete image" button - ToolAccessMode mode = (ToolAccessMode) sessionMap.get(AttributeNames.ATTR_MODE); - boolean isAuthor = !mode.isTeacher() && !image.isCreateByAuthor() && (createdBy != null) - && (createdBy.getUserId().equals(imageGalleryUser.getUserId())); sessionMap.put(ImageGalleryConstants.PARAM_IS_AUTHOR, isAuthor); - request.setAttribute(ImageGalleryConstants.ATTR_SESSION_MAP_ID, sessionMapID); return mapping.findForward(ImageGalleryConstants.SUCCESS); }