Index: lams_monitoring/src/java/org/lamsfoundation/lams/monitoring/monitoringApplicationContext.xml =================================================================== diff -u -ra7766445094c16b343aa56dc1f73fd3051cef1db -r3438d5b8ee54bba2e782f62d77db620df3e0faef --- lams_monitoring/src/java/org/lamsfoundation/lams/monitoring/monitoringApplicationContext.xml (.../monitoringApplicationContext.xml) (revision a7766445094c16b343aa56dc1f73fd3051cef1db) +++ lams_monitoring/src/java/org/lamsfoundation/lams/monitoring/monitoringApplicationContext.xml (.../monitoringApplicationContext.xml) (revision 3438d5b8ee54bba2e782f62d77db620df3e0faef) @@ -43,6 +43,7 @@ + Index: lams_monitoring/src/java/org/lamsfoundation/lams/monitoring/service/MonitoringService.java =================================================================== diff -u -rabe8b6d5222e4efc0e5bb332066d481f50d38e8c -r3438d5b8ee54bba2e782f62d77db620df3e0faef --- lams_monitoring/src/java/org/lamsfoundation/lams/monitoring/service/MonitoringService.java (.../MonitoringService.java) (revision abe8b6d5222e4efc0e5bb332066d481f50d38e8c) +++ lams_monitoring/src/java/org/lamsfoundation/lams/monitoring/service/MonitoringService.java (.../MonitoringService.java) (revision 3438d5b8ee54bba2e782f62d77db620df3e0faef) @@ -61,6 +61,8 @@ import org.lamsfoundation.lams.lesson.LessonClass; import org.lamsfoundation.lams.lesson.dao.ILessonClassDAO; import org.lamsfoundation.lams.lesson.dao.ILessonDAO; +import org.lamsfoundation.lams.lesson.dto.LessonDetailsDTO; +import org.lamsfoundation.lams.lesson.service.ILessonService; import org.lamsfoundation.lams.monitoring.MonitoringConstants; import org.lamsfoundation.lams.tool.ToolSession; import org.lamsfoundation.lams.tool.exception.DataMissingException; @@ -125,6 +127,7 @@ private IGroupingDAO groupingDAO; private IAuthoringService authoringService; private ILearnerService learnerService; + private ILessonService lessonService; private ILamsCoreToolService lamsCoreToolService; private IUserManagementService userManagementService; private Scheduler scheduler; @@ -176,6 +179,13 @@ public void setLearnerService(ILearnerService learnerService) { this.learnerService = learnerService; } + /** + * + * @param lessonService + */ + public void setLessonService(ILessonService lessonService) { + this.lessonService = lessonService; + } /** * @param authoringService The authoringService to set. */ @@ -767,7 +777,7 @@ learnerService.performGrouping(lessonId, groupActivity); // grouping = groupActivity.getCreateGrouping(); // myGroup = grouping.getGroupBy(learner); - learnerService.completeActivity(learner,activity,newLesson); + learnerService.completeActivity(learner.getUserId(),activity,newLesson); log.debug("Grouping activity [" + activity.getActivityId() + "] is completed."); }else{ //except random grouping, stop here @@ -776,14 +786,14 @@ } }else{ //if group already exist - learnerService.completeActivity(learner,activity,newLesson); + learnerService.completeActivity(learner.getUserId(),activity,newLesson); log.debug("Grouping activity [" + activity.getActivityId() + "] is completed."); } }else if ( activity.isGateActivity() ) { GateActivity gate = (GateActivity) activity; if(learnerService.knockGate(lessonId,gate,learner)){ //the gate is opened, continue to next activity to complete - learnerService.completeActivity(learner,activity,newLesson); + learnerService.completeActivity(learner.getUserId(),activity,newLesson); log.debug("Gate activity [" + gate.getActivityId() + "] is completed."); }else{ //the gate is closed, stop here @@ -796,7 +806,7 @@ ToolActivity toolActivity = (ToolActivity) activity; try { ToolSession toolSession = lamsCoreToolService.getToolSessionByActivity(learner,toolActivity); - learnerService.completeToolSession(toolSession.getToolSessionId(),new Long(learnerId.intValue())); + learnerService.completeToolSession(toolSession.getToolSessionId(),new Long(learnerId.longValue())); log.debug("Tool activity [" + activity.getActivityId() + "] is completed."); } catch (LamsToolServiceException e) { throw new MonitoringServiceException(e); @@ -839,10 +849,10 @@ * @see org.lamsfoundation.lams.monitoring.service.IMonitoringService#getLessonDetails(java.lang.Long) */ public String getLessonDetails(Long lessonID)throws IOException{ - Lesson lesson = lessonDAO.getLesson(lessonID); + LessonDetailsDTO dto = lessonService.getLessonDetails(lessonID); FlashMessage flashMessage; - if(lesson!=null){ - flashMessage = new FlashMessage("getLessonDetails",lesson.getLessonDetails()); + if(dto!=null){ + flashMessage = new FlashMessage("getLessonDetails",dto); }else flashMessage = new FlashMessage("getLessonDetails", messageService.getMessage("NO.SUCH.LESSON",new Object[]{lessonID}),