Index: lams_central/src/java/org/lamsfoundation/lams/web/LessonOrderServlet.java =================================================================== diff -u -r97336c15381a6a96013b087b0844f173d1ddedd7 -r63e702b998e24ba405d876e2439864a8f606ba50 --- lams_central/src/java/org/lamsfoundation/lams/web/LessonOrderServlet.java (.../LessonOrderServlet.java) (revision 97336c15381a6a96013b087b0844f173d1ddedd7) +++ lams_central/src/java/org/lamsfoundation/lams/web/LessonOrderServlet.java (.../LessonOrderServlet.java) (revision 63e702b998e24ba405d876e2439864a8f606ba50) @@ -77,7 +77,7 @@ // TODO: remove this validation when the javascript sortable's // containment parameter is working. List idList = Arrays.asList(ids.split(",")); - Set lessons = org.getLessons(); + List lessons = service.findByProperty(Lesson.class, "organisation", org); for (String id : idList) { try { Long l = new Long(Long.parseLong(id)); @@ -98,11 +98,13 @@ } - private boolean contains(Set lessons, Long id) { - Iterator it = lessons.iterator(); - while (it.hasNext()) { - Lesson lesson = (Lesson)it.next(); - if (lesson.getLessonId().equals(id)) return true; + private boolean contains(List lessons, Long id) { + if (lessons != null) { + Iterator it = lessons.iterator(); + while (it.hasNext()) { + Lesson lesson = (Lesson)it.next(); + if (lesson.getLessonId().equals(id)) return true; + } } return false; }