Index: lams_gradebook/src/java/org/lamsfoundation/lams/gradebook/service/GradebookService.java =================================================================== RCS file: /usr/local/cvsroot/lams_gradebook/src/java/org/lamsfoundation/lams/gradebook/service/GradebookService.java,v diff -u -r1.14 -r1.15 --- lams_gradebook/src/java/org/lamsfoundation/lams/gradebook/service/GradebookService.java 10 Aug 2011 18:08:10 -0000 1.14 +++ lams_gradebook/src/java/org/lamsfoundation/lams/gradebook/service/GradebookService.java 12 Aug 2011 12:30:56 -0000 1.15 @@ -50,6 +50,7 @@ import org.lamsfoundation.lams.gradebook.dto.GBUserGridRowDTO; import org.lamsfoundation.lams.gradebook.dto.GradebookGridRowDTO; import org.lamsfoundation.lams.gradebook.util.GBGridView; +import org.lamsfoundation.lams.gradebook.util.LessonComparator; import org.lamsfoundation.lams.learningdesign.Activity; import org.lamsfoundation.lams.learningdesign.ActivityEvaluation; import org.lamsfoundation.lams.learningdesign.CompetenceMapping; @@ -645,7 +646,7 @@ ExcelCell[] lessonAverageTimeTaken = new ExcelCell[2]; lessonAverageTimeTaken[0] = new ExcelCell(getMessage("gradebook.export.average.lesson.time.taken"), true); - lessonAverageTimeTaken[1] = new ExcelCell(gradebookDAO.getAverageDurationLesson(lesson.getLessonId()), false); + lessonAverageTimeTaken[1] = new ExcelCell(gradebookDAO.getAverageDurationLesson(lesson.getLessonId())/1000, false); rowList.add(lessonAverageTimeTaken); rowList.add(EMPTY_ROW); // ------------------------------------------------------------------ @@ -722,7 +723,8 @@ List rowList = new LinkedList(); User user = (User) getUserService().findById(User.class, userId); - List lessonsFromDB = lessonService.getLessonsByGroupAndUser(userId, organisationId); + Set lessonsFromDB = new TreeSet(new LessonComparator()); + lessonsFromDB.addAll(lessonService.getLessonsByGroupAndUser(userId, organisationId)); List lessons = new LinkedList(); // Dont include lesson in list if the user doesnt have permission @@ -736,7 +738,7 @@ } if (lessons == null || (lessons.size() == 0)) { - return null; + return rowList.toArray(new ExcelCell[][] {}); } //collect users from all lessons Fisheye: Tag 1.1 refers to a dead (removed) revision in file `lams_gradebook/src/java/org/lamsfoundation/lams/gradebook/util/LessonComparator.java'. Fisheye: No comparison available. Pass `N' to diff?