Index: lams_central/src/java/org/lamsfoundation/lams/web/DisplayGroupAction.java =================================================================== RCS file: /usr/local/cvsroot/lams_central/src/java/org/lamsfoundation/lams/web/DisplayGroupAction.java,v diff -u -r1.48 -r1.49 --- lams_central/src/java/org/lamsfoundation/lams/web/DisplayGroupAction.java 3 Nov 2014 13:39:58 -0000 1.48 +++ lams_central/src/java/org/lamsfoundation/lams/web/DisplayGroupAction.java 8 Sep 2015 23:42:11 -0000 1.49 @@ -177,7 +177,7 @@ // Adding gradebook course monitor links if enabled if (org.getEnableGradebookForMonitors() - && (roles.contains(Role.ROLE_GROUP_MANAGER) || roles.contains(Role.ROLE_MONITOR))) { + && (roles.contains(Role.ROLE_GROUP_MANAGER) || roles.contains(Role.ROLE_GROUP_ADMIN))) { String link = "javascript:showGradebookCourseDialog(" + org.getOrganisationId() + ")"; moreLinks.add(new IndexLinkBean("index.coursegradebook", link, "course-gradebook-button", "index.coursegradebook.tooltip")); @@ -194,7 +194,7 @@ // Adding gradebook course monitor links if enabled if (org.getParentOrganisation().getEnableGradebookForMonitors() - && (roles.contains(Role.ROLE_GROUP_MANAGER) || roles.contains(Role.ROLE_MONITOR))) { + && (roles.contains(Role.ROLE_GROUP_MANAGER) || roles.contains(Role.ROLE_GROUP_ADMIN))) { String link = "javascript:showGradebookCourseDialog(" + org.getOrganisationId() + ")"; moreLinks.add(new IndexLinkBean("index.coursegradebook.subgroup", link, "my-grades-button", null)); } Index: lams_gradebook/src/java/org/lamsfoundation/lams/gradebook/web/action/GradebookMonitoringAction.java =================================================================== RCS file: /usr/local/cvsroot/lams_gradebook/src/java/org/lamsfoundation/lams/gradebook/web/action/GradebookMonitoringAction.java,v diff -u -r1.19 -r1.20 --- lams_gradebook/src/java/org/lamsfoundation/lams/gradebook/web/action/GradebookMonitoringAction.java 19 Feb 2015 15:00:47 -0000 1.19 +++ lams_gradebook/src/java/org/lamsfoundation/lams/gradebook/web/action/GradebookMonitoringAction.java 8 Sep 2015 23:42:10 -0000 1.20 @@ -46,6 +46,7 @@ import org.lamsfoundation.lams.lesson.service.ILessonService; import org.lamsfoundation.lams.security.ISecurityService; import org.lamsfoundation.lams.usermanagement.Organisation; +import org.lamsfoundation.lams.usermanagement.Role; import org.lamsfoundation.lams.usermanagement.User; import org.lamsfoundation.lams.usermanagement.dto.UserDTO; import org.lamsfoundation.lams.usermanagement.service.IUserManagementService; @@ -129,7 +130,8 @@ GradebookMonitoringAction.log.error("User missing from session. "); return mapping.findForward("error"); } - if (!getSecurityService().isGroupMonitor(organisationID, user.getUserID(), "get course monitor", false)) { + if (!getSecurityService().hasOrgRole(organisationID, user.getUserID(), + new String[] { Role.GROUP_MANAGER, Role.GROUP_ADMIN }, "get course gradebook page", false)) { response.sendError(HttpServletResponse.SC_FORBIDDEN, "User is not a monitor in the organisation"); return null; }