diff --git a/src/main/java/seedu/address/logic/commands/EditCommand.java b/src/main/java/seedu/address/logic/commands/EditCommand.java index e59a3cdf94c..4f38d10ebd3 100644 --- a/src/main/java/seedu/address/logic/commands/EditCommand.java +++ b/src/main/java/seedu/address/logic/commands/EditCommand.java @@ -154,17 +154,20 @@ public static class EditPersonDescriptor { private Email email; private Address address; + /** + * Constructs an empty EditPersonDescriptor instance. + */ public EditPersonDescriptor() {} /** - * Copy constructor. + * Constructs an EditPersonDescriptor instance with the given {@code descriptionToCopy}. */ - public EditPersonDescriptor(EditPersonDescriptor toCopy) { - requireNonNull(toCopy); - setName(toCopy.name); - setPhone(toCopy.phone); - setEmail(toCopy.email); - setAddress(toCopy.address); + public EditPersonDescriptor(EditPersonDescriptor descriptionToCopy) { + requireNonNull(descriptionToCopy); + setName(descriptionToCopy.name); + setPhone(descriptionToCopy.phone); + setEmail(descriptionToCopy.email); + setAddress(descriptionToCopy.address); } /** diff --git a/src/main/java/seedu/address/logic/commands/HelpCommand.java b/src/main/java/seedu/address/logic/commands/HelpCommand.java index 78122b1c7f5..cc8b4f98c7e 100644 --- a/src/main/java/seedu/address/logic/commands/HelpCommand.java +++ b/src/main/java/seedu/address/logic/commands/HelpCommand.java @@ -3,7 +3,7 @@ import seedu.address.model.Model; /** - * Format full help instructions for every command for display. + * Formats full help instructions for every command for display. */ public class HelpCommand extends Command { diff --git a/src/main/java/seedu/address/logic/parser/EditCommandParser.java b/src/main/java/seedu/address/logic/parser/EditCommandParser.java index 58725152b97..b288406d744 100644 --- a/src/main/java/seedu/address/logic/parser/EditCommandParser.java +++ b/src/main/java/seedu/address/logic/parser/EditCommandParser.java @@ -92,7 +92,7 @@ private EditPersonDescriptor getEditPersonDescriptor(ArgumentMultimap argMultima } /** - * Check if {@code prefixes} are present in the {@code argMultimap}. + * Checks if {@code prefixes} are present in the {@code argMultimap}. * * @param argMultimap that we are performing search on. * @param prefixes to search for. diff --git a/src/main/java/seedu/address/model/person/Person.java b/src/main/java/seedu/address/model/person/Person.java index 9e7421fed87..7a5ce80e972 100644 --- a/src/main/java/seedu/address/model/person/Person.java +++ b/src/main/java/seedu/address/model/person/Person.java @@ -37,7 +37,6 @@ public class Person { /** * Every field must be present and not null. */ - public Person(Name name, Phone phone, Email email, Address address, Remark remark, Set tags, Set insurances, Appointment appointment, AppointmentCount count) { @@ -59,7 +58,6 @@ public Person(Name name, Phone phone, Email email, /** * Every field must be present and not null. */ - public Person(Name name, Phone phone, Email email, Address address, Remark remark, Set tags, Set insurances, Appointment appointment, AppointmentCount count, Priority priority) { @@ -195,7 +193,7 @@ public static Person createPersonWithUpdatedInsurances(Person source, Collection } /** - * Creates and returns a {@code Person} with details of {@code source}, assigning priority of + * Creates and returns a {@code Person} with details of {@code personToUpdate}, assigning priority of * {@code newPriority}. */ public static Person createPersonWithUpdatedPriority(Person personToUpdate, Priority newPriority) { diff --git a/src/main/java/seedu/address/model/priority/Priority.java b/src/main/java/seedu/address/model/priority/Priority.java index 9698c092a45..e2359e8e7f8 100644 --- a/src/main/java/seedu/address/model/priority/Priority.java +++ b/src/main/java/seedu/address/model/priority/Priority.java @@ -46,7 +46,7 @@ public class Priority { + HIGH_PRIORITY_KEYWORD + ", " + MEDIUM_PRIORITY_KEYWORD + ", " + LOW_PRIORITY_KEYWORD + ", " + NONE_PRIORITY_KEYWORD + " }"; - private Level level; + private final Level level; /** * Constructs a {@code Priority}. @@ -79,7 +79,7 @@ public static Level parsePriorityLevel(String priority) { } /** - * Checks if the priority given is valid. + * Returns true if the {@code priority} given is valid. * * @param priority to check. */ @@ -87,9 +87,6 @@ public static boolean isValidPriority(String priority) { return VALID_PRIORITY_KEYWORDS.contains(priority); } - /** - * Returns priority level. - */ public Level getPriorityLevel() { return this.level; }