From b7fb4e11afc3dbc622652f54d96470da102436ea Mon Sep 17 00:00:00 2001
From: Felix Kehrer <felix.kehrer@gmail.com>
Date: Wed, 23 May 2018 12:33:55 +0200
Subject: Fix employees being added twice instead of removed #25963

---
 .../controller/RegistrationWindowController.java                 | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

(limited to 'src/main/java/at/ac/tuwien')

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");
-- 
cgit v1.2.3-70-g09d2