Merge pull request AY2324S1-CS2103T-W10-1#177 from nabonitasen/branch…

…-clear-details

Fix bug in Clear Command

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

@@ -18,6 +18,7 @@ public class ClearCommand extends Command {
     public CommandResult execute(Model model) {
         requireNonNull(model);
         model.setAddressBook(new AddressBook());
+        model.clearPersonDetails();
         return new CommandResult(MESSAGE_SUCCESS);
     }
 }

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

@@ -103,6 +103,11 @@ public interface Model {
      */
     void showPersonAtIndex(Index index);
 
+    /**
+     * CLears details of person displayed in detail view of UI.
+     */
+    void clearPersonDetails();
+
     /** Returns an unmodifiable view of the filtered person list */
     ObservableList<Person> getFilteredPersonList();
 

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

@@ -111,6 +111,11 @@ public void showPersonAtIndex(Index index) {
         }
     }
 
+    @Override
+    public void clearPersonDetails() {
+        currentPerson.set(Optional.empty());
+    }
+
     @Override
     public void deletePerson(Person target) {
         addressBook.removePerson(target);

src/test/java/seedu/address/logic/commands/AddCommandTest.java

@@ -157,6 +157,11 @@ public void showPersonAtIndex(Index index) {
             throw new AssertionError("This method should not be called.");
         }
 
+        @Override
+        public void clearPersonDetails() {
+            throw new AssertionError("This method should not be called.");
+        }
+
         @Override
         public ObservableList<Person> getFilteredPersonList() {
             throw new AssertionError("This method should not be called.");