Versio 2.0.1

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

.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="951" />
+      <option name="minLength" value="735" />
     </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,12 +1,3 @@
-# 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
-
 # 2.0.1
 - Vaihdettu versioloki avautumaan paikallisena tiedostona
 
@@ -25,4 +16,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>3.0.0</version>
+    <version>2.0.1</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,17 +3,13 @@ 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, lauantaiKpl, sunnuntaiKpl;
+    private int maanantaiKpl, tiistaiKpl, keskiviikkoKpl, torstaiKpl, perjantaiKpl;
     private int salaatit, pääruoat, jälkiruoat;
     private String erityisruokavalio;
     private String avainTieto;
@@ -44,8 +40,6 @@ 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ä
@@ -65,8 +59,6 @@ public class Asiakas implements TietokantaOlio {
                     int keskiviikkoKpl,
                     int torstaiKpl,
                     int perjantaiKpl,
-                    int lauantaiKpl,
-                    int sunnuntaiKpl,
                     int salaatit,
                     int pääruoat,
                     int jälkiruoat,
@@ -85,8 +77,6 @@ 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;
@@ -116,7 +106,7 @@ public class Asiakas implements TietokantaOlio {
     }
 
     /**
-     * Hakee tietokannasta asiakasidn perusteella asiakkaat ja palauttaa ne annetussa järjestyksessä
+     * Hakee tietokannasta asiakasidn perusteella asiakkaat
      * @param IDt haettavat asiakkkaat
      * @return Haetut asiakkaat tai null
      */
@@ -127,17 +117,10 @@ public class Asiakas implements TietokantaOlio {
             sbIDt.append(", ").append(IDt[i]);
         }
         try (Connection tietokanta = Tietokanta.haeYhteys()) {
-            String sql = "select * from asiakkaat where id in (" + sbIDt + ")";
+            String sql = "select * from asiakkaat where id in (" + sbIDt + ") order by instr('" + sbIDt + "', id)";
             Statement stmt = tietokanta.createStatement();
             ResultSet rs = stmt.executeQuery(sql);
-
+            return parsiResultSet(rs);
-            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) {
-                järjestetyt.add(asiakkaat.get(id));
-            }
-            return järjestetyt;
         }
         catch (SQLException e) {
             SiirtoVirhe alert = new SiirtoVirhe(e);
@@ -156,27 +139,26 @@ public class Asiakas implements TietokantaOlio {
 
         try {
             while (rs.next()) {
+                int i = 1;
                 Asiakas uusi = new Asiakas(
-                        rs.getInt("id"),
+                        rs.getInt(i++),
-                        rs.getString("nimi"),
+                        rs.getString(i++),
-                        rs.getString("osoite"),
+                        rs.getString(i++),
-                        rs.getInt("maanantaiKpl"),
+                        rs.getInt(i++),
-                        rs.getInt("tiistaiKpl"),
+                        rs.getInt(i++),
-                        rs.getInt("keskiviikkoKpl"),
+                        rs.getInt(i++),
-                        rs.getInt("torstaiKpl"),
+                        rs.getInt(i++),
-                        rs.getInt("perjantaiKpl"),
+                        rs.getInt(i++),
-                        rs.getInt("lauantaiKpl"),
+                        rs.getInt(i++),
-                        rs.getInt("sunnuntaiKpl"),
+                        rs.getInt(i++),
-                        rs.getInt("salaatit"),
+                        rs.getInt(i++),
-                        rs.getInt("pääruoat"),
+                        rs.getString(i++),
-                        rs.getInt("jälkiruoat"),
+                        rs.getString(i++),
-                        rs.getString("erityisruokavalio"),
+                        rs.getString(i++),
-                        rs.getString("avainTieto"),
+                        rs.getString(i++),
-                        rs.getString("lisätiedot"),
+                        rs.getString(i++),
-                        rs.getString("laskutusOsoite"),
+                        rs.getString(i++),
-                        rs.getString("yhteyshenkilönNimi"),
+                        rs.getString(i++)
-                        rs.getString("yhteyshenkilönPuhelinnumero"),
-                        rs.getString("yhteyshenkilönSähköposti")
                 );
                 asiakkaat.add(uusi);
             }
@@ -195,7 +177,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, 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);
+            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);
             int i = 1;
             stmt.setString(i++, nimi);
             stmt.setString(i++, osoite);
@@ -204,8 +186,6 @@ 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);
@@ -244,7 +224,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 = ?, lauantaiKpl = ?, sunnuntaiKpl = ?, 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 = ?, 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);
@@ -253,8 +233,6 @@ 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);
@@ -310,33 +288,6 @@ 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;
     }
@@ -401,22 +352,6 @@ 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,9 +36,17 @@ public class Kuljetus implements TietokantaOlio {
         this.nimi = asiakas.getNimi();
         this.kuljetusPäivämäärä = pvm;
         this.kuljetusKierros = kuljetusKierros;
-        this.salaatit = asiakas.getPäivänSalaatit(pvm);
+        int kuljetusKerroin = switch (kuljetusPäivämäärä.getDayOfWeek()) {
-        this.pääruoat = asiakas.getPäivänPääruoat(pvm);
+            case MONDAY -> asiakas.getMaanantaiKpl();
-        this.jälkiruoat = asiakas.getPäivänJälkiruoat(pvm);
+            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.osoite = asiakas.getOsoite();
         this.avainTieto = asiakas.getAvainTieto();
         this.lisätieto = asiakas.getLisätiedot();
@@ -218,16 +226,23 @@ public class Kuljetus implements TietokantaOlio {
     }
 
     public void palautaMäärät(Asiakas asiakas) {
-        setSalaatit(asiakas.getPäivänSalaatit(kuljetusPäivämäärä));
+        int kuljetusKerroin = switch (kuljetusPäivämäärä.getDayOfWeek()) {
-        setPääruoat(asiakas.getPäivänPääruoat(kuljetusPäivämäärä));
+            case MONDAY -> asiakas.getMaanantaiKpl();
-        setJälkiruoat(asiakas.getPäivänJälkiruoat(kuljetusPäivämäärä));
+            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);
         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ää kuljetuspäivät asiakassarakkeeseen (v2.0.2->3.x.x)
+        //Lisää laskutustiedot asiakastauluun
-        String[] sarakkeet = {"lauantaiKpl", "sunnuntaiKpl"};
+        String[] sarakkeet = {"laskutusosoite", "yhteyshenkilönNimi", "yhteyshenkilönPuhelinnumero", "yhteyshenkilönSähköposti"};
         for (String sarake : sarakkeet) {
             try (Connection conn = haeYhteys()) {
-                PreparedStatement stmt = conn.prepareStatement("alter table asiakkaat add column " + sarake + " int not null default 0");
+                PreparedStatement stmt = conn.prepareStatement("alter table asiakkaat add column " + sarake + " varchar(255)");
                 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, tfLauantai, tfSunnuntai, tfSalaatit, tfPääruoat, tfJälkiruoat;
+    final IntegerSpinner tfMaanantai, tfTiistai, tfKeskiviikko, tfTorstai, tfPerjantai, tfSalaatit, tfPääruoat, tfJälkiruoat;
 
     /**
      * Luo ikkunan jossa muokataan saatua asiakasta
@@ -46,82 +46,22 @@ public class AsiakasIkkuna extends Stage {
                 default: break;
             }});
 
-        Tab tabKuljetusTiedot, tabAsiakastiedot;
+        Tab tabAsiakasTiedot, tabLaskutusTiedot;
-        tabKuljetusTiedot = new Tab("Kuljetustiedot");
+        tabAsiakasTiedot = new Tab("Asiakastiedot");
-        tabAsiakastiedot = new Tab("Asiakastiedot");
+        tabLaskutusTiedot = new Tab("Laskutustiedot");
-        tabPane.getTabs().addAll(tabKuljetusTiedot, tabAsiakastiedot);
+        tabPane.getTabs().addAll(tabAsiakasTiedot, tabLaskutusTiedot);
         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);
+        tabAsiakasTiedot.setContent(asiakasTiedot);
+
+        asiakasTiedot.addRow(asiakasTiedot.getRowCount(), new Label("Vahvistetut kentät pakollisia"));
 
         Label nimi = new Label("Asiakkaan etu- ja sukunimi");
         nimi.setStyle("-fx-font-weight: bold;");
@@ -135,21 +75,77 @@ 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();
-        asiakasTiedot.addRow(asiakasTiedot.getRowCount(), yhteyshenkilönNimi, tfYhteyshenkilönNimi);
+        laskutusTiedot.addRow(laskutusTiedot.getRowCount(), yhteyshenkilönNimi, tfYhteyshenkilönNimi);
 
         Label laskutusOsoite = new Label("Laskutusosoite");
         tfLaskutusOsoite = new TextField();
-        asiakasTiedot.addRow(asiakasTiedot.getRowCount(), laskutusOsoite, tfLaskutusOsoite);
+        laskutusTiedot.addRow(laskutusTiedot.getRowCount(), laskutusOsoite, tfLaskutusOsoite);
 
         Label yhteyshenkilönPuhelinnumero = new Label("Puhelinnumero");
         tfYhteyshenkilönPuhelinnumero = new TextField();
-        asiakasTiedot.addRow(asiakasTiedot.getRowCount(), yhteyshenkilönPuhelinnumero, tfYhteyshenkilönPuhelinnumero);
+        laskutusTiedot.addRow(laskutusTiedot.getRowCount(), yhteyshenkilönPuhelinnumero, tfYhteyshenkilönPuhelinnumero);
 
         Label yhteyshenkilönSähköposti = new Label("Sähköposti");
         tfYhteyshenkilönSähköposti = new TextField();
-        asiakasTiedot.addRow(asiakasTiedot.getRowCount(), yhteyshenkilönSähköposti, tfYhteyshenkilönSähköposti);
+        laskutusTiedot.addRow(laskutusTiedot.getRowCount(), yhteyshenkilönSähköposti, tfYhteyshenkilönSähköposti);
 
         HBox napit = new HBox();
         Button peruuta = new Button("Peruuta");
@@ -189,8 +185,6 @@ 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());
@@ -224,8 +218,6 @@ 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());
@@ -254,14 +246,7 @@ public class AsiakasIkkuna extends Stage {
         }
 
         if (läpi) {
-            int viikonKuljetukset = käsiteltäväAsiakas.getMaanantaiKpl() +
+            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.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

@@ -40,12 +40,12 @@ public class Asiakashallinta extends TabPohja {
         yläpalkinNapit.setSpacing(5);
         yläpalkki.setRight(yläpalkinNapit);
 
-        Button avaaVersioloki = new Button("VERSIOLOKI");
+        Button avaaVersioloki = new Button("Avaa muutosloki");
         avaaVersioloki.setFont(buttonFont);
         avaaVersioloki.setOnAction(_ -> {
             try {
-                Desktop.getDesktop().browse(new URI("https://git.lpam.fi/laurimaaninka/Ruokamanageri/src/branch/master/CHANGELOG.md"));
+                Desktop.getDesktop().browse(new File("CHANGELOG.md").toURI());
-            } catch (IOException | URISyntaxException e) {
+            } catch (IOException e) {
                 System.out.println(e.getMessage());
             }
         });
@@ -90,30 +90,25 @@ public class Asiakashallinta extends TabPohja {
         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", minWidth);
+        MaaraTableColumn<Asiakas, Integer> tcMaanantaiKpl = new MaaraTableColumn<>("MA", 20);
         tcMaanantaiKpl.setCellValueFactory(new PropertyValueFactory<>("maanantaiKpl"));
-        MaaraTableColumn<Asiakas, Integer> tcTiistaiKpl = new MaaraTableColumn<>("TI", minWidth);
+        MaaraTableColumn<Asiakas, Integer> tcTiistaiKpl = new MaaraTableColumn<>("TI", 20);
         tcTiistaiKpl.setCellValueFactory(new PropertyValueFactory<>("tiistaiKpl"));
-        MaaraTableColumn<Asiakas, Integer> tcKeskiviikkoKpl = new MaaraTableColumn<>("KE", minWidth);
+        MaaraTableColumn<Asiakas, Integer> tcKeskiviikkoKpl = new MaaraTableColumn<>("KE", 20);
         tcKeskiviikkoKpl.setCellValueFactory(new PropertyValueFactory<>("keskiviikkoKpl"));
-        MaaraTableColumn<Asiakas, Integer> tcTorstaiKpl = new MaaraTableColumn<>("TO", minWidth);
+        MaaraTableColumn<Asiakas, Integer> tcTorstaiKpl = new MaaraTableColumn<>("TO", 20);
         tcTorstaiKpl.setCellValueFactory(new PropertyValueFactory<>("torstaiKpl"));
-        MaaraTableColumn<Asiakas, Integer> tcPerjantaiKpl = new MaaraTableColumn<>("PE", minWidth);
+        MaaraTableColumn<Asiakas, Integer> tcPerjantaiKpl = new MaaraTableColumn<>("PE", 20);
         tcPerjantaiKpl.setCellValueFactory(new PropertyValueFactory<>("perjantaiKpl"));
-        MaaraTableColumn<Asiakas, Integer> tcLauantaiKpl = new MaaraTableColumn<>("LA", minWidth);
+        tcKuljetuspäivät.getColumns().addAll(tcMaanantaiKpl, tcTiistaiKpl, tcKeskiviikkoKpl, tcTorstaiKpl, tcPerjantaiKpl);
-        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", 80);
+        MaaraTableColumn<Asiakas, Integer> tcSalaatit = new MaaraTableColumn<>("Salaatit", 90);
         tcSalaatit.setCellValueFactory(new PropertyValueFactory<>("salaatit"));
-        MaaraTableColumn<Asiakas, Integer> tcPääruoat = new MaaraTableColumn<>("Pääruoat", 80);
+        MaaraTableColumn<Asiakas, Integer> tcPääruoat = new MaaraTableColumn<>("Pääruoat", 90);
         tcPääruoat.setCellValueFactory(new PropertyValueFactory<>("pääruoat"));
-        MaaraTableColumn<Asiakas, Integer> tcJälkiruoat = new MaaraTableColumn<>("Jälkiruoat", 80);
+        MaaraTableColumn<Asiakas, Integer> tcJälkiruoat = new MaaraTableColumn<>("Jälkiruoat", 90);
         tcJälkiruoat.setCellValueFactory(new PropertyValueFactory<>("jälkiruoat"));
         tcKuljetusmäärät.getColumns().addAll(tcSalaatit, tcPääruoat, tcJälkiruoat);
 

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

@@ -104,7 +104,6 @@ public class Kierroshallinta extends TabPohja {
         int index = 0;
         for (int i = 0; i < kierrokset.size();) {
             Kierros k = kierrokset.get(i);
-            k.setAsiakkaat(FXCollections.observableArrayList(new ArrayList<>()));
             k.setAsiakkaat(mclv.getColumns().get(index).getItems());
             index++;
 

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

@@ -3,8 +3,7 @@ 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.tableCell.IntegerFieldTableCellFactory;
+import fi.lpam.ruokamanageri.gui.elementit.KuljetaTableCell;
-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;
@@ -14,6 +13,7 @@ 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(new IntegerFieldTableCellFactory<>());
+        tcSalaatit.setCellFactory(ChoiceBoxTableCell.forTableColumn(määräVaihtoehdot));
         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(new IntegerFieldTableCellFactory<>());
+        tcPääruoat.setCellFactory(ChoiceBoxTableCell.forTableColumn(määräVaihtoehdot));
         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(new IntegerFieldTableCellFactory<>());
+        tcJälkiruoat.setCellFactory(ChoiceBoxTableCell.forTableColumn(määräVaihtoehdot));
         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.tableCell.PaivamaaraTableCell;
+import fi.lpam.ruokamanageri.gui.elementit.PaivamaaraTableCell;
 import fi.lpam.ruokamanageri.gui.elementit.TabPohja;
 import javafx.collections.FXCollections;
 import javafx.geometry.Pos;

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

@@ -4,8 +4,11 @@ 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, 500));
+        super(new SpinnerValueFactory.IntegerSpinnerValueFactory(0, 50));
         this.setEditable(true);
     }
 }

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

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

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

@@ -3,19 +3,10 @@ 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 width) {
+    public MaaraTableColumn(String columnLabel, int minWidth) {
-        super(columnLabel);
-        this.setResizable(false);
-        this.setReorderable(false);
-        this.setMinWidth(width);
-        this.setMaxWidth(width);
-    }
-
-    public MaaraTableColumn(String columnLabel, int minWidth, int maxWidth) {
         super(columnLabel);
         this.setResizable(false);
         this.setReorderable(false);
         this.setMinWidth(minWidth);
-        this.setMaxWidth(maxWidth);
     }
 }

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

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

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

@@ -1,30 +0,0 @@
-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/resources/tietokanta.sql

@@ -12,11 +12,7 @@ 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 (