Merge pull request '#8 Kosmeettisia parannuksia kierroshallintaan ja muuallekin' (#9) from feature into master

Reviewed-on: #9

.idea/inspectionProfiles/Project_Default.xml

@@ -2,7 +2,7 @@
   <profile version="1.0">
     <option name="myName" value="Project Default" />
     <inspection_tool class="ExtractMethodRecommender" enabled="true" level="WARNING" enabled_by_default="true">
-      <option name="minLength" value="735" />
+      <option name="minLength" value="951" />
     </inspection_tool>
     <inspection_tool class="GrazieInspection" enabled="false" level="GRAMMAR_ERROR" enabled_by_default="false" />
     <inspection_tool class="LanguageDetectionInspection" enabled="false" level="WARNING" enabled_by_default="false" />

CHANGELOG.md

@@ -1,3 +1,9 @@
+# 3.0.0
+- Lisätty lauantai ja sunnuntai mahdollisiksi kuljetuspäiviksi
+- Mahdollisuus kuljettaa (ja muokata listaa) yli 10 ateriaa per asiakas
+- Versioloki näkyy taas
+- Pieniä viilauksia asiakashallinnan näkymään
+
 # 2.0.2
 - Korjattu virhe kierroksen asiakkaiden hakemisessa, joka aiheutti vääriä järjestyksiä kierroksella
 
@@ -19,4 +25,4 @@
 - Kierroshallinnassa asiakasta raahattaessa placeholderin fiksaus
 
 # 1.0.0
-Ohjelman kieli vaihdettu Pythonista Javaan, sekä tietokanta pickle-tiedostosta SQLite tietokantaan
+- Ohjelman kieli vaihdettu Pythonista Javaan, sekä tietokanta pickle-tiedostosta SQLite tietokantaan

pom.xml

@@ -3,7 +3,7 @@
     <modelVersion>4.0.0</modelVersion>
     <groupId>fi.lpam.ruokamanageri</groupId>
     <artifactId>Ruokamanageri</artifactId>
-    <version>2.0.2</version>
+    <version>3.0.0</version>
     <properties>
         <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
         <maven.compiler.source>25</maven.compiler.source>

src/main/java/fi/lpam/ruokamanageri/dataluokat/Asiakas.java

@@ -3,13 +3,17 @@ package fi.lpam.ruokamanageri.dataluokat;
 import fi.lpam.ruokamanageri.gui.virheDialogit.SiirtoVirhe;
 
 import java.sql.*;
+import java.time.DayOfWeek;
+import java.time.LocalDate;
 import java.util.ArrayList;
+import java.util.HashMap;
+
 @SuppressWarnings("DuplicatedCode")
 public class Asiakas implements TietokantaOlio {
     private int id = -1;
     private String nimi;
     private String osoite;
-    private int maanantaiKpl, tiistaiKpl, keskiviikkoKpl, torstaiKpl, perjantaiKpl;
+    private int maanantaiKpl, tiistaiKpl, keskiviikkoKpl, torstaiKpl, perjantaiKpl, lauantaiKpl, sunnuntaiKpl;
     private int salaatit, pääruoat, jälkiruoat;
     private String erityisruokavalio;
     private String avainTieto;
@@ -40,6 +44,8 @@ public class Asiakas implements TietokantaOlio {
      * @param keskiviikkoKpl kuljetettava määrä
      * @param torstaiKpl kuljetettava määrä
      * @param perjantaiKpl kuljetettava määrä
+     * @param lauantaiKpl kuljetettava määrä
+     * @param sunnuntaiKpl kuljetettava määrä
      * @param salaatit kuljetettava määrä
      * @param pääruoat kuljetettava määrä
      * @param jälkiruoat kuljetettava määrä
@@ -59,6 +65,8 @@ public class Asiakas implements TietokantaOlio {
                     int keskiviikkoKpl,
                     int torstaiKpl,
                     int perjantaiKpl,
+                    int lauantaiKpl,
+                    int sunnuntaiKpl,
                     int salaatit,
                     int pääruoat,
                     int jälkiruoat,
@@ -77,6 +85,8 @@ public class Asiakas implements TietokantaOlio {
         this.keskiviikkoKpl = keskiviikkoKpl;
         this.torstaiKpl = torstaiKpl;
         this.perjantaiKpl = perjantaiKpl;
+        this.lauantaiKpl = lauantaiKpl;
+        this.sunnuntaiKpl = sunnuntaiKpl;
         this.salaatit = salaatit;
         this.pääruoat = pääruoat;
         this.jälkiruoat = jälkiruoat;
@@ -117,18 +127,15 @@ public class Asiakas implements TietokantaOlio {
             sbIDt.append(", ").append(IDt[i]);
         }
         try (Connection tietokanta = Tietokanta.haeYhteys()) {
-            String sql = "select * from asiakkaat where id in (" + sbIDt + ") order by instr('" + sbIDt + "', id)";
+            String sql = "select * from asiakkaat where id in (" + sbIDt + ")";
             Statement stmt = tietokanta.createStatement();
             ResultSet rs = stmt.executeQuery(sql);
-            ArrayList<Asiakas> asiakkaat =  parsiResultSet(rs);
 
-            ArrayList<Asiakas> järjestetyt = new ArrayList<Asiakas>();
+            HashMap<Integer, Asiakas> asiakkaat = new HashMap<>(IDt.length);
+            parsiResultSet(rs).forEach(asiakas -> asiakkaat.put(asiakas.getId(), asiakas));
+            ArrayList<Asiakas> järjestetyt = new ArrayList<>(asiakkaat.size());
             for (int id : IDt) {
-                for (Asiakas asiakas : asiakkaat) {
-                    if (asiakas.getId() == id) {
-                        järjestetyt.add(asiakas);
-                    }
-                }
+                järjestetyt.add(asiakkaat.get(id));
             }
             return järjestetyt;
         }
@@ -149,26 +156,27 @@ public class Asiakas implements TietokantaOlio {
 
         try {
             while (rs.next()) {
-                int i = 1;
                 Asiakas uusi = new Asiakas(
-                        rs.getInt(i++),
-                        rs.getString(i++),
-                        rs.getString(i++),
-                        rs.getInt(i++),
-                        rs.getInt(i++),
-                        rs.getInt(i++),
-                        rs.getInt(i++),
-                        rs.getInt(i++),
-                        rs.getInt(i++),
-                        rs.getInt(i++),
-                        rs.getInt(i++),
-                        rs.getString(i++),
-                        rs.getString(i++),
-                        rs.getString(i++),
-                        rs.getString(i++),
-                        rs.getString(i++),
-                        rs.getString(i++),
-                        rs.getString(i++)
+                        rs.getInt("id"),
+                        rs.getString("nimi"),
+                        rs.getString("osoite"),
+                        rs.getInt("maanantaiKpl"),
+                        rs.getInt("tiistaiKpl"),
+                        rs.getInt("keskiviikkoKpl"),
+                        rs.getInt("torstaiKpl"),
+                        rs.getInt("perjantaiKpl"),
+                        rs.getInt("lauantaiKpl"),
+                        rs.getInt("sunnuntaiKpl"),
+                        rs.getInt("salaatit"),
+                        rs.getInt("pääruoat"),
+                        rs.getInt("jälkiruoat"),
+                        rs.getString("erityisruokavalio"),
+                        rs.getString("avainTieto"),
+                        rs.getString("lisätiedot"),
+                        rs.getString("laskutusOsoite"),
+                        rs.getString("yhteyshenkilönNimi"),
+                        rs.getString("yhteyshenkilönPuhelinnumero"),
+                        rs.getString("yhteyshenkilönSähköposti")
                 );
                 asiakkaat.add(uusi);
             }
@@ -187,7 +195,7 @@ public class Asiakas implements TietokantaOlio {
     @Override
     public boolean tallennaUusi() {
         try (Connection tietokanta = Tietokanta.haeYhteys()) {
-            PreparedStatement stmt = tietokanta.prepareStatement("INSERT INTO asiakkaat (nimi, osoite, maanantaiKpl, tiistaiKpl, keskiviikkoKpl, torstaiKpl, perjantaiKpl, salaatit, pääruoat, jälkiruoat, erityisruokavalio, avainTieto, lisätiedot, laskutusosoite, yhteyshenkilönNimi, yhteyshenkilönPuhelinnumero, yhteyshenkilönSähköposti) VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)", Statement.RETURN_GENERATED_KEYS);
+            PreparedStatement stmt = tietokanta.prepareStatement("INSERT INTO asiakkaat (nimi, osoite, maanantaiKpl, tiistaiKpl, keskiviikkoKpl, torstaiKpl, perjantaiKpl, lauantaiKpl, sunnuntaiKpl, salaatit, pääruoat, jälkiruoat, erityisruokavalio, avainTieto, lisätiedot, laskutusosoite, yhteyshenkilönNimi, yhteyshenkilönPuhelinnumero, yhteyshenkilönSähköposti) VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)", Statement.RETURN_GENERATED_KEYS);
             int i = 1;
             stmt.setString(i++, nimi);
             stmt.setString(i++, osoite);
@@ -196,6 +204,8 @@ public class Asiakas implements TietokantaOlio {
             stmt.setInt(i++, keskiviikkoKpl);
             stmt.setInt(i++, torstaiKpl);
             stmt.setInt(i++, perjantaiKpl);
+            stmt.setInt(i++, lauantaiKpl);
+            stmt.setInt(i++, sunnuntaiKpl);
             stmt.setInt(i++, salaatit);
             stmt.setInt(i++, pääruoat);
             stmt.setInt(i++, jälkiruoat);
@@ -234,7 +244,7 @@ public class Asiakas implements TietokantaOlio {
     @Override
     public boolean päivitäTietokantaan() {
         try (Connection tietokanta = Tietokanta.haeYhteys()) {
-            PreparedStatement stmt = tietokanta.prepareStatement("UPDATE asiakkaat set nimi = ?, osoite = ?, maanantaiKpl = ?, tiistaiKpl = ?, keskiviikkoKpl = ?, torstaiKpl = ?, perjantaiKpl = ?, salaatit = ?, pääruoat = ?, jälkiruoat = ?, erityisruokavalio = ?, avainTieto = ?, lisätiedot = ?, laskutusosoite = ?, yhteyshenkilönNimi = ?, yhteyshenkilönPuhelinnumero = ?, yhteyshenkilönSähköposti = ? WHERE id = ?");
+            PreparedStatement stmt = tietokanta.prepareStatement("UPDATE asiakkaat set nimi = ?, osoite = ?, maanantaiKpl = ?, tiistaiKpl = ?, keskiviikkoKpl = ?, torstaiKpl = ?, perjantaiKpl = ?, lauantaiKpl = ?, sunnuntaiKpl = ?, salaatit = ?, pääruoat = ?, jälkiruoat = ?, erityisruokavalio = ?, avainTieto = ?, lisätiedot = ?, laskutusosoite = ?, yhteyshenkilönNimi = ?, yhteyshenkilönPuhelinnumero = ?, yhteyshenkilönSähköposti = ? WHERE id = ?");
             int i = 1;
             stmt.setString(i++, nimi);
             stmt.setString(i++, osoite);
@@ -243,6 +253,8 @@ public class Asiakas implements TietokantaOlio {
             stmt.setInt(i++, keskiviikkoKpl);
             stmt.setInt(i++, torstaiKpl);
             stmt.setInt(i++, perjantaiKpl);
+            stmt.setInt(i++, lauantaiKpl);
+            stmt.setInt(i++, sunnuntaiKpl);
             stmt.setInt(i++, salaatit);
             stmt.setInt(i++, pääruoat);
             stmt.setInt(i++, jälkiruoat);
@@ -298,6 +310,33 @@ public class Asiakas implements TietokantaOlio {
         return this.getNimi();
     }
 
+    public int getPäivänSalaatit(LocalDate päivä) {
+        int kerroin = getKuljetusKerroin(päivä);
+        return kerroin * getSalaatit();
+    }
+
+    public int getPäivänPääruoat(LocalDate päivä) {
+        int kerroin = getKuljetusKerroin(päivä);
+        return kerroin * getPääruoat();
+    }
+
+    public int getPäivänJälkiruoat(LocalDate päivä) {
+        int kerroin = getKuljetusKerroin(päivä);
+        return kerroin * getJälkiruoat();
+    }
+
+    private int getKuljetusKerroin(LocalDate päivä) {
+        return switch (päivä.getDayOfWeek()) {
+            case DayOfWeek.MONDAY -> getMaanantaiKpl();
+            case DayOfWeek.TUESDAY -> getTiistaiKpl();
+            case DayOfWeek.WEDNESDAY -> getKeskiviikkoKpl();
+            case DayOfWeek.THURSDAY -> getTorstaiKpl();
+            case DayOfWeek.FRIDAY -> getPerjantaiKpl();
+            case DayOfWeek.SATURDAY -> getLauantaiKpl();
+            case DayOfWeek.SUNDAY -> getSunnuntaiKpl();
+        };
+    }
+
     public int getId() {
         return id;
     }
@@ -362,6 +401,22 @@ public class Asiakas implements TietokantaOlio {
         this.perjantaiKpl = perjantaiKpl;
     }
 
+    public int getLauantaiKpl() {
+        return lauantaiKpl;
+    }
+
+    public void setLauantaiKpl(int lauantaiKpl) {
+        this.lauantaiKpl = lauantaiKpl;
+    }
+
+    public int getSunnuntaiKpl() {
+        return sunnuntaiKpl;
+    }
+
+    public void setSunnuntaiKpl(int sunnuntaiKpl) {
+        this.sunnuntaiKpl = sunnuntaiKpl;
+    }
+
     public int getSalaatit() {
         return salaatit;
     }

src/main/java/fi/lpam/ruokamanageri/dataluokat/Kuljetus.java

@@ -36,17 +36,9 @@ public class Kuljetus implements TietokantaOlio {
         this.nimi = asiakas.getNimi();
         this.kuljetusPäivämäärä = pvm;
         this.kuljetusKierros = kuljetusKierros;
-        int kuljetusKerroin = switch (kuljetusPäivämäärä.getDayOfWeek()) {
-            case MONDAY -> asiakas.getMaanantaiKpl();
-            case TUESDAY -> asiakas.getTiistaiKpl();
-            case WEDNESDAY -> asiakas.getKeskiviikkoKpl();
-            case THURSDAY -> asiakas.getTorstaiKpl();
-            case FRIDAY -> asiakas.getPerjantaiKpl();
-            default -> 0;
-        };
-        this.salaatit = kuljetusKerroin * asiakas.getSalaatit();
-        this.pääruoat = kuljetusKerroin * asiakas.getPääruoat();
-        this.jälkiruoat = kuljetusKerroin * asiakas.getJälkiruoat();
+        this.salaatit = asiakas.getPäivänSalaatit(pvm);
+        this.pääruoat = asiakas.getPäivänPääruoat(pvm);
+        this.jälkiruoat = asiakas.getPäivänJälkiruoat(pvm);
         this.osoite = asiakas.getOsoite();
         this.avainTieto = asiakas.getAvainTieto();
         this.lisätieto = asiakas.getLisätiedot();
@@ -226,23 +218,16 @@ public class Kuljetus implements TietokantaOlio {
     }
 
     public void palautaMäärät(Asiakas asiakas) {
-        int kuljetusKerroin = switch (kuljetusPäivämäärä.getDayOfWeek()) {
-            case MONDAY -> asiakas.getMaanantaiKpl();
-            case TUESDAY -> asiakas.getTiistaiKpl();
-            case WEDNESDAY -> asiakas.getKeskiviikkoKpl();
-            case THURSDAY -> asiakas.getTorstaiKpl();
-            case FRIDAY -> asiakas.getPerjantaiKpl();
-            default -> 0;
-        };
-        setSalaatit(asiakas.getSalaatit() * kuljetusKerroin);
-        setPääruoat(asiakas.getPääruoat() * kuljetusKerroin);
-        setJälkiruoat(asiakas.getJälkiruoat() * kuljetusKerroin);
+        setSalaatit(asiakas.getPäivänSalaatit(kuljetusPäivämäärä));
+        setPääruoat(asiakas.getPäivänPääruoat(kuljetusPäivämäärä));
+        setJälkiruoat(asiakas.getPäivänJälkiruoat(kuljetusPäivämäärä));
         setLisätieto(asiakas.getLisätiedot());
     }
 
     public boolean kuljetetaan() {
         return salaatit > 0 || pääruoat > 0 || jälkiruoat > 0 || !lisätieto.isBlank();
     }
+
     public int getId() {
         return id;
     }

src/main/java/fi/lpam/ruokamanageri/dataluokat/Tietokanta.java

@@ -38,11 +38,11 @@ public class Tietokanta {
             handleException(e);
         }
 
-        //Lisää laskutustiedot asiakastauluun
-        String[] sarakkeet = {"laskutusosoite", "yhteyshenkilönNimi", "yhteyshenkilönPuhelinnumero", "yhteyshenkilönSähköposti"};
+        //Lisää kuljetuspäivät asiakassarakkeeseen (v2.0.2->3.x.x)
+        String[] sarakkeet = {"lauantaiKpl", "sunnuntaiKpl"};
         for (String sarake : sarakkeet) {
             try (Connection conn = haeYhteys()) {
-                PreparedStatement stmt = conn.prepareStatement("alter table asiakkaat add column " + sarake + " varchar(255)");
+                PreparedStatement stmt = conn.prepareStatement("alter table asiakkaat add column " + sarake + " int not null default 0");
                 stmt.execute();
             } catch (SQLException e) {
                 if (!e.getMessage().contains("duplicate column name")) {

src/main/java/fi/lpam/ruokamanageri/gui/AsiakasIkkuna.java

@@ -22,7 +22,7 @@ import java.util.Objects;
 public class AsiakasIkkuna extends Stage {
     final Asiakas käsiteltäväAsiakas;
     final TextField tfNimi, tfOsoite, tfRuokarajoitteet, tfAvainTieto, tfLisätiedot, tfYhteyshenkilönNimi, tfLaskutusOsoite, tfYhteyshenkilönPuhelinnumero, tfYhteyshenkilönSähköposti;
-    final IntegerSpinner tfMaanantai, tfTiistai, tfKeskiviikko, tfTorstai, tfPerjantai, tfSalaatit, tfPääruoat, tfJälkiruoat;
+    final IntegerSpinner tfMaanantai, tfTiistai, tfKeskiviikko, tfTorstai, tfPerjantai, tfLauantai, tfSunnuntai, tfSalaatit, tfPääruoat, tfJälkiruoat;
 
     /**
      * Luo ikkunan jossa muokataan saatua asiakasta
@@ -30,7 +30,12 @@ public class AsiakasIkkuna extends Stage {
      */
     public AsiakasIkkuna(Asiakas käsiteltäväAsiakas) {
         this.käsiteltäväAsiakas = käsiteltäväAsiakas;
-        this.setTitle("Asiakasikkuna");
+        if (käsiteltäväAsiakas.getNimi() != null) {
+            this.setTitle(käsiteltäväAsiakas.getNimi());
+        }
+        else {
+            this.setTitle("Uusi asiakas");
+        }
 
         TabPane tabPane = new TabPane();
         tabPane.setSide(Side.LEFT);
@@ -46,22 +51,82 @@ public class AsiakasIkkuna extends Stage {
                 default: break;
             }});
 
-        Tab tabAsiakasTiedot, tabLaskutusTiedot;
-        tabAsiakasTiedot = new Tab("Asiakastiedot");
-        tabLaskutusTiedot = new Tab("Laskutustiedot");
-        tabPane.getTabs().addAll(tabAsiakasTiedot, tabLaskutusTiedot);
+        Tab tabKuljetusTiedot, tabAsiakastiedot;
+        tabKuljetusTiedot = new Tab("Kuljetustiedot");
+        tabAsiakastiedot = new Tab("Asiakastiedot");
+        tabPane.getTabs().addAll(tabKuljetusTiedot, tabAsiakastiedot);
         for (Tab tab : tabPane.getTabs()) {
             tab.setClosable(false);
             tab.setStyle("-fx-font-size: 18px; -fx-font-weight: bold; -fx-font-family: Open Sans");
         }
 
+        GridPane kuljetusTiedot = new GridPane();
+        kuljetusTiedot.setPadding(new Insets(15));
+        kuljetusTiedot.setHgap(10);
+        kuljetusTiedot.setVgap(10);
+        tabKuljetusTiedot.setContent(kuljetusTiedot);
+
+        kuljetusTiedot.addRow(kuljetusTiedot.getRowCount(), new Label("Määrät per kuljetus"));
+
+        Label salaatit = new Label("Salaatit");
+        tfSalaatit = new IntegerSpinner();
+        kuljetusTiedot.addRow(kuljetusTiedot.getRowCount(), salaatit, tfSalaatit);
+
+        Label pääruoat = new Label("Pääruoat");
+        tfPääruoat = new IntegerSpinner();
+        kuljetusTiedot.addRow(kuljetusTiedot.getRowCount(), pääruoat, tfPääruoat);
+
+        Label jälkiruoat = new Label("Jälkiruoat");
+        tfJälkiruoat = new IntegerSpinner();
+        kuljetusTiedot.addRow(kuljetusTiedot.getRowCount(), jälkiruoat, tfJälkiruoat);
+
+        kuljetusTiedot.addRow(kuljetusTiedot.getRowCount(), new Label("Kuljetukset per päivä"));
+
+        Label maanantai = new Label("Maanantai");
+        tfMaanantai = new IntegerSpinner();
+        kuljetusTiedot.addRow(kuljetusTiedot.getRowCount(), maanantai, tfMaanantai);
+
+        Label tiistai = new Label("Tiistai");
+        tfTiistai = new IntegerSpinner();
+        kuljetusTiedot.addRow(kuljetusTiedot.getRowCount(), tiistai, tfTiistai);
+
+        Label keskiviikko = new Label("Keskiviikko");
+        tfKeskiviikko = new IntegerSpinner();
+        kuljetusTiedot.addRow(kuljetusTiedot.getRowCount(), keskiviikko, tfKeskiviikko);
+
+        Label torstai = new Label("Torstai");
+        tfTorstai = new IntegerSpinner();
+        kuljetusTiedot.addRow(kuljetusTiedot.getRowCount(), torstai, tfTorstai);
+
+        Label perjantai = new Label("Perjantai");
+        tfPerjantai = new IntegerSpinner();
+        kuljetusTiedot.addRow(kuljetusTiedot.getRowCount(), perjantai, tfPerjantai);
+
+        tfLauantai = new IntegerSpinner();
+        kuljetusTiedot.addRow(kuljetusTiedot.getRowCount(), new Label("Lauantai"), tfLauantai);
+
+        tfSunnuntai = new IntegerSpinner();
+        kuljetusTiedot.addRow(kuljetusTiedot.getRowCount(), new Label("Sunnuntai"), tfSunnuntai);
+
+        Label ruokarajoitteet = new Label("Ruokarajoitteet");
+        tfRuokarajoitteet = new TextField();
+        kuljetusTiedot.addRow(kuljetusTiedot.getRowCount(), ruokarajoitteet, tfRuokarajoitteet);
+
+        Label avainTieto = new Label("Avaintieto");
+        tfAvainTieto = new TextField();
+        kuljetusTiedot.addRow(kuljetusTiedot.getRowCount(), avainTieto, tfAvainTieto);
+
+        Label lisätiedot = new Label("Kuljetuksen lisätiedot");
+        tfLisätiedot = new TextField();
+        tfLisätiedot.setPromptText("Näkyy raporteilla");
+        kuljetusTiedot.addRow(kuljetusTiedot.getRowCount(), lisätiedot, tfLisätiedot);
+
+
         GridPane asiakasTiedot = new GridPane();
         asiakasTiedot.setPadding(new Insets(15));
         asiakasTiedot.setHgap(10);
         asiakasTiedot.setVgap(10);
-        tabAsiakasTiedot.setContent(asiakasTiedot);
-
-        asiakasTiedot.addRow(asiakasTiedot.getRowCount(), new Label("Vahvistetut kentät pakollisia"));
+        tabAsiakastiedot.setContent(asiakasTiedot);
 
         Label nimi = new Label("Asiakkaan etu- ja sukunimi");
         nimi.setStyle("-fx-font-weight: bold;");
@@ -75,77 +140,21 @@ public class AsiakasIkkuna extends Stage {
         tfOsoite.setPromptText("Pakollinen tieto");
         asiakasTiedot.addRow(asiakasTiedot.getRowCount(), osoite, tfOsoite);
 
-        asiakasTiedot.addRow(asiakasTiedot.getRowCount(), new Label("Määrät per kuljetus"));
-
-        Label salaatit = new Label("Salaatit");
-        tfSalaatit = new IntegerSpinner();
-        asiakasTiedot.addRow(asiakasTiedot.getRowCount(), salaatit, tfSalaatit);
-
-        Label pääruoat = new Label("Pääruoat");
-        tfPääruoat = new IntegerSpinner();
-        asiakasTiedot.addRow(asiakasTiedot.getRowCount(), pääruoat, tfPääruoat);
-
-        Label jälkiruoat = new Label("Jälkiruoat");
-        tfJälkiruoat = new IntegerSpinner();
-        asiakasTiedot.addRow(asiakasTiedot.getRowCount(), jälkiruoat, tfJälkiruoat);
-
-        asiakasTiedot.addRow(asiakasTiedot.getRowCount(), new Label("Kuljetukset per päivä"));
-
-        Label maanantai = new Label("Maanantai");
-        tfMaanantai = new IntegerSpinner();
-        asiakasTiedot.addRow(asiakasTiedot.getRowCount(), maanantai, tfMaanantai);
-
-        Label tiistai = new Label("Tiistai");
-        tfTiistai = new IntegerSpinner();
-        asiakasTiedot.addRow(asiakasTiedot.getRowCount(), tiistai, tfTiistai);
-
-        Label keskiviikko = new Label("Keskiviikko");
-        tfKeskiviikko = new IntegerSpinner();
-        asiakasTiedot.addRow(asiakasTiedot.getRowCount(), keskiviikko, tfKeskiviikko);
-
-        Label torstai = new Label("Torstai");
-        tfTorstai = new IntegerSpinner();
-        asiakasTiedot.addRow(asiakasTiedot.getRowCount(), torstai, tfTorstai);
-
-        Label perjantai = new Label("Perjantai");
-        tfPerjantai = new IntegerSpinner();
-        asiakasTiedot.addRow(asiakasTiedot.getRowCount(), perjantai, tfPerjantai);
-
-        Label ruokarajoitteet = new Label("Ruokarajoitteet");
-        tfRuokarajoitteet = new TextField();
-        asiakasTiedot.addRow(asiakasTiedot.getRowCount(), ruokarajoitteet, tfRuokarajoitteet);
-
-        Label avainTieto = new Label("Avaintieto");
-        tfAvainTieto = new TextField();
-        asiakasTiedot.addRow(asiakasTiedot.getRowCount(), avainTieto, tfAvainTieto);
-
-        Label lisätiedot = new Label("Kuljetuksen lisätiedot");
-        tfLisätiedot = new TextField();
-        tfLisätiedot.setPromptText("Näkyy raporteilla");
-        asiakasTiedot.addRow(asiakasTiedot.getRowCount(), lisätiedot, tfLisätiedot);
-
-
-        GridPane laskutusTiedot = new GridPane();
-        laskutusTiedot.setPadding(new Insets(15));
-        laskutusTiedot.setHgap(10);
-        laskutusTiedot.setVgap(10);
-        tabLaskutusTiedot.setContent(laskutusTiedot);
-
         Label yhteyshenkilönNimi = new Label("Yhteyshenkilön etu- ja sukunimi");
         tfYhteyshenkilönNimi = new TextField();
-        laskutusTiedot.addRow(laskutusTiedot.getRowCount(), yhteyshenkilönNimi, tfYhteyshenkilönNimi);
+        asiakasTiedot.addRow(asiakasTiedot.getRowCount(), yhteyshenkilönNimi, tfYhteyshenkilönNimi);
 
         Label laskutusOsoite = new Label("Laskutusosoite");
         tfLaskutusOsoite = new TextField();
-        laskutusTiedot.addRow(laskutusTiedot.getRowCount(), laskutusOsoite, tfLaskutusOsoite);
+        asiakasTiedot.addRow(asiakasTiedot.getRowCount(), laskutusOsoite, tfLaskutusOsoite);
 
         Label yhteyshenkilönPuhelinnumero = new Label("Puhelinnumero");
         tfYhteyshenkilönPuhelinnumero = new TextField();
-        laskutusTiedot.addRow(laskutusTiedot.getRowCount(), yhteyshenkilönPuhelinnumero, tfYhteyshenkilönPuhelinnumero);
+        asiakasTiedot.addRow(asiakasTiedot.getRowCount(), yhteyshenkilönPuhelinnumero, tfYhteyshenkilönPuhelinnumero);
 
         Label yhteyshenkilönSähköposti = new Label("Sähköposti");
         tfYhteyshenkilönSähköposti = new TextField();
-        laskutusTiedot.addRow(laskutusTiedot.getRowCount(), yhteyshenkilönSähköposti, tfYhteyshenkilönSähköposti);
+        asiakasTiedot.addRow(asiakasTiedot.getRowCount(), yhteyshenkilönSähköposti, tfYhteyshenkilönSähköposti);
 
         HBox napit = new HBox();
         Button peruuta = new Button("Peruuta");
@@ -185,6 +194,8 @@ public class AsiakasIkkuna extends Stage {
         tfKeskiviikko.getValueFactory().setValue(käsiteltäväAsiakas.getKeskiviikkoKpl());
         tfTorstai.getValueFactory().setValue(käsiteltäväAsiakas.getTorstaiKpl());
         tfPerjantai.getValueFactory().setValue(käsiteltäväAsiakas.getPerjantaiKpl());
+        tfLauantai.getValueFactory().setValue(käsiteltäväAsiakas.getLauantaiKpl());
+        tfSunnuntai.getValueFactory().setValue(käsiteltäväAsiakas.getSunnuntaiKpl());
         tfSalaatit.getValueFactory().setValue(käsiteltäväAsiakas.getSalaatit());
         tfPääruoat.getValueFactory().setValue(käsiteltäväAsiakas.getPääruoat());
         tfJälkiruoat.getValueFactory().setValue(käsiteltäväAsiakas.getJälkiruoat());
@@ -218,6 +229,8 @@ public class AsiakasIkkuna extends Stage {
         käsiteltäväAsiakas.setKeskiviikkoKpl(tfKeskiviikko.getValue());
         käsiteltäväAsiakas.setTorstaiKpl(tfTorstai.getValue());
         käsiteltäväAsiakas.setPerjantaiKpl(tfPerjantai.getValue());
+        käsiteltäväAsiakas.setLauantaiKpl(tfLauantai.getValue());
+        käsiteltäväAsiakas.setSunnuntaiKpl(tfSunnuntai.getValue());
         käsiteltäväAsiakas.setSalaatit(tfSalaatit.getValue());
         käsiteltäväAsiakas.setPääruoat(tfPääruoat.getValue());
         käsiteltäväAsiakas.setJälkiruoat(tfJälkiruoat.getValue());
@@ -246,7 +259,14 @@ public class AsiakasIkkuna extends Stage {
         }
 
         if (läpi) {
-            int viikonKuljetukset = käsiteltäväAsiakas.getMaanantaiKpl() + käsiteltäväAsiakas.getTiistaiKpl() + käsiteltäväAsiakas.getKeskiviikkoKpl() + käsiteltäväAsiakas.getTorstaiKpl() + käsiteltäväAsiakas.getPerjantaiKpl();
+            int viikonKuljetukset = käsiteltäväAsiakas.getMaanantaiKpl() +
+                    käsiteltäväAsiakas.getTiistaiKpl() +
+                    käsiteltäväAsiakas.getKeskiviikkoKpl() +
+                    käsiteltäväAsiakas.getTorstaiKpl() +
+                    käsiteltäväAsiakas.getPerjantaiKpl() +
+                    käsiteltäväAsiakas.getLauantaiKpl() +
+                    käsiteltäväAsiakas.getSunnuntaiKpl();
+
             Alert alert = new Alert(Alert.AlertType.INFORMATION);
             alert.setTitle("Onnistuimme!");
             alert.setHeaderText("Asiakkaan tallentaminen onnistui!");

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

@@ -15,6 +15,8 @@ import javafx.scene.layout.*;
 import java.awt.*;
 import java.io.File;
 import java.io.IOException;
+import java.net.URI;
+import java.net.URISyntaxException;
 import java.util.Objects;
 import java.util.Optional;
 
@@ -38,12 +40,12 @@ public class Asiakashallinta extends TabPohja {
         yläpalkinNapit.setSpacing(5);
         yläpalkki.setRight(yläpalkinNapit);
 
-        Button avaaVersioloki = new Button("Avaa muutosloki");
+        Button avaaVersioloki = new Button("VERSIOLOKI");
         avaaVersioloki.setFont(buttonFont);
         avaaVersioloki.setOnAction(_ -> {
             try {
-                Desktop.getDesktop().browse(new File("CHANGELOG.md").toURI());
-            } catch (IOException e) {
+                Desktop.getDesktop().browse(new URI("https://git.lpam.fi/laurimaaninka/Ruokamanageri/src/branch/master/CHANGELOG.md"));
+            } catch (IOException | URISyntaxException e) {
                 System.out.println(e.getMessage());
             }
         });
@@ -81,63 +83,68 @@ public class Asiakashallinta extends TabPohja {
         });
         root.setCenter(tvAsiakkaat);
 
+        TableColumn<Asiakas, String> tcId = new TableColumn<>("Id");
+        tcId.setCellValueFactory(new PropertyValueFactory<>("id"));
         TableColumn<Asiakas, String> tcNimi = new TableColumn<>("Nimi");
-        tcNimi.setMinWidth(180);
         tcNimi.setCellValueFactory(new PropertyValueFactory<>("nimi"));
         TableColumn<Asiakas, String> tcOsoite = new TableColumn<>("Osoite");
-        tcOsoite.setMinWidth(250);
         tcOsoite.setCellValueFactory(new PropertyValueFactory<>("osoite"));
 
+        int minWidth = 40;
         TableColumn<Asiakas, MaaraTableColumn<Asiakas, Integer>> tcKuljetuspäivät = new TableColumn<>("Kuljetukset per päivä");
-        MaaraTableColumn<Asiakas, Integer> tcMaanantaiKpl = new MaaraTableColumn<>("MA", 20);
+        MaaraTableColumn<Asiakas, Integer> tcMaanantaiKpl = new MaaraTableColumn<>("MA", minWidth);
         tcMaanantaiKpl.setCellValueFactory(new PropertyValueFactory<>("maanantaiKpl"));
-        MaaraTableColumn<Asiakas, Integer> tcTiistaiKpl = new MaaraTableColumn<>("TI", 20);
+        MaaraTableColumn<Asiakas, Integer> tcTiistaiKpl = new MaaraTableColumn<>("TI", minWidth);
         tcTiistaiKpl.setCellValueFactory(new PropertyValueFactory<>("tiistaiKpl"));
-        MaaraTableColumn<Asiakas, Integer> tcKeskiviikkoKpl = new MaaraTableColumn<>("KE", 20);
+        MaaraTableColumn<Asiakas, Integer> tcKeskiviikkoKpl = new MaaraTableColumn<>("KE", minWidth);
         tcKeskiviikkoKpl.setCellValueFactory(new PropertyValueFactory<>("keskiviikkoKpl"));
-        MaaraTableColumn<Asiakas, Integer> tcTorstaiKpl = new MaaraTableColumn<>("TO", 20);
+        MaaraTableColumn<Asiakas, Integer> tcTorstaiKpl = new MaaraTableColumn<>("TO", minWidth);
         tcTorstaiKpl.setCellValueFactory(new PropertyValueFactory<>("torstaiKpl"));
-        MaaraTableColumn<Asiakas, Integer> tcPerjantaiKpl = new MaaraTableColumn<>("PE", 20);
+        MaaraTableColumn<Asiakas, Integer> tcPerjantaiKpl = new MaaraTableColumn<>("PE", minWidth);
         tcPerjantaiKpl.setCellValueFactory(new PropertyValueFactory<>("perjantaiKpl"));
-        tcKuljetuspäivät.getColumns().addAll(tcMaanantaiKpl, tcTiistaiKpl, tcKeskiviikkoKpl, tcTorstaiKpl, tcPerjantaiKpl);
+        MaaraTableColumn<Asiakas, Integer> tcLauantaiKpl = new MaaraTableColumn<>("LA", minWidth);
+        tcLauantaiKpl.setCellValueFactory(new PropertyValueFactory<>("lauantaiKpl"));
+        MaaraTableColumn<Asiakas, Integer> tcSunnuntaiKpl = new MaaraTableColumn<>("SU", minWidth);
+        tcSunnuntaiKpl.setCellValueFactory(new PropertyValueFactory<>("sunnuntaiKpl"));
+        tcKuljetuspäivät.getColumns().addAll(tcMaanantaiKpl, tcTiistaiKpl, tcKeskiviikkoKpl, tcTorstaiKpl, tcPerjantaiKpl, tcLauantaiKpl, tcSunnuntaiKpl);
 
         TableColumn<Asiakas, MaaraTableColumn<Asiakas, Integer>> tcKuljetusmäärät = new TableColumn<>("Määrät per kuljetus");
-        MaaraTableColumn<Asiakas, Integer> tcSalaatit = new MaaraTableColumn<>("Salaatit", 90);
+        MaaraTableColumn<Asiakas, Integer> tcSalaatit = new MaaraTableColumn<>("Salaatit", minWidth, 150);
         tcSalaatit.setCellValueFactory(new PropertyValueFactory<>("salaatit"));
-        MaaraTableColumn<Asiakas, Integer> tcPääruoat = new MaaraTableColumn<>("Pääruoat", 90);
+        MaaraTableColumn<Asiakas, Integer> tcPääruoat = new MaaraTableColumn<>("Pääruoat", minWidth, 150);
         tcPääruoat.setCellValueFactory(new PropertyValueFactory<>("pääruoat"));
-        MaaraTableColumn<Asiakas, Integer> tcJälkiruoat = new MaaraTableColumn<>("Jälkiruoat", 90);
+        MaaraTableColumn<Asiakas, Integer> tcJälkiruoat = new MaaraTableColumn<>("Jälkiruoat", minWidth, 150);
         tcJälkiruoat.setCellValueFactory(new PropertyValueFactory<>("jälkiruoat"));
         tcKuljetusmäärät.getColumns().addAll(tcSalaatit, tcPääruoat, tcJälkiruoat);
 
         TableColumn<Asiakas, String> tcErityisruokavalio = new TableColumn<>("Ruokarajoitteet");
-        tcErityisruokavalio.setMinWidth(150);
+//        tcErityisruokavalio.setMinWidth(150);
         tcErityisruokavalio.setCellValueFactory(new PropertyValueFactory<>("erityisruokavalio"));
         TableColumn<Asiakas, String> tcAvainTieto = new TableColumn<>("Avaintieto");
-        tcAvainTieto.setMinWidth(150);
+//        tcAvainTieto.setMinWidth(150);
         tcAvainTieto.setCellValueFactory(new PropertyValueFactory<>("avainTieto"));
         TableColumn<Asiakas, String> tcLisätiedot = new TableColumn<>("Lisätieto kuljetukselle");
-        tcLisätiedot.setMinWidth(250);
+//        tcLisätiedot.setMinWidth(250);
         tcLisätiedot.setCellValueFactory(new PropertyValueFactory<>("lisätiedot"));
         TableColumn<Asiakas, String> tcYhteyshenkilönNimi = new TableColumn<>("Yhteyshenkilö");
-        tcYhteyshenkilönNimi.setMinWidth(250);
+//        tcYhteyshenkilönNimi.setMinWidth(250);
         tcYhteyshenkilönNimi.setCellValueFactory(new PropertyValueFactory<>("yhteyshenkilönNimi"));
         TableColumn<Asiakas, String> tcLaskutusOsoite = new TableColumn<>("Laskutusosoite");
-        tcLaskutusOsoite.setMinWidth(250);
+//        tcLaskutusOsoite.setMinWidth(250);
         tcLaskutusOsoite.setCellValueFactory(new PropertyValueFactory<>("laskutusOsoite"));
         TableColumn<Asiakas, String> tcYhteyshenkilönPuhelinnumero = new TableColumn<>("Yhteyshenkilön puhelinnumero");
-        tcYhteyshenkilönPuhelinnumero.setMinWidth(250);
+//        tcYhteyshenkilönPuhelinnumero.setMinWidth(250);
         tcYhteyshenkilönPuhelinnumero.setCellValueFactory(new PropertyValueFactory<>("yhteyshenkilönPuhelinnumero"));
         TableColumn<Asiakas, String> tcYhteyshenkilönSähköposti = new TableColumn<>("Yhteyshenkilön sähköposti");
-        tcYhteyshenkilönSähköposti.setMinWidth(250);
+//        tcYhteyshenkilönSähköposti.setMinWidth(250);
         tcYhteyshenkilönSähköposti.setCellValueFactory(new PropertyValueFactory<>("yhteyshenkilönSähköposti"));
 
-        tvAsiakkaat.getColumns().addAll(tcNimi, tcOsoite, tcKuljetuspäivät, tcKuljetusmäärät, tcErityisruokavalio, tcAvainTieto, tcLisätiedot, tcYhteyshenkilönNimi, tcLaskutusOsoite, tcYhteyshenkilönPuhelinnumero, tcYhteyshenkilönSähköposti);
+        tvAsiakkaat.getColumns().addAll(tcId, tcNimi, tcOsoite, tcKuljetuspäivät, tcKuljetusmäärät, tcErityisruokavalio, tcAvainTieto, tcLisätiedot, tcYhteyshenkilönNimi, tcLaskutusOsoite, tcYhteyshenkilönPuhelinnumero, tcYhteyshenkilönSähköposti);
 
         for (TableColumn<Asiakas, ?> column : tvAsiakkaat.getColumns()) {
             column.setEditable(false);
             column.setReorderable(false);
-            column.setResizable(false);
+            column.setResizable(true);
             column.setStyle("-fx-alignment: CENTER; -fx-font-size: 16px;");
 
         }

src/main/java/fi/lpam/ruokamanageri/gui/Kierroshallinta.java

@@ -12,6 +12,7 @@ import javafx.collections.ObservableList;
 import javafx.scene.control.Alert;
 import javafx.scene.control.Button;
 import javafx.scene.control.Label;
+import javafx.scene.control.ScrollPane;
 import javafx.scene.layout.BorderPane;
 import javafx.scene.layout.HBox;
 
@@ -47,7 +48,10 @@ public class Kierroshallinta extends TabPohja {
         yläpalkki.setRight(yläpalkinNapit);
 
         Button lisääKierros = new Button("Lisää uusi kierros");
-        lisääKierros.setOnAction(_ -> lisaaKierros());
+        lisääKierros.setOnAction(_ -> {
+            int uudenKierroksenNumero = mclv.getColumns().size();
+            lisaaKierros(new Kierros(uudenKierroksenNumero, FXCollections.observableArrayList(new ArrayList<>())));
+        });
         lisääKierros.setFont(TabPohja.buttonFont);
         yläpalkinNapit.getChildren().add(lisääKierros);
 
@@ -67,14 +71,11 @@ public class Kierroshallinta extends TabPohja {
         HashSet<Integer> kierroksellaOlevatAsiakasIDt = new HashSet<>();
 
         if (!kierrokset.isEmpty()) {
-            for (Kierros k : kierrokset) {
-                MultiColumnListView.ListViewColumn<Asiakas> sarake = new MultiColumnListView.ListViewColumn<>();
-                sarake.setHeader(new Label("Kierros " + k.getKierrosNumero()));
-                sarake.setItems(k.getAsiakkaat());
-                for (Asiakas asiakas : k.getAsiakkaat()) {
+            for (Kierros kierros : kierrokset) {
+                lisaaKierros(kierros);
+                for (Asiakas asiakas : kierros.getAsiakkaat()) {
                     kierroksellaOlevatAsiakasIDt.add(asiakas.getId());
                 }
-                mclv.getColumns().add(sarake);
             }
         }
 
@@ -89,14 +90,12 @@ public class Kierroshallinta extends TabPohja {
         mclv.getColumns().add(colEiKierroksella);
     }
 
-    private void lisaaKierros() {
-        int uudenKierroksenNumero = mclv.getColumns().size();
-        Kierros lisättäväKierros = new Kierros(uudenKierroksenNumero, FXCollections.observableArrayList(new ArrayList<>()));
-        kierrokset.add(lisättäväKierros);
+    private void lisaaKierros(Kierros kierros) {
+        if (kierros.getId() == -1) kierrokset.add(kierros);
         MultiColumnListView.ListViewColumn<Asiakas> uusiSarake = new MultiColumnListView.ListViewColumn<>();
-        uusiSarake.setHeader(new Label("Kierros " + uudenKierroksenNumero));
-        uusiSarake.setItems(lisättäväKierros.getAsiakkaat());
-        mclv.getColumns().add(uudenKierroksenNumero-1, uusiSarake);
+        uusiSarake.setHeader(new Label("Kierros " + kierros.getKierrosNumero()));
+        uusiSarake.setItems(kierros.getAsiakkaat());
+        mclv.getColumns().add(kierros.getKierrosNumero()-1, uusiSarake);
     }
 
     private void tallennaKierrokset() {

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

@@ -3,7 +3,8 @@ package fi.lpam.ruokamanageri.gui;
 import fi.lpam.ruokamanageri.dataluokat.Asiakas;
 import fi.lpam.ruokamanageri.dataluokat.Kierros;
 import fi.lpam.ruokamanageri.dataluokat.Kuljetus;
-import fi.lpam.ruokamanageri.gui.elementit.KuljetaTableCell;
+import fi.lpam.ruokamanageri.gui.tableCell.IntegerFieldTableCellFactory;
+import fi.lpam.ruokamanageri.gui.tableCell.KuljetaTableCell;
 import fi.lpam.ruokamanageri.tulostajat.KuljetusListaTulostaja;
 import fi.lpam.ruokamanageri.gui.elementit.MaaraTableColumn;
 import fi.lpam.ruokamanageri.gui.elementit.TabPohja;
@@ -13,7 +14,6 @@ import javafx.collections.FXCollections;
 import javafx.collections.ObservableList;
 import javafx.geometry.Pos;
 import javafx.scene.control.*;
-import javafx.scene.control.cell.ChoiceBoxTableCell;
 import javafx.scene.control.cell.PropertyValueFactory;
 import javafx.scene.control.cell.TextFieldTableCell;
 import javafx.scene.layout.BorderPane;
@@ -109,17 +109,17 @@ public class KuljetusListat extends TabPohja {
         tcOsoite.setOnEditCommit((TableColumn.CellEditEvent<Kuljetus, String> event) -> {event.getTableView().getItems().get(event.getTablePosition().getRow()).setOsoite(event.getNewValue()); tallennusStatus.setText("Tallennettu: Ei");});
 
         TableColumn<Kuljetus, Integer> tcSalaatit = new MaaraTableColumn<>("Salaatit", 100);
-        tcSalaatit.setCellFactory(ChoiceBoxTableCell.forTableColumn(määräVaihtoehdot));
+        tcSalaatit.setCellFactory(new IntegerFieldTableCellFactory<>());
         tcSalaatit.setCellValueFactory(new PropertyValueFactory<>("salaatit"));
         tcSalaatit.setOnEditCommit((TableColumn.CellEditEvent<Kuljetus, Integer> event) -> {event.getTableView().getItems().get(event.getTablePosition().getRow()).setSalaatit(event.getNewValue()); tallennusStatus.setText("Tallennettu: Ei");});
 
         TableColumn<Kuljetus, Integer> tcPääruoat = new MaaraTableColumn<>("Pääruoat", 100);
-        tcPääruoat.setCellFactory(ChoiceBoxTableCell.forTableColumn(määräVaihtoehdot));
+        tcPääruoat.setCellFactory(new IntegerFieldTableCellFactory<>());
         tcPääruoat.setCellValueFactory(new PropertyValueFactory<>("pääruoat"));
         tcPääruoat.setOnEditCommit((TableColumn.CellEditEvent<Kuljetus, Integer> event) -> {event.getTableView().getItems().get(event.getTablePosition().getRow()).setPääruoat(event.getNewValue()); tallennusStatus.setText("Tallennettu: Ei");});
 
         TableColumn<Kuljetus, Integer> tcJälkiruoat = new MaaraTableColumn<>("Jälkiruoat", 100);
-        tcJälkiruoat.setCellFactory(ChoiceBoxTableCell.forTableColumn(määräVaihtoehdot));
+        tcJälkiruoat.setCellFactory(new IntegerFieldTableCellFactory<>());
         tcJälkiruoat.setCellValueFactory(new PropertyValueFactory<>("jälkiruoat"));
         tcJälkiruoat.setOnEditCommit((TableColumn.CellEditEvent<Kuljetus, Integer> event) -> {event.getTableView().getItems().get(event.getTablePosition().getRow()).setJälkiruoat(event.getNewValue()); tallennusStatus.setText("Tallennettu: Ei");});
 

src/main/java/fi/lpam/ruokamanageri/gui/KuljetusRaportit.java

@@ -2,7 +2,7 @@ package fi.lpam.ruokamanageri.gui;
 
 import fi.lpam.ruokamanageri.dataluokat.RaporttiRivi;
 import fi.lpam.ruokamanageri.tulostajat.KuljetusRaporttiTulostaja;
-import fi.lpam.ruokamanageri.gui.elementit.PaivamaaraTableCell;
+import fi.lpam.ruokamanageri.gui.tableCell.PaivamaaraTableCell;
 import fi.lpam.ruokamanageri.gui.elementit.TabPohja;
 import javafx.collections.FXCollections;
 import javafx.geometry.Pos;

src/main/java/fi/lpam/ruokamanageri/gui/elementit/AsiakasListCell.java

@@ -4,6 +4,7 @@ import com.dlsc.gemsfx.MultiColumnListView;
 import fi.lpam.ruokamanageri.dataluokat.Asiakas;
 import javafx.beans.property.BooleanProperty;
 import javafx.beans.property.SimpleBooleanProperty;
+import javafx.geometry.Pos;
 import javafx.scene.Node;
 import javafx.scene.control.ContentDisplay;
 import javafx.scene.control.Label;
@@ -14,12 +15,10 @@ import javafx.scene.layout.VBox;
 public class AsiakasListCell extends MultiColumnListView.ColumnListCell<Asiakas> {
     private final StackPane wrapper;
 
-    public AsiakasListCell(MultiColumnListView multiColumnListView) {
-        //noinspection unchecked
+    public AsiakasListCell(MultiColumnListView<Asiakas> multiColumnListView) {
         super(multiColumnListView);
-        this.setHeight(150);
-        this.setPrefHeight(50);
-
+        this.setMinHeight(60);
+        this.setMaxHeight(60);
 
         VBox content = new VBox();
         content.visibleProperty().bind(placeholder.not().and(emptyProperty().not()));
@@ -30,9 +29,11 @@ public class AsiakasListCell extends MultiColumnListView.ColumnListCell<Asiakas>
         contentPlaceholder.managedProperty().bind(placeholder);
 
         Label label = new Label();
+        label.setFont(TabPohja.tekstiFont);
         label.textProperty().bind(textProperty());
 
         wrapper = new StackPane(content, contentPlaceholder, label);
+        wrapper.setAlignment(Pos.CENTER_LEFT);
         setGraphic(wrapper);
         setContentDisplay(ContentDisplay.GRAPHIC_ONLY);
     }
@@ -59,7 +60,7 @@ public class AsiakasListCell extends MultiColumnListView.ColumnListCell<Asiakas>
                 placeholder.set(true);
                 setText("");
             } else {
-                setText(asiakas.getNimi() + ": " + asiakas.getOsoite());
+                setText(asiakas.getNimi() + "\n" + asiakas.getOsoite());
             }
         } else {
             setText("");

src/main/java/fi/lpam/ruokamanageri/gui/elementit/IntegerSpinner.java

@@ -4,11 +4,8 @@ import javafx.scene.control.Spinner;
 import javafx.scene.control.SpinnerValueFactory;
 
 public class IntegerSpinner extends Spinner<Integer> {
-    /**
-     * Min 0, Max 50
-     */
     public IntegerSpinner() {
-        super(new SpinnerValueFactory.IntegerSpinnerValueFactory(0, 50));
+        super(new SpinnerValueFactory.IntegerSpinnerValueFactory(0, 500));
         this.setEditable(true);
     }
 }

src/main/java/fi/lpam/ruokamanageri/gui/elementit/MaaraTableColumn.java

@@ -3,10 +3,17 @@ package fi.lpam.ruokamanageri.gui.elementit;
 import javafx.scene.control.TableColumn;
 
 public class MaaraTableColumn<Asiakas, Integer> extends TableColumn<Asiakas, Integer> {
-    public MaaraTableColumn(String columnLabel, int minWidth) {
+    public MaaraTableColumn(String columnLabel, int minWidth, int maxWidth) {
         super(columnLabel);
-        this.setResizable(false);
+        this.setResizable(true);
         this.setReorderable(false);
         this.setMinWidth(minWidth);
+        this.setMaxWidth(maxWidth);
+        this.setStyle("-fx-alignment: CENTER; -fx-font-size: 16px;");
+    }
+
+    public MaaraTableColumn(String columnLabel, int width) {
+        this(columnLabel, width, width);
+        this.setResizable(false);
     }
 }

src/main/java/fi/lpam/ruokamanageri/gui/tableCell/IntegerFieldTableCellFactory.java

@@ -0,0 +1,30 @@
+package fi.lpam.ruokamanageri.gui.tableCell;
+
+import javafx.scene.control.TableCell;
+import javafx.scene.control.TableColumn;
+import javafx.scene.control.cell.TextFieldTableCell;
+import javafx.scene.input.KeyEvent;
+import javafx.util.Callback;
+import javafx.util.converter.IntegerStringConverter;
+
+public class IntegerFieldTableCellFactory<T> implements Callback<TableColumn<T, Integer>, TableCell<T, Integer>> {
+    @Override
+    public TableCell<T, Integer> call(TableColumn<T, Integer> param ) {
+        return new TextFieldTableCell<>(new IntegerStringConverter()) {
+            private boolean filterSet = false;
+
+            @Override
+            public void startEdit() {
+                super.startEdit();
+                if (!filterSet) {
+                    getGraphic().addEventFilter(KeyEvent.KEY_TYPED, KE -> {
+                        if (!"0123456789".contains(KE.getCharacter())) {
+                            KE.consume();
+                        }
+                    });
+                    filterSet = true;
+                }
+            }
+        };
+    }
+}

src/main/java/fi/lpam/ruokamanageri/gui/tableCell/KuljetaTableCell.java

@@ -1,4 +1,4 @@
-package fi.lpam.ruokamanageri.gui.elementit;
+package fi.lpam.ruokamanageri.gui.tableCell;
 
 import fi.lpam.ruokamanageri.dataluokat.Asiakas;
 import fi.lpam.ruokamanageri.dataluokat.Kuljetus;

src/main/java/fi/lpam/ruokamanageri/gui/tableCell/PaivamaaraTableCell.java

@@ -1,4 +1,4 @@
-package fi.lpam.ruokamanageri.gui.elementit;
+package fi.lpam.ruokamanageri.gui.tableCell;
 
 import fi.lpam.ruokamanageri.dataluokat.RaporttiRivi;
 import javafx.scene.control.TableCell;

src/main/resources/tietokanta.sql

@@ -12,7 +12,11 @@ create table if not exists asiakkaat (
     jälkiruoat int not null,
     erityisruokavalio varchar(255),
     avainTieto varchar(255),
-    lisätiedot text
+    lisätiedot text,
+    laskutusosoite varchar(255),
+    yhteyshenkilönNimi varchar(255),
+    yhteyshenkilönPuhelinnumero varchar(255),
+    yhteyshenkilönSähköposti varchar(255)
 );
 
 create table if not exists kierrokset (