coding standard

CS2103JAN2018-T11-B1 · Mar 24, 2018 · 0f3cbd5 · 0f3cbd5
1 parent c52e305
commit 0f3cbd5
Show file tree

Hide file tree

Showing 5 changed files with 1 addition and 14 deletions.
diff --git a/src/main/java/seedu/address/model/AddressBook.java b/src/main/java/seedu/address/model/AddressBook.java
@@ -72,7 +72,6 @@ public void resetData(ReadOnlyAddressBook newData) {
 
         try {
             setPersons(syncedPersonList);
-            setTutees(syncedPersonList);
         } catch (DuplicatePersonException e) {
             throw new AssertionError("AddressBooks should not have duplicate persons");
         }
@@ -135,7 +134,7 @@ private Person syncWithMasterTagList(Person person) {
         personTags.forEach(tag -> correctTagReferences.add(masterTagObjects.get(tag)));
 
         if (person instanceof Tutee) {
-            return new Tutee( person.getName(), person.getPhone(), person.getEmail(), person.getAddress(),
+            return new Tutee(person.getName(), person.getPhone(), person.getEmail(), person.getAddress(),
                     ((Tutee) person).getSubject(), ((Tutee) person).getGrade(),
                     ((Tutee) person).getEducationLevel(),
                     ((Tutee) person).getSchool(), correctTagReferences);

diff --git a/src/main/java/seedu/address/model/Model.java b/src/main/java/seedu/address/model/Model.java
@@ -7,7 +7,6 @@
 import seedu.address.model.person.exceptions.DuplicatePersonException;
 import seedu.address.model.person.exceptions.PersonNotFoundException;
 import seedu.address.model.tag.Tag;
-import seedu.address.model.tutee.Tutee;
 
 /**
  * The API of the Model component.
@@ -47,12 +46,6 @@ void updatePerson(Person target, Person editedPerson)
      */
     void updateFilteredPersonList(Predicate<Person> predicate);
 
-    ///**
-     //* Updates the filter of the filtered tutee list to filter by the given {@code predicate}.
-     //* @throws NullPointerException if {@code predicate} is null.
-     //*/
-    //void updateFilteredTuteeList(String category, String[] keywords);
-
     /**
      * Removes the given {@code tag} from the specified {@code person}.
      */

diff --git a/src/main/java/seedu/address/model/ModelManager.java b/src/main/java/seedu/address/model/ModelManager.java
@@ -16,7 +16,6 @@
 import seedu.address.model.person.exceptions.DuplicatePersonException;
 import seedu.address.model.person.exceptions.PersonNotFoundException;
 import seedu.address.model.tag.Tag;
-import seedu.address.model.tutee.Tutee;
 
 /**
  * Represents the in-memory model of the address book data.
@@ -89,8 +88,6 @@ public void deleteTag(Tag tag, Person person) {
         addressBook.removeTagFromPerson(tag, person);
     }
 
-    //=========== Filtered Person List Accessors =============================================================
-
     /**
      * Returns an unmodifiable view of the list of {@code Person} backed by the internal list of
      * {@code addressBook}

diff --git a/src/main/java/seedu/address/model/person/UniquePersonList.java b/src/main/java/seedu/address/model/person/UniquePersonList.java
@@ -11,7 +11,6 @@
 import seedu.address.commons.util.CollectionUtil;
 import seedu.address.model.person.exceptions.DuplicatePersonException;
 import seedu.address.model.person.exceptions.PersonNotFoundException;
-import seedu.address.model.tutee.Tutee;
 
 /**
  * A list of persons that enforces uniqueness between its elements and does not allow nulls.

diff --git a/src/test/java/seedu/address/storage/XmlAdaptedPersonTest.java b/src/test/java/seedu/address/storage/XmlAdaptedPersonTest.java
@@ -4,7 +4,6 @@
 import static seedu.address.storage.XmlAdaptedPerson.MISSING_FIELD_MESSAGE_FORMAT;
 import static seedu.address.testutil.TypicalPersons.BENSON;
 import static seedu.address.testutil.TypicalTutees.ALICETUTEE;
-import static seedu.address.testutil.TypicalTutees.CARLTUTEE;
 
 import java.util.ArrayList;
 import java.util.List;