Index: lams_monitoring/src/java/org/lamsfoundation/lams/monitoring/monitoringApplicationContext.xml =================================================================== diff -u -r1b3256ab2070e6b8b91c1c5df04b5ad0e52635a2 -r2592aad04b8de7be2f430cb66273f9bf78f75b11 --- lams_monitoring/src/java/org/lamsfoundation/lams/monitoring/monitoringApplicationContext.xml (.../monitoringApplicationContext.xml) (revision 1b3256ab2070e6b8b91c1c5df04b5ad0e52635a2) +++ lams_monitoring/src/java/org/lamsfoundation/lams/monitoring/monitoringApplicationContext.xml (.../monitoringApplicationContext.xml) (revision 2592aad04b8de7be2f430cb66273f9bf78f75b11) @@ -76,6 +76,7 @@ PROPAGATION_REQUIRED PROPAGATION_REQUIRED,readOnly PROPAGATION_REQUIRED + PROPAGATION_REQUIRED Index: lams_monitoring/src/java/org/lamsfoundation/lams/monitoring/service/IMonitoringService.java =================================================================== diff -u -rd0b6f213cba1026b0c9fdbdaa5dd44a49eddd3aa -r2592aad04b8de7be2f430cb66273f9bf78f75b11 --- lams_monitoring/src/java/org/lamsfoundation/lams/monitoring/service/IMonitoringService.java (.../IMonitoringService.java) (revision d0b6f213cba1026b0c9fdbdaa5dd44a49eddd3aa) +++ lams_monitoring/src/java/org/lamsfoundation/lams/monitoring/service/IMonitoringService.java (.../IMonitoringService.java) (revision 2592aad04b8de7be2f430cb66273f9bf78f75b11) @@ -264,7 +264,7 @@ * @return new value for learnerPresenceAvailable. Normally will be same as input parameter, will only be different * if the value cannot be updated for some reason. */ - Boolean setPresenceAvailable(long lessonId, Integer userId, Boolean learnerPresenceAvailable); + Boolean togglePresenceAvailable(long lessonId, Integer userId, Boolean learnerPresenceAvailable); /** * Set whether or not the learner IM button is available in monitor. Sets it to FALSE if learnerExportAvailable is @@ -276,7 +276,7 @@ * @return new value for learnerPresenceImAvailable. Normally will be same as input parameter, will only be * different if the value cannot be updated for some reason. */ - Boolean setPresenceImAvailable(long lessonId, Integer userId, Boolean learnerPresenceImAvailable); + Boolean togglePresenceImAvailable(long lessonId, Integer userId, Boolean learnerPresenceImAvailable); /** * Set whether or not the live edit is available in monitor. Sets it to FALSE if learnerExportAvailable is null. @@ -288,7 +288,7 @@ * @return new value for liveEditEnabled. Normally will be same as input parameter, will only be different if the * value cannot be updated for some reason. */ - Boolean setLiveEditEnabled(long lessonId, Integer userId, Boolean liveEditEnabled); + Boolean toggleLiveEditEnabled(long lessonId, Integer userId, Boolean liveEditEnabled); String forceCompleteActivitiesByUser(Integer learnerId, Integer requesterId, long lessonId, Long activityId, boolean removeLearnerContent); Index: lams_monitoring/src/java/org/lamsfoundation/lams/monitoring/service/MonitoringService.java =================================================================== diff -u -rd0b6f213cba1026b0c9fdbdaa5dd44a49eddd3aa -r2592aad04b8de7be2f430cb66273f9bf78f75b11 --- lams_monitoring/src/java/org/lamsfoundation/lams/monitoring/service/MonitoringService.java (.../MonitoringService.java) (revision d0b6f213cba1026b0c9fdbdaa5dd44a49eddd3aa) +++ lams_monitoring/src/java/org/lamsfoundation/lams/monitoring/service/MonitoringService.java (.../MonitoringService.java) (revision 2592aad04b8de7be2f430cb66273f9bf78f75b11) @@ -1006,7 +1006,7 @@ } @Override - public Boolean setPresenceAvailable(long lessonId, Integer userId, Boolean presenceAvailable) { + public Boolean togglePresenceAvailable(long lessonId, Integer userId, Boolean presenceAvailable) { securityService.isLessonMonitor(lessonId, userId, "set presence available", true); Lesson requestedLesson = lessonDAO.getLesson(new Long(lessonId)); requestedLesson.setLearnerPresenceAvailable(presenceAvailable != null ? presenceAvailable : Boolean.FALSE); @@ -1015,7 +1015,7 @@ } @Override - public Boolean setPresenceImAvailable(long lessonId, Integer userId, Boolean presenceImAvailable) { + public Boolean togglePresenceImAvailable(long lessonId, Integer userId, Boolean presenceImAvailable) { securityService.isLessonMonitor(lessonId, userId, "set presence instant messaging available", true); Lesson requestedLesson = lessonDAO.getLesson(new Long(lessonId)); requestedLesson.setLearnerImAvailable(presenceImAvailable != null ? presenceImAvailable : Boolean.FALSE); @@ -1024,7 +1024,7 @@ } @Override - public Boolean setLiveEditEnabled(long lessonId, Integer userId, Boolean liveEditEnabled) { + public Boolean toggleLiveEditEnabled(long lessonId, Integer userId, Boolean liveEditEnabled) { securityService.isLessonMonitor(lessonId, userId, "set live edit available", true); Lesson requestedLesson = lessonDAO.getLesson(new Long(lessonId)); requestedLesson.setLiveEditEnabled(liveEditEnabled != null ? liveEditEnabled : Boolean.FALSE); Index: lams_monitoring/src/java/org/lamsfoundation/lams/monitoring/web/MonitoringAction.java =================================================================== diff -u -rd0b6f213cba1026b0c9fdbdaa5dd44a49eddd3aa -r2592aad04b8de7be2f430cb66273f9bf78f75b11 --- lams_monitoring/src/java/org/lamsfoundation/lams/monitoring/web/MonitoringAction.java (.../MonitoringAction.java) (revision d0b6f213cba1026b0c9fdbdaa5dd44a49eddd3aa) +++ lams_monitoring/src/java/org/lamsfoundation/lams/monitoring/web/MonitoringAction.java (.../MonitoringAction.java) (revision 2592aad04b8de7be2f430cb66273f9bf78f75b11) @@ -1553,10 +1553,10 @@ Boolean presenceAvailable = WebUtil.readBooleanParam(request, "presenceAvailable", false); try { - getMonitoringService().setPresenceAvailable(lessonID, userID, presenceAvailable); + getMonitoringService().togglePresenceAvailable(lessonID, userID, presenceAvailable); if (!presenceAvailable) { - getMonitoringService().setPresenceImAvailable(lessonID, userID, false); + getMonitoringService().togglePresenceImAvailable(lessonID, userID, false); } } catch (SecurityException e) { response.sendError(HttpServletResponse.SC_FORBIDDEN, "User is not a monitor in the lesson"); @@ -1575,7 +1575,7 @@ Boolean presenceImAvailable = WebUtil.readBooleanParam(request, "presenceImAvailable", false); try { - getMonitoringService().setPresenceImAvailable(lessonID, userID, presenceImAvailable); + getMonitoringService().togglePresenceImAvailable(lessonID, userID, presenceImAvailable); } catch (SecurityException e) { response.sendError(HttpServletResponse.SC_FORBIDDEN, "User is not a monitor in the lesson"); }