diff options
author | Felix Kehrer <felix.kehrer@gmail.com> | 2018-05-23 12:33:55 +0200 |
---|---|---|
committer | Felix Kehrer <felix.kehrer@gmail.com> | 2018-05-23 12:34:25 +0200 |
commit | b7fb4e11afc3dbc622652f54d96470da102436ea (patch) | |
tree | a90652143704accf9a979625a4e1fbfa85b9a42a /src | |
parent | 37f18628822e9c2281c4627e71981b2131f8c33c (diff) | |
download | sepm-groupproject-b7fb4e11afc3dbc622652f54d96470da102436ea.tar.gz sepm-groupproject-b7fb4e11afc3dbc622652f54d96470da102436ea.tar.xz sepm-groupproject-b7fb4e11afc3dbc622652f54d96470da102436ea.zip |
Fix employees being added twice instead of removed #25963
Diffstat (limited to 'src')
-rw-r--r-- | src/main/java/at/ac/tuwien/sepm/assignment/groupphase/einsatzverwaltung/controller/RegistrationWindowController.java | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/src/main/java/at/ac/tuwien/sepm/assignment/groupphase/einsatzverwaltung/controller/RegistrationWindowController.java b/src/main/java/at/ac/tuwien/sepm/assignment/groupphase/einsatzverwaltung/controller/RegistrationWindowController.java index fa79e27..7e533de 100644 --- a/src/main/java/at/ac/tuwien/sepm/assignment/groupphase/einsatzverwaltung/controller/RegistrationWindowController.java +++ b/src/main/java/at/ac/tuwien/sepm/assignment/groupphase/einsatzverwaltung/controller/RegistrationWindowController.java @@ -120,10 +120,15 @@ public class RegistrationWindowController { tvEmployees.setOnMousePressed( mouseEvent -> { if (mouseEvent.isPrimaryButtonDown() && mouseEvent.getClickCount() == 2) { - chosenEmployees.add(tvEmployees.getSelectionModel().getSelectedItem()); - if (chosenEmployees == null) { + Employee selection = tvEmployees.getSelectionModel().getSelectedItem(); + if (selection == null) { return; + } else if (chosenEmployees.contains(selection)) { + chosenEmployees.remove(selection); + } else { + chosenEmployees.add(selection); } + StringBuilder text = new StringBuilder(); for (Employee employee : chosenEmployees) { text.append(employee.name()).append("\n"); |