-
Notifications
You must be signed in to change notification settings - Fork 5
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Implement ContactList to encapsulate Contact related objects and methods #228
Implement ContactList to encapsulate Contact related objects and methods #228
Conversation
…into branch-contact-list
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The implementation is good, consistent, and it is also similar to the current ModuleList. Aside from conflict with ModuleListTest class, looks good for merging 😃
|
||
contactList.setContact(target, editedContact); | ||
} | ||
|
||
//=========== Filtered Module List Accessors ============================================================= |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we can create sections for the accessor methods for each type of list? Since there are going to be todolist and possibly schedule list also. Just to make things more organized 😃
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes this will be done!
setContact(newData.getContactList()); | ||
} | ||
|
||
//// person-level operations |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe can change "person" to "contact"?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes will change it.
Fixes #225
This PR implements a ContactList class to encapsulate Contact related objects and methods. Additionally, this new ContactList object has been added as a field of the ModelManager class which will be responsible for managing the ContactList class and all methods that relate to ContactList.
Junit tests that test ModelManager have been temporarily commented out.