Index: lams_tool_images/src/java/org/lamsfoundation/lams/tool/imageGallery/web/action/LearningAction.java =================================================================== diff -u -rdbbdbe1fcf86e07c4d5163178b4c679b4dedfed9 -rfac97347394b311e0f7123bc72d54c056be5275e --- lams_tool_images/src/java/org/lamsfoundation/lams/tool/imageGallery/web/action/LearningAction.java (.../LearningAction.java) (revision dbbdbe1fcf86e07c4d5163178b4c679b4dedfed9) +++ lams_tool_images/src/java/org/lamsfoundation/lams/tool/imageGallery/web/action/LearningAction.java (.../LearningAction.java) (revision fac97347394b311e0f7123bc72d54c056be5275e) @@ -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); }