Merge pull request nus-cs2103-AY2021S1#69 from Nijnxw/branch-v1.2-viewlsn

Fix bug for view lesson command

Author: xinyee20

src/main/java/seedu/address/logic/commands/ViewLsnCommand.java

@@ -46,6 +46,7 @@ private String getMessage(Model model) {
     @Override
     public CommandResult execute(Model model) {
         requireNonNull(model);
+        model.updateFilteredGroupList(grpPredicate);
         model.updateFilteredLessonList(lsnPredicate);
         return new CommandResult(this.getMessage(model));
     }

src/main/java/seedu/address/logic/parser/ViewLsnCommandParser.java

@@ -43,4 +43,5 @@ public ViewLsnCommand parse(String args) throws ParseException {
     private static boolean arePrefixesPresent(ArgumentMultimap argumentMultimap, Prefix... prefixes) {
         return Stream.of(prefixes).allMatch(prefix -> argumentMultimap.getValue(prefix).isPresent());
     }
+
 }

src/main/java/seedu/address/model/ModelManager.java

@@ -58,7 +58,7 @@ public ModelManager(ReadOnlyAddressBook addressBook, ReadOnlyUserPrefs userPrefs
         filteredGroups = new FilteredList<>(this.serenity.getGroupList());
         students = new ArrayObservableList<>(new UniqueStudentList().asUnmodifiableObservableList());
         lessons = new ArrayObservableList<>(new UniqueLessonList().asUnmodifiableObservableList());
-        filteredLessons = new FilteredList<>(new UniqueLessonList().asUnmodifiableObservableList());
+        filteredLessons = new FilteredList<>(lessons);
         studentsInfo = new ArrayObservableList<>(new UniqueStudentInfoList().asUnmodifiableObservableList());
     }
 
@@ -79,7 +79,7 @@ public ModelManager(ReadOnlyAddressBook addressBook, ReadOnlyUserPrefs userPrefs
         filteredGroups = new FilteredList<>(this.serenity.getGroupList());
         students = new ArrayObservableList<>(new UniqueStudentList().asUnmodifiableObservableList());
         lessons = new ArrayObservableList<>(new UniqueLessonList().asUnmodifiableObservableList());
-        filteredLessons = new FilteredList<>(new UniqueLessonList().asUnmodifiableObservableList());
+        filteredLessons = new FilteredList<>(lessons);
         studentsInfo = new ArrayObservableList<>(new UniqueStudentInfoList().asUnmodifiableObservableList());
     }
 
@@ -223,8 +223,8 @@ public void updateFilteredLessonList(Predicate<Lesson> predicate) {
 
     @Override
     public void updateStudentInfoList() {
-        if (!filteredGroups.isEmpty() || !lessons.isEmpty()) {
-            this.studentsInfo.setAll(this.lessons.get(0).getStudentsInfoAsUnmodifiableObservableList());
+        if (!filteredGroups.isEmpty() || !filteredLessons.isEmpty()) {
+            this.studentsInfo.setAll(this.filteredLessons.get(0).getStudentsInfoAsUnmodifiableObservableList());
         }
     }
 
@@ -245,7 +245,7 @@ public ObservableList<Lesson> getLessonList() {
 
     @Override
     public ObservableList<Lesson> getFilteredLessonList() {
-        return lessons;
+        return filteredLessons;
     }
 
     @Override

src/main/java/seedu/address/model/group/Group.java

@@ -143,6 +143,7 @@ public int hashCode() {
 
     @Override
     public String toString() {
-        return name + "," + students.toString();
+        return String.format("Group %s", name);
     }
+
 }

src/main/java/seedu/address/model/group/Lesson.java

@@ -64,5 +64,10 @@ public boolean equals(Object obj) {
         return otherClass.getName().equals(getName())
                 && otherClass.getStudentsInfo().equals(getStudentsInfo());
     }
+
+    @Override
+    public String toString() {
+        return String.format("Lesson %s", name);
+    }
 }