Merge pull request #221 from chrisjohntan/fix-cattag-ui

Fix cattag not updating person list in the UI

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

@@ -51,6 +51,7 @@ public CommandResult execute(Model model) throws CommandException {
             throw new CommandException(String.format(MESSAGE_DUPLICATE_CATEGORY, targetTag, updatedCategory));
         }
         model.setTagsCategory(targetTag, updatedCategory);
+        model.refreshCampusConnect();
         return new CommandResult(String.format(MESSAGE_CAT_TAG_SUCCESS, targetTag));
     }
 

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

@@ -56,6 +56,11 @@ public interface Model {
     /** Returns the CampusConnect */
     ReadOnlyCampusConnect getCampusConnect();
 
+    /**
+     * Refreshes the current data in CampusConnect.
+     */
+    void refreshCampusConnect();
+
     /**
      * Returns true if a person with the same identity as {@code person} exists in the address book.
      */

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

@@ -97,6 +97,11 @@ public ReadOnlyCampusConnect getCampusConnect() {
         return campusConnect;
     }
 
+    @Override
+    public void refreshCampusConnect() {
+        this.setCampusConnect(campusConnect);
+    }
+
     @Override
     public boolean hasPerson(Person person) {
         requireNonNull(person);

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

@@ -136,6 +136,11 @@ public ReadOnlyCampusConnect getCampusConnect() {
             throw new AssertionError("This method should not be called.");
         }
 
+        @Override
+        public void refreshCampusConnect() {
+            throw new AssertionError("This method should not be called.");
+        }
+
         @Override
         public boolean hasPerson(Person person) {
             throw new AssertionError("This method should not be called.");