Revert "Checkstyle errors [2]"

This reverts commit 52210e6.
Jeevz10 · Oct 31, 2018 · bb6bcaa · bb6bcaa
1 parent 52210e6
commit bb6bcaa
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 7 deletions.
diff --git a/src/main/java/seedu/address/logic/commands/ListCommand.java b/src/main/java/seedu/address/logic/commands/ListCommand.java
@@ -32,9 +32,8 @@ public class ListCommand extends Command {
     private final DepartmentContainsKeywordsPredicate predicate;
 
     public ListCommand(String sortByParam, DepartmentContainsKeywordsPredicate predicate) {
-        requireNonNull(sortByParam);
-        //requireNonNull(predicate);
         this.predicate = predicate;
+        requireNonNull(sortByParam);
         listKey = sortByParam;
     }
 
@@ -43,12 +42,12 @@ public CommandResult execute(Model model, CommandHistory history) {
         requireNonNull(model);
 
         switch (listKey) {
-        case (LIST_KEY_DEPARTMENT): model.updateFilteredPersonList(predicate);
-           return new CommandResult(
+            case (LIST_KEY_DEPARTMENT): model.updateFilteredPersonList(predicate);
+               return new CommandResult(
                         String.format(Messages.MESSAGE_PERSONS_LISTED_OVERVIEW, model.getFilteredPersonList().size()));
-        case (LIST_KEY_ALL): model.updateFilteredPersonList(PREDICATE_SHOW_ALL_PERSONS);
-            break;
-        default:
+            case (LIST_KEY_ALL): model.updateFilteredPersonList(PREDICATE_SHOW_ALL_PERSONS);
+                break;
+            default:
         }
 
         return new CommandResult(MESSAGE_SUCCESS);

diff --git a/src/main/java/seedu/address/logic/parser/ListCommandParser.java b/src/main/java/seedu/address/logic/parser/ListCommandParser.java
@@ -39,6 +39,7 @@ public ListCommand parse(String userInput) throws ParseException {
 
     /**
      * Checks if first word of the userInput is equals to "all"
+     * @param userInput
      * @return boolean true if it is equal, if not, false
      */
     private boolean parseAll(String userInput) {
@@ -47,6 +48,7 @@ private boolean parseAll(String userInput) {
 
     /**
      * Checks if first word of the userInput is equals to "dep"
+     * @param userInput
      * @return boolean true if it is equal, if not, false
      */
     private boolean parseDep(String userInput) {
@@ -55,6 +57,7 @@ private boolean parseDep(String userInput) {
 
     /**
      * Splits the userInput by whitespace and places it into a DepartmentContainsKeywordPredicate
+     * @param userInput
      * @return a new predicate
      */
     private DepartmentContainsKeywordsPredicate preparePredicate(String userInput) {