Ominaisuus valmis?

Signed-off-by: laurimaaninka <lauri.maaninka@gmail.com>

src/main/java/fi/lpam/gui/Asiakashallinta.java

@@ -141,7 +141,7 @@ public class Asiakashallinta extends TabPohja {
         if (valittuAsiakas != null) {
             Alert alert = new Alert(Alert.AlertType.CONFIRMATION);
             alert.setTitle("Oletko varma?");
-            alert.setHeaderText("Poistetaanko valittu valittu asiakas?");
+            alert.setHeaderText("Poistetaanko valittu asiakas?");
             alert.setContentText(valittuAsiakas.getNimi());
             Optional<ButtonType> result = alert.showAndWait();
             if (result.isPresent() && result.get() == ButtonType.OK) {

src/main/java/fi/lpam/gui/KuljetusListat.java

@@ -9,7 +9,6 @@ import fi.lpam.gui.elementit.MaaraTableColumn;
 import fi.lpam.gui.elementit.TabPohja;
 
 import javafx.beans.property.SimpleBooleanProperty;
-import javafx.beans.value.ObservableValue;
 import javafx.collections.FXCollections;
 import javafx.collections.ObservableList;
 import javafx.geometry.Pos;
@@ -19,7 +18,6 @@ import javafx.scene.control.cell.PropertyValueFactory;
 import javafx.scene.control.cell.TextFieldTableCell;
 import javafx.scene.layout.BorderPane;
 import javafx.scene.layout.HBox;
-import javafx.util.Callback;
 
 import java.time.LocalDate;
 import java.util.ArrayList;
@@ -95,7 +93,7 @@ public class KuljetusListat extends TabPohja {
 
         TableColumn<Kuljetus, Boolean> tcNappi = new TableColumn<>("Kuljeta");
         tcNappi.setMinWidth(100);
-        tcNappi.setCellFactory(_ -> new KuljetaTableCell(asiakkaat, tableView));
+        tcNappi.setCellFactory(_ -> new KuljetaTableCell(asiakkaat));
         tcNappi.setCellValueFactory(features -> new SimpleBooleanProperty(features.getValue() != null));
 
         TableColumn<Kuljetus, String> tcNimi =  new TableColumn<>("Nimi");

src/main/java/fi/lpam/gui/elementit/KuljetaTableCell.java

@@ -3,49 +3,36 @@ package fi.lpam.gui.elementit;
 import fi.lpam.dataluokat.Asiakas;
 import fi.lpam.dataluokat.Kuljetus;
 import fi.lpam.gui.KuljetusListat;
-import javafx.collections.FXCollections;
-import javafx.collections.ObservableList;
 import javafx.geometry.Insets;
 import javafx.scene.control.Button;
 import javafx.scene.control.ContentDisplay;
 import javafx.scene.control.TableCell;
-import javafx.scene.control.TableView;
 import javafx.scene.layout.StackPane;
 
-import java.sql.Array;
-import java.util.ArrayList;
-import java.util.Collections;
 import java.util.HashMap;
 
 public class KuljetaTableCell extends TableCell<Kuljetus, Boolean> {
-    private final Button nappi = new Button("Vaihda");
     private final StackPane paddedNappi = new StackPane();
 
-    public KuljetaTableCell(final HashMap<Integer, Asiakas> asiakkaat, final TableView<Kuljetus> table) {
+    public KuljetaTableCell(HashMap<Integer, Asiakas> asiakkaat) {
         paddedNappi.setPadding(new Insets(3));
+        Button nappi = new Button("Muuta");
         paddedNappi.getChildren().add(nappi);
-        nappi.setOnAction(_ -> {
+        nappi.setOnMouseClicked(_ -> {
+            getTableView().getSelectionModel().select(getIndex());
             Kuljetus kuljetus = getTableRow().getItem();
-            System.out.println(kuljetus);
+
             if (kuljetus == null) return;
             if (kuljetus.kuljetetaan()) {
                 kuljetus.nollaa();
-                System.out.println("Nollattu");
-                System.out.println(kuljetus);
+            } else {
+                kuljetus.palautaMäärät(asiakkaat.get(kuljetus.getAsiakasID()));
             }
-            else {
-                Asiakas asiakas = asiakkaat.get(kuljetus.getAsiakasID());
-                System.out.println(asiakas);
-                kuljetus.palautaMäärät(asiakas);
-                System.out.println("Palautettu");
-            }
-            table.refresh();
+            getTableRow().setItem(new Kuljetus(kuljetus));
             KuljetusListat.tallennusStatus.setText("Tallennettu: Ei");
         });
     }
 
-
-    /** places an add button in the row only if the row is not empty. */
     @Override protected void updateItem(Boolean item, boolean empty) {
         super.updateItem(item, empty);
         if (!empty) {