Fix style issues

AY2223S1-CS2103T-T09-4 · Oct 16, 2022 · ec5b6ac · ec5b6ac
1 parent d422cfd
commit ec5b6ac
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 7 deletions.
diff --git a/src/main/java/seedu/address/model/person/Person.java b/src/main/java/seedu/address/model/person/Person.java
@@ -140,11 +140,11 @@ public boolean isSamePerson(Person otherPerson) {
     }
 
     /**
-     * Determines whether the person owes money.
+     * Determines whether the person is owing money.
      *
-     * @return true if the person owes money
+     * @return true if the person is owing money
      */
-    public boolean owesMoney() {
+    public boolean isOwingMoney() {
         return moneyOwed.isGreaterThanZero();
     }
 

diff --git a/src/main/java/seedu/address/ui/ScheduleCard.java b/src/main/java/seedu/address/ui/ScheduleCard.java
@@ -12,7 +12,6 @@
 public class ScheduleCard extends UiPart<Region> {
 
     private static final String FXML = "ScheduleListCard.fxml";
-
     private static final String COLOUR_OF_DEBTOR = "-fx-text-fill: #FF0000;";
 
     /**
@@ -56,7 +55,7 @@ public ScheduleCard(Person person, int displayedIndex) {
      * @param person to check whether he/she is a debtor
      */
     private void setWarningIfOwed(Person person) {
-        if (person.owesMoney()) {
+        if (person.isOwingMoney()) {
             String nameWithAmount = person.getName().fullName + " - To collect $" + person.getMoneyOwed().value;
             name.setText(nameWithAmount);
             name.setStyle(COLOUR_OF_DEBTOR);

diff --git a/src/test/java/seedu/address/model/person/PersonTest.java b/src/test/java/seedu/address/model/person/PersonTest.java
@@ -105,10 +105,10 @@ public void execute_setClassSuccess() {
     @Test
     public void owesMoneyTest() {
         Person debtor = AVA;
-        assertTrue(debtor.owesMoney());
+        assertTrue(debtor.isOwingMoney());
 
         Person nonDebtor = new PersonBuilder(AVA).withMoneyOwed(0).build();
-        assertFalse(nonDebtor.owesMoney());
+        assertFalse(nonDebtor.isOwingMoney());
     }
 
 }