diff --git a/src/main/java/seedu/address/logic/commands/RemarkCommand.java b/src/main/java/seedu/address/logic/commands/RemarkCommand.java index 69a4905937a..bccf8004d76 100644 --- a/src/main/java/seedu/address/logic/commands/RemarkCommand.java +++ b/src/main/java/seedu/address/logic/commands/RemarkCommand.java @@ -3,14 +3,15 @@ import static seedu.address.commons.util.CollectionUtil.requireAllNonNull; import static seedu.address.model.Model.PREDICATE_SHOW_ALL_PERSONS; +import java.util.List; + import seedu.address.commons.core.index.Index; -import seedu.address.logic.commands.exceptions.CommandException; import seedu.address.logic.Messages; +import seedu.address.logic.commands.exceptions.CommandException; import seedu.address.model.Model; import seedu.address.model.person.Person; import seedu.address.model.person.Remark; -import java.util.List; /** * Changes the remark of an existing person in the address book. diff --git a/src/main/java/seedu/address/logic/parser/FindCommandParser.java b/src/main/java/seedu/address/logic/parser/FindCommandParser.java index fae171ecb04..3fa53a0f1ac 100644 --- a/src/main/java/seedu/address/logic/parser/FindCommandParser.java +++ b/src/main/java/seedu/address/logic/parser/FindCommandParser.java @@ -3,6 +3,7 @@ import static seedu.address.logic.Messages.MESSAGE_INVALID_COMMAND_FORMAT; import java.util.Arrays; + import seedu.address.logic.commands.FindCommand; import seedu.address.logic.parser.exceptions.ParseException; import seedu.address.model.person.NameContainsKeywordsPredicate; diff --git a/src/main/java/seedu/address/logic/parser/RemarkCommandParser.java b/src/main/java/seedu/address/logic/parser/RemarkCommandParser.java index 4986ab94f5f..b3d6e17e70f 100644 --- a/src/main/java/seedu/address/logic/parser/RemarkCommandParser.java +++ b/src/main/java/seedu/address/logic/parser/RemarkCommandParser.java @@ -2,16 +2,25 @@ import static java.util.Objects.requireNonNull; import static seedu.address.logic.Messages.MESSAGE_INVALID_COMMAND_FORMAT; +import static seedu.address.logic.parser.CliSyntax.PREFIX_REMARK; import seedu.address.commons.core.index.Index; import seedu.address.commons.exceptions.IllegalValueException; import seedu.address.logic.commands.RemarkCommand; -import static seedu.address.logic.parser.CliSyntax.PREFIX_REMARK; import seedu.address.logic.parser.exceptions.ParseException; import seedu.address.model.person.Remark; -public class RemarkCommandParser implements Parser{ - +/** + * Parses user input for the {@code remark} command. + */ +public class RemarkCommandParser implements Parser { + /** + * Parses the given {@code String} of arguments into a {@code RemarkCommand}. + * + * @param args The String of arguments. + * @return the RemarkCommand object. + * @throws ParseException If the user input does not conform to the expected format. + */ public RemarkCommand parse(String args) throws ParseException { requireNonNull(args); ArgumentMultimap argMultimap = ArgumentTokenizer.tokenize(args, PREFIX_REMARK); @@ -23,9 +32,7 @@ public RemarkCommand parse(String args) throws ParseException { throw new ParseException(String.format(MESSAGE_INVALID_COMMAND_FORMAT, RemarkCommand.MESSAGE_USAGE), ive); } - Remark remark = new Remark(argMultimap.getValue(PREFIX_REMARK).orElse("")); - return new RemarkCommand(index, remark); } diff --git a/src/main/java/seedu/address/ui/PersonCard.java b/src/main/java/seedu/address/ui/PersonCard.java index 21415b51685..dfac506daf8 100644 --- a/src/main/java/seedu/address/ui/PersonCard.java +++ b/src/main/java/seedu/address/ui/PersonCard.java @@ -14,11 +14,10 @@ */ public class PersonCard extends UiPart { + private static final String FXML = "PersonListCard.fxml"; + public final Person person; @FXML private Label remark; - - private static final String FXML = "PersonListCard.fxml"; - /** * Note: Certain keywords such as "location" and "resources" are reserved keywords in JavaFX. * As a consequence, UI elements' variable names cannot be set to such keywords @@ -26,9 +25,6 @@ public class PersonCard extends UiPart { * * @see The issue on AddressBook level 4 */ - - public final Person person; - @FXML private HBox cardPane; @FXML