-
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
Changed all instances of person to record #58
Conversation
} | ||
|
||
/** | ||
* Creates and returns a {@code Person} with the details of {@code personToEdit} | ||
* edited with {@code editPersonDescriptor}. | ||
* Creates and returns a {@code Record} with the details of {@code RecordToEdit} |
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.
Should be recordToEdit
instead of RecordToEdit
*/ | ||
public void setPersons(List<Person> persons) { | ||
this.persons.setPersons(persons); | ||
public void setRecords(List<Record> people) { |
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.
Should people
be records
instead?
@@ -26,8 +26,8 @@ | |||
|
|||
private final VersionedAddressBook versionedAddressBook; | |||
private final UserPrefs userPrefs; | |||
private final FilteredList<Person> filteredPersons; | |||
private final SimpleObjectProperty<Person> selectedPerson = new SimpleObjectProperty<>(); | |||
private final FilteredList<Record> filteredPeople; |
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.
Should name filteredRecords
instead of filteredPeople
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.
Good job changing all instances of Person to Record including those in test cases, comments and docs
As per developerGuide, all instances of
person
are changed torecord
to change the application from an addressBook to a financeTracker.