Index: lams_monitoring/src/java/org/lamsfoundation/lams/monitoring/web/EmailNotificationsAction.java =================================================================== diff -u -r030734e714da9ef57693dc1e5b6c4c769874b28d -rc94832dbca0118ed21134a009f7b8f310e963b44 --- lams_monitoring/src/java/org/lamsfoundation/lams/monitoring/web/EmailNotificationsAction.java (.../EmailNotificationsAction.java) (revision 030734e714da9ef57693dc1e5b6c4c769874b28d) +++ lams_monitoring/src/java/org/lamsfoundation/lams/monitoring/web/EmailNotificationsAction.java (.../EmailNotificationsAction.java) (revision c94832dbca0118ed21134a009f7b8f310e963b44) @@ -52,6 +52,7 @@ import org.lamsfoundation.lams.lesson.Lesson; import org.lamsfoundation.lams.lesson.dao.ILearnerProgressDAO; import org.lamsfoundation.lams.lesson.service.ILessonService; +import org.lamsfoundation.lams.lesson.util.LessonComparator; import org.lamsfoundation.lams.monitoring.MonitoringConstants; import org.lamsfoundation.lams.monitoring.dto.EmailScheduleMessageJobDTO; import org.lamsfoundation.lams.monitoring.service.IMonitoringService; @@ -138,8 +139,16 @@ // getting the organisation Organisation org = (Organisation) learnerService.getUserManagementService().findById(Organisation.class, orgId); - Set lessons = org.getLessons(); + //sort and filter lesson list + Set lessons = new TreeSet(new LessonComparator()); + for (Lesson lesson : (Set)org.getLessons()) { + if (!Lesson.REMOVED_STATE.equals(lesson.getLessonStateId()) + && !Lesson.FINISHED_STATE.equals(lesson.getLessonStateId())) { + lessons.add(lesson); + } + } + Lesson firstLesson = null; Iterator lessonIter = lessons.iterator(); if (lessonIter.hasNext()) {