From 53f3c1ae018ddfd5dd4ea14208f2f7b2cd35a9ab Mon Sep 17 00:00:00 2001 From: sk2001git <122198303+sk2001git@users.noreply.github.com> Date: Tue, 16 Jan 2024 15:20:14 +0800 Subject: [PATCH] Fix Remark storing bug --- .../java/seedu/address/logic/commands/RemarkCommand.java | 9 ++++----- src/main/java/seedu/address/model/person/Person.java | 3 +++ 2 files changed, 7 insertions(+), 5 deletions(-) diff --git a/src/main/java/seedu/address/logic/commands/RemarkCommand.java b/src/main/java/seedu/address/logic/commands/RemarkCommand.java index 6915fca4c92..7deadfcec31 100644 --- a/src/main/java/seedu/address/logic/commands/RemarkCommand.java +++ b/src/main/java/seedu/address/logic/commands/RemarkCommand.java @@ -78,15 +78,14 @@ public CommandResult execute(Model model) throws CommandException { newRemark = RemarkSyntaxHandler.generateKeepRemarkCommand(remark.value, currentRemark); } - Person editedPerson = new Person( - personToEdit.getName(), personToEdit.getPhone(), personToEdit.getEmail(), - personToEdit.getAddress(), newRemark, personToEdit.getTags()); - model.setPerson(personToEdit, editedPerson); + + personToEdit.setRemark(newRemark); + model.setLastViewedPersonIndex(index); model.updateFilteredPersonList(PREDICATE_SHOW_ALL_PERSONS); - return new CommandResult(generateSuccessMessage(editedPerson), true); + return new CommandResult(generateSuccessMessage(personToEdit), true); } /** diff --git a/src/main/java/seedu/address/model/person/Person.java b/src/main/java/seedu/address/model/person/Person.java index c7c3af02dbc..17ed2026082 100644 --- a/src/main/java/seedu/address/model/person/Person.java +++ b/src/main/java/seedu/address/model/person/Person.java @@ -126,6 +126,9 @@ public void setStatus(Status newStatus) { this.currentStatus = newStatus; } + public void setRemark(Remark remark) { + this.remark = remark; + } /**