Fixed sorting issue with lessons, in particular the challenges need to be ordered so the intro is displayed first

This commit is contained in:
Nanne Baars 2017-04-16 05:57:40 +02:00
parent 3ccfcac8ff
commit 7054c44c40
3 changed files with 5 additions and 3 deletions

View File

@ -36,13 +36,14 @@ import org.owasp.webgoat.lessons.LessonMenuItemType;
import org.owasp.webgoat.session.Course; import org.owasp.webgoat.session.Course;
import org.owasp.webgoat.session.LessonTracker; import org.owasp.webgoat.session.LessonTracker;
import org.owasp.webgoat.session.UserTracker; import org.owasp.webgoat.session.UserTracker;
import org.owasp.webgoat.session.WebSession;
import org.springframework.stereotype.Controller; import org.springframework.stereotype.Controller;
import org.springframework.web.bind.annotation.RequestMapping; import org.springframework.web.bind.annotation.RequestMapping;
import org.springframework.web.bind.annotation.ResponseBody; import org.springframework.web.bind.annotation.ResponseBody;
import java.util.ArrayList; import java.util.ArrayList;
import java.util.Comparator;
import java.util.List; import java.util.List;
import java.util.stream.Collectors;
/** /**
* <p>LessonMenuService class.</p> * <p>LessonMenuService class.</p>
@ -75,6 +76,7 @@ public class LessonMenuService {
categoryItem.setType(LessonMenuItemType.CATEGORY); categoryItem.setType(LessonMenuItemType.CATEGORY);
// check for any lessons for this category // check for any lessons for this category
List<AbstractLesson> lessons = course.getLessons(category); List<AbstractLesson> lessons = course.getLessons(category);
lessons = lessons.stream().sorted(Comparator.comparing(l -> l.getTitle())).collect(Collectors.toList());
for (AbstractLesson lesson : lessons) { for (AbstractLesson lesson : lessons) {
LessonMenuItem lessonItem = new LessonMenuItem(); LessonMenuItem lessonItem = new LessonMenuItem();
lessonItem.setName(lesson.getTitle()); lessonItem.setName(lesson.getTitle());

View File

@ -29,7 +29,7 @@ public class ChallengeIntro extends NewLesson {
@Override @Override
public String getTitle() { public String getTitle() {
return "challenge.title"; return "challenge0.title";
} }
@Override @Override

View File

@ -1,4 +1,4 @@
challenge.title=WebGoat Challenge challenge0.title=WebGoat Challenge
challenge1.title=Admin lost password challenge1.title=Admin lost password
challenge2.title=Get it for free challenge2.title=Get it for free
challenge3.title=Photo comments challenge3.title=Photo comments