Merge pull request #269 from ethan-goh/fix-bugs-pe-d

Fix bugs pertaining to PE-D
AY2425S1-CS2103T-F15-2 · Nov 12, 2024 · 973e162 · 973e162
2 parents f30993f + d0c2bfa
commit 973e162
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/src/main/java/seedu/address/logic/commands/ClearCommand.java b/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.setEduContacts(new EduContacts());
-        return new CommandResult(MESSAGE_SUCCESS);
+        model.setPersonToDisplay(null);
+        return new CommandResult(MESSAGE_SUCCESS, true);
     }
 }
diff --git a/src/main/java/seedu/address/logic/parser/EditCommandParser.java b/src/main/java/seedu/address/logic/parser/EditCommandParser.java
@@ -49,7 +49,7 @@ public EditCommand parse(String args) throws ParseException {
                 throw new ParseException(String.format(MESSAGE_INVALID_COMMAND_FORMAT, EditCommand.MESSAGE_USAGE));
             }
 
-            String[] modules = argMultimap.getValue(PREFIX_MODULE).get().split(" ");
+            String[] modules = argMultimap.getValue(PREFIX_MODULE).get().trim().replaceAll("\\s+", " ").split(" ");
             if (modules.length != 2) {
                 throw new ParseException(String.format(MESSAGE_INVALID_COMMAND_FORMAT, EditCommand.MESSAGE_USAGE));
             }