Merge pull request nus-cs2103-AY2425S1#213 from potaotototo/display-l…

…essons Update display after lesson changes
potaotototo · Nov 6, 2024 · d2bf819 · d2bf819
2 parents 529cff9 + 0dac9ba
commit d2bf819
Show file tree

Hide file tree

Showing 4 changed files with 2 additions and 4 deletions.
diff --git a/src/main/java/tuteez/logic/commands/AddLessonCommand.java b/src/main/java/tuteez/logic/commands/AddLessonCommand.java
@@ -66,6 +66,7 @@ public CommandResult execute(Model model) throws CommandException {
         Person updatedPerson = addLessonsToPerson(personToUpdate, successfulAdditions);
         model.setPerson(personToUpdate, updatedPerson);
         model.updateFilteredPersonList(Model.PREDICATE_SHOW_ALL_PERSONS);
+        model.updateLastViewedPerson(updatedPerson);
 
         String resultMessage = generateResultMessage(personToUpdate, successfulAdditions, failureMessages);
         return new CommandResult(resultMessage);

diff --git a/src/main/java/tuteez/logic/commands/AddRemarkCommand.java b/src/main/java/tuteez/logic/commands/AddRemarkCommand.java
@@ -51,8 +51,6 @@ public CommandResult execute(Model model) throws CommandException {
 
         model.setPerson(personToUpdate, updatedPerson);
         model.updateFilteredPersonList(Model.PREDICATE_SHOW_ALL_PERSONS);
-
-        // Update the last viewed person in the model after updating remarks
         model.updateLastViewedPerson(updatedPerson);
 
         return new CommandResult(String.format("Added remark to %1$s: %2$s",

diff --git a/src/main/java/tuteez/logic/commands/DeleteLessonCommand.java b/src/main/java/tuteez/logic/commands/DeleteLessonCommand.java
@@ -58,6 +58,7 @@ public CommandResult execute(Model model) throws CommandException {
         Person updatedPerson = updatePersonLessons(personToUpdate, currentLessons);
         model.setPerson(personToUpdate, updatedPerson);
         model.updateFilteredPersonList(Model.PREDICATE_SHOW_ALL_PERSONS);
+        model.updateLastViewedPerson(updatedPerson);
 
         return new CommandResult(generateResultMessage(personToUpdate, lessonsToDelete));
     }

diff --git a/src/main/java/tuteez/logic/commands/DeleteRemarkCommand.java b/src/main/java/tuteez/logic/commands/DeleteRemarkCommand.java
@@ -51,8 +51,6 @@ public CommandResult execute(Model model) throws CommandException {
 
         model.setPerson(personToUpdate, updatedPerson);
         model.updateFilteredPersonList(Model.PREDICATE_SHOW_ALL_PERSONS);
-
-        // Update the last viewed person in the model after updating remarks
         model.updateLastViewedPerson(updatedPerson);
 
         return new CommandResult(String.format("Deleted remark at index %1$s from %2$s", remarkIndex.getOneBased(),