Skip to content

Commit

Permalink
[115] Move event handler registration to child class
Browse files Browse the repository at this point in the history
  • Loading branch information
damithc authored Sep 27, 2016
1 parent 5f6299d commit daf5657
Show file tree
Hide file tree
Showing 3 changed files with 11 additions and 5 deletions.
9 changes: 5 additions & 4 deletions src/main/java/seedu/address/model/ModelManager.java
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,7 @@ public ModelManager(ReadOnlyAddressBook initialData, UserPrefs userPrefs) {
@Override
public void resetData(ReadOnlyAddressBook newData) {
addressBook.resetData(newData);
indicateModelChanged();
indicateAddressBookChanged();
}

@Override
Expand All @@ -63,21 +63,22 @@ public ReadOnlyAddressBook getAddressBook() {
}

/** Raises an event to indicate the model has changed */
private void indicateModelChanged() {
private void indicateAddressBookChanged() {
logger.info("Raising AddressBookChangedEvent");
raise(new AddressBookChangedEvent(addressBook));
}

@Override
public synchronized void deletePerson(ReadOnlyPerson target) throws PersonNotFoundException {
addressBook.removePerson(target);
indicateModelChanged();
indicateAddressBookChanged();
}

@Override
public synchronized void addPerson(Person person) throws UniquePersonList.DuplicatePersonException {
addressBook.addPerson(person);
updateFilteredListToShowAll();
indicateModelChanged();
indicateAddressBookChanged();
}

@Override
Expand Down
5 changes: 5 additions & 0 deletions src/main/java/seedu/address/ui/StatusBarFooter.java
Original file line number Diff line number Diff line change
Expand Up @@ -7,18 +7,22 @@
import javafx.stage.Stage;

import java.util.Date;
import java.util.logging.Logger;

import org.controlsfx.control.StatusBar;

import com.google.common.eventbus.Subscribe;

import seedu.address.commons.core.EventsCenter;
import seedu.address.commons.core.LogsCenter;
import seedu.address.commons.events.model.AddressBookChangedEvent;
import seedu.address.commons.util.FxViewUtil;

/**
* A controller for the status bar that is displayed at the footer of the application.
*/
public class StatusBarFooter extends UiPart {
private static final Logger logger = LogsCenter.getLogger(StatusBarFooter.class);
private StatusBar syncStatus;
private StatusBar saveLocationStatus;

Expand Down Expand Up @@ -46,6 +50,7 @@ public void configure(String saveLocation) {
setSyncStatus("Not updated yet in this session");
addSaveLocation();
setSaveLocation("./" + saveLocation);
EventsCenter.getInstance().registerHandler(this);
}

private void addMainPane() {
Expand Down
2 changes: 1 addition & 1 deletion src/main/java/seedu/address/ui/UiPart.java
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ public abstract class UiPart {
Stage primaryStage;

public UiPart(){
EventsCenter.getInstance().registerHandler(this);

}

/**
Expand Down

0 comments on commit daf5657

Please sign in to comment.