From 9d010d452c9bbff4c4560eff47e25afcf8dcd5de Mon Sep 17 00:00:00 2001
From: Georg-Notebook <georg@dergeorg.at>
Date: Sat, 17 Apr 2021 22:40:36 +0200
Subject: [PATCH] Settings -> Deutsch/Englisch verbessert/erweitert

---
 conf.ini.sample                             |  4 ++
 langde.ini                                  | 38 +++++++++-
 langen.ini                                  | 37 +++++++++-
 src/tourplaner/tourplaner.fxml              | 20 +++---
 src/tourplaner/ui/TourplanerController.java | 56 +++++++++++----
 src/tourplaner/viewmodels/ViewModel.java    | 79 ++++++++++++---------
 6 files changed, 175 insertions(+), 59 deletions(-)

diff --git a/conf.ini.sample b/conf.ini.sample
index eb24114..44d0e68 100644
--- a/conf.ini.sample
+++ b/conf.ini.sample
@@ -6,6 +6,10 @@ logo = logo.png
 [lang]
 lang = langde
 
+[settings]
+openpdf = 1
+openmap = 0
+
 [db]
 dbname = tourplaner
 url =
diff --git a/langde.ini b/langde.ini
index 4cddf83..67e1ca1 100644
--- a/langde.ini
+++ b/langde.ini
@@ -19,7 +19,7 @@ zielpunkttitle = Tour zielpunkt
 zielpunktheader = Bitte geben Sie den Zielpunkt der Tour an!
 zielpunktmsg = Zielpunkt:
 deltournoselect = ViewModel -> DelTour
-deltournoselectmsg = Bitte wählen Sie zuerst eine Tour aus, bevor Sie auf löschen klicken!
+deltournoselectmsg = Bitte wählen Sie zuerst eine Tour aus!
 exceptionstackheader = Exception stacktrace:
 keinelogsvorhanden = Fügen Sie erst ein Log hinzu um es anzuzeigen!
 datum = Datum
@@ -59,4 +59,38 @@ loggegangen = Davon gegangen:
 logavg = Berechnete Durchschnittliche Geschwindigkeit:
 logstrecke = Entfernung:
 loghight = Höhenmeter:
-logbemerkung = Bemerkung:
\ No newline at end of file
+logbemerkung = Bemerkung:
+
+fkarte = Karte
+fbeschreibung = Beschreibung
+fedit = Bearbeiten
+fname = Tourname
+fstrecke = Berechnete Strecke
+fdauer =  Berechnete Dauer
+fziel = Ziel
+fstart = Start
+favg = Durchschnittsgeschwindigkeit
+fbemerkung = Bemerkung
+fdatum = Datum
+fdauer = Dauer
+fgegangen = Davon gegangen
+fhight = Höhenmeter
+fpause = Davon Pause
+fstrecke = Strecke
+
+keinlogselected = Kein Logeintrag ausgewählt
+keinlogselectedmsg = Bitte zuerst einen Logeintrag auswählen
+
+fsuche = Suchen
+fopenmapsetting = Map automatisch öffnen
+fopenpdfsetting = Report automatisch öffnen
+flangenglisch = Englisch
+flangdeutsch = Deutsch
+fdatei = Datei
+fbearbeiten = Bearbeiten
+foptionen = Optionen
+fsprache = Sprache auswählen
+fhilfe = Hilfe
+freposum = Erstelle Report über alle Touren
+frepotour = Erstelle Report über aktuelle Tour
+fbeenden = Beenden
\ No newline at end of file
diff --git a/langen.ini b/langen.ini
index 320ce3b..072aef9 100644
--- a/langen.ini
+++ b/langen.ini
@@ -59,4 +59,39 @@ loggegangen = Gone away:
 logavg = Calculated averrage speed:
 logstrecke = Distance:
 loghight = Vertical meters:
-logbemerkung = Comment:
\ No newline at end of file
+logbemerkung = Comment:
+
+fkarte = Map
+fbeschreibung = Description
+fedit = Edit
+fname = Tourname
+fstrecke = Calculated Distance
+fdauer =  Calculated Duration
+fziel = Destination
+fstart = Start
+favg = Average speed
+fbemerkung = Comment
+fdatum = Date
+fdauer = Duration
+fgegangen = Gone away
+fhight = Vertical meters
+fpause = Break from that
+fstrecke = Distance
+
+keinlogselected = No log selected
+keinlogselectedmsg = Please select a log first!
+
+
+fsuche = Search
+fopenmapsetting = Open map automatic
+fopenpdfsetting = Open report automatic
+flangenglisch = English
+flangdeutsch = German
+fdatei = File
+fbearbeiten = Edit
+foptionen = Options
+fsprache = Choose Language
+fhilfe = Help
+freposum = Generate summarize-report
+frepotour = Generate tour-report
+fbeenden = Quit
\ No newline at end of file
diff --git a/src/tourplaner/tourplaner.fxml b/src/tourplaner/tourplaner.fxml
index 1d57ad6..7ec6177 100644
--- a/src/tourplaner/tourplaner.fxml
+++ b/src/tourplaner/tourplaner.fxml
@@ -45,22 +45,22 @@
       <menus>
         <Menu fx:id="menueFile" mnemonicParsing="false" text="Datei">
           <items>
-                  <MenuItem mnemonicParsing="false" onAction="#reportSum" text="Report Summary" />
-                  <MenuItem mnemonicParsing="false" onAction="#tourReport" text="Tour Report" />
+                  <MenuItem fx:id="reportsummary" mnemonicParsing="false" onAction="#reportSum" text="Report Summary" />
+                  <MenuItem fx:id="tourreport" mnemonicParsing="false" onAction="#tourReport" text="Tour Report" />
             <MenuItem fx:id="beendenButton" mnemonicParsing="false" onAction="#quitApp" text="Beenden" />
           </items>
         </Menu>
-        <Menu mnemonicParsing="false" text="Bearbeiten">
+        <Menu fx:id="menuebearbeiten" mnemonicParsing="false" text="Bearbeiten">
                <items>
-                  <MenuItem mnemonicParsing="false" onAction="#importBtn" text="Import" />
-                  <MenuItem mnemonicParsing="false" onAction="#exportBtn" text="Export" />
+                  <MenuItem fx:id="menueimport" mnemonicParsing="false" onAction="#importBtn" text="Import" />
+                  <MenuItem fx:id="menueexport" mnemonicParsing="false" onAction="#exportBtn" text="Export" />
                </items>
         </Menu>
-            <Menu mnemonicParsing="false" text="Optionen">
+            <Menu fx:id="menueoptionen" mnemonicParsing="false" text="Optionen">
                <items>
                   <CheckMenuItem fx:id="openmap" mnemonicParsing="false" onAction="#openmapaction" text="Map automatisch öffnen" />
                   <CheckMenuItem fx:id="openpdf" mnemonicParsing="false" onAction="#openpdfaction" text="Report automatisch öffnen" />
-                  <Menu mnemonicParsing="false" text="Sprache">
+                  <Menu fx:id="menuesprachen" mnemonicParsing="false" text="Sprache">
                     <items>
                         <RadioMenuItem fx:id="langdeutsch" mnemonicParsing="false" onAction="#onlangdeutsch" text="Deutsch" />
                         <RadioMenuItem fx:id="langenglisch" mnemonicParsing="false" onAction="#onlangenglisch" text="Englisch" />
@@ -68,7 +68,7 @@
                   </Menu>
                </items>
             </Menu>
-        <Menu mnemonicParsing="false" text="Hilfe">
+        <Menu fx:id="menuehilfe" mnemonicParsing="false" text="Hilfe">
           <items>
             <MenuItem mnemonicParsing="false" onAction="#gitWebBrowser" text="Git Repo" />
                   <MenuItem mnemonicParsing="false" onAction="#javaDocBrowser" text="JavaDoc" />
@@ -84,7 +84,7 @@
                   <Button fx:id="tourAdd" layoutX="58.0" mnemonicParsing="false" onAction="#addTour" text="+" />
                   <Label layoutX="14.0" layoutY="4.0" text="Tours" />
                   <Button fx:id="tourDel" layoutX="89.0" mnemonicParsing="false" onAction="#delTour" text="-" />
-                  <Button layoutX="117.0" mnemonicParsing="false" onAction="#editTourBtn" text="Edit" />
+                  <Button fx:id="editTourBtn" layoutX="117.0" mnemonicParsing="false" onAction="#editTourBtn" text="Edit" />
                </children></AnchorPane>
             <TextField fx:id="sucheInput" promptText="Suche..." />
             <Button fx:id="sucheButton" mnemonicParsing="false" onAction="#suche" text="Suchen" />
@@ -173,7 +173,7 @@
                                                 <Insets />
                                              </HBox.margin>
                                           </Button>
-                                          <Button mnemonicParsing="false" onAction="#editLogBtn" text="Edit" />
+                                          <Button fx:id="editBtn" mnemonicParsing="false" onAction="#editLogBtn" text="Edit" />
                                        </children>
                                        <padding>
                                           <Insets bottom="3.0" left="3.0" right="3.0" top="3.0" />
diff --git a/src/tourplaner/ui/TourplanerController.java b/src/tourplaner/ui/TourplanerController.java
index ae07c5d..a49a401 100644
--- a/src/tourplaner/ui/TourplanerController.java
+++ b/src/tourplaner/ui/TourplanerController.java
@@ -1,12 +1,10 @@
 package tourplaner.ui;
 
-import javafx.collections.ObservableList;
-import javafx.event.ActionEvent;
-import javafx.event.EventHandler;
 import javafx.fxml.FXML;
 import javafx.fxml.Initializable;
 import javafx.geometry.Orientation;
 import javafx.scene.control.*;
+import javafx.scene.control.Button;
 import javafx.scene.control.Label;
 import javafx.scene.control.TextField;
 import javafx.scene.control.cell.PropertyValueFactory;
@@ -14,21 +12,13 @@ import javafx.scene.image.ImageView;
 import javafx.scene.input.MouseEvent;
 import tourplaner.business.ConfigHelper;
 import tourplaner.business.LogHelper;
-import tourplaner.business.TourPlaner;
 import tourplaner.object.Log;
 import tourplaner.object.Tour;
 import tourplaner.viewmodels.ViewModel;
 
-import javax.swing.*;
-import java.awt.*;
 import java.io.IOException;
-import java.net.URI;
-import java.net.URISyntaxException;
 import java.net.URL;
-import java.time.LocalDate;
 import java.util.ResourceBundle;
-import java.util.UUID;
-import java.util.concurrent.atomic.AtomicReference;
 
 public class TourplanerController implements Initializable {
     //VM
@@ -48,16 +38,22 @@ public class TourplanerController implements Initializable {
     //Menue
     public CheckMenuItem openmap, openpdf;
     public RadioMenuItem langenglisch, langdeutsch;
-    public ToggleGroup langgroup;
+    private ToggleGroup langgroup;
+    public Menu menueFile, menuebearbeiten, menueoptionen, menuesprachen, menuehilfe;
+    public MenuItem reportsummary, tourreport, beendenButton;
+    //Elemente
+    public Button sucheButton, editBtn, editTourBtn;
 
     @FXML
     private void onlangenglisch(){
         ConfigHelper.setIniString(ConfigHelper.getStandartConfig(), "lang", "lang", "langen");
+        langToFxml();
     }
 
     @FXML
     private void onlangdeutsch(){
         ConfigHelper.setIniString(ConfigHelper.getStandartConfig(), "lang", "lang", "langde");
+        langToFxml();
     }
 
     @FXML
@@ -270,6 +266,42 @@ public class TourplanerController implements Initializable {
             this.langdeutsch.setSelected(false);
             this.langenglisch.setSelected(true);
         }
+
+        langToFxml();
+    }
+
+    private void langToFxml(){
+        this.kartenTab.setText(ConfigHelper.getLangIniString("fkarte"));
+        this.beschreibungTab.setText(ConfigHelper.getLangIniString("fbeschreibung"));
+        this.editBtn.setText(ConfigHelper.getLangIniString("fedit"));
+        this.editTourBtn.setText(ConfigHelper.getLangIniString("fedit"));
+        this.nameCol.setText(ConfigHelper.getLangIniString("fname"));
+        this.streckeCol.setText(ConfigHelper.getLangIniString("fstrecke"));
+        this.dauerCol.setText(ConfigHelper.getLangIniString("fdauer"));
+        this.zielCol.setText(ConfigHelper.getLangIniString("fziel"));
+        this.startCol .setText(ConfigHelper.getLangIniString("fstart"));
+        this.logAvgCol.setText(ConfigHelper.getLangIniString("favg"));
+        this.logBemerkungCol.setText(ConfigHelper.getLangIniString("fbemerkung"));
+        this.logDatumCol.setText(ConfigHelper.getLangIniString("fdatum"));
+        this.logDauerCol.setText(ConfigHelper.getLangIniString("fdauer"));
+        this.logGegangenCol.setText(ConfigHelper.getLangIniString("fgegangen"));
+        this.logHightCol.setText(ConfigHelper.getLangIniString("fhight"));
+        this.logPauseCol.setText(ConfigHelper.getLangIniString("fpause"));
+        this.logStreckeCol.setText(ConfigHelper.getLangIniString("fstrecke"));
+
+        this.sucheButton.setText(ConfigHelper.getLangIniString("fsuche"));
+        this.openmap.setText(ConfigHelper.getLangIniString("fopenmapsetting"));
+        this.openpdf.setText(ConfigHelper.getLangIniString("fopenpdfsetting"));
+        this.langenglisch.setText(ConfigHelper.getLangIniString("flangenglisch"));
+        this.langdeutsch.setText(ConfigHelper.getLangIniString("flangdeutsch"));
+        this.menueFile.setText(ConfigHelper.getLangIniString("fdatei"));
+        this.menuebearbeiten.setText(ConfigHelper.getLangIniString("fbearbeiten"));
+        this.menueoptionen.setText(ConfigHelper.getLangIniString("foptionen"));
+        this.menuesprachen.setText(ConfigHelper.getLangIniString("fsprache"));
+        this.menuehilfe.setText(ConfigHelper.getLangIniString("fhilfe"));
+        this.reportsummary.setText(ConfigHelper.getLangIniString("freposum"));
+        this.tourreport.setText(ConfigHelper.getLangIniString("frepotour"));
+        this.beendenButton.setText(ConfigHelper.getLangIniString("fbeenden"));
     }
 
     @FXML
diff --git a/src/tourplaner/viewmodels/ViewModel.java b/src/tourplaner/viewmodels/ViewModel.java
index 7db3d76..fbbe993 100644
--- a/src/tourplaner/viewmodels/ViewModel.java
+++ b/src/tourplaner/viewmodels/ViewModel.java
@@ -195,48 +195,59 @@ public class ViewModel {
      * Bearbeitet einen gewählten Log eintrag
      */
     public void editLog(){
-        AtomicReference<Tour> tourAkt = new AtomicReference<>();
-        AtomicReference<Log> logAkt = new AtomicReference<>();
-        this.tourData.forEach(tourd -> {
-            if(tourd.getName().equals(this.selectedTour.getName())) {
-                tourd.getLogs().forEach(log -> {
-                    if(this.selectedLog.getId().equals(log.getId())){
-                        logAkt.set(log);
-                        tourAkt.set(tourd);
+        System.out.println(this.selectedLog);
+        if(this.selectedTour != null) {
+            if(this.selectedLog != null) {
+                AtomicReference<Tour> tourAkt = new AtomicReference<>();
+                AtomicReference<Log> logAkt = new AtomicReference<>();
+                this.tourData.forEach(tourd -> {
+                    if (tourd.getName().equals(this.selectedTour.getName())) {
+                        tourd.getLogs().forEach(log -> {
+                            if (this.selectedLog.getId().equals(log.getId())) {
+                                logAkt.set(log);
+                                tourAkt.set(tourd);
+                            }
+                        });
                     }
                 });
-            }
-        });
-        double dauer = dauerInput(this.selectedLog.getDauer());
-        double strecke = -1;
-        double pause = -1.0;
-        double hightmeter = -1.0;
-        boolean bemerkung = false;
-        LocalDate datum = null;
-        if (dauer >= 0) {
-            strecke = streckeInput(this.selectedLog.getStrecke() + "");
-            if (strecke >= 0) {
-                pause = pauseInput(this.selectedLog.getPause() + "");
-                if(pause >= 0.0) {
-                    hightmeter = hightmeterInput(this.selectedLog.getHightmeter() + "");
-                    if (hightmeter >= 0.0) {
-                        bemerkung = bemerkungInput(this.selectedLog.getBemerkung());
-                        if (bemerkung) {
-                            datum = dateInput(this.selectedLog.getDatum());
-                            if (datum != null) {
-                                Log newLog = new Log(this.selectedLog.getId(), dauer, datum, strecke, this.neueBemerkung, hightmeter, pause);
-                                this.logData.removeIf(ld -> ld.getId().equals(this.selectedLog.getId()));
-                                this.logData.add(newLog);
+                double dauer = dauerInput(this.selectedLog.getDauer());
+                double strecke = -1;
+                double pause = -1.0;
+                double hightmeter = -1.0;
+                boolean bemerkung = false;
+                LocalDate datum = null;
+                if (dauer >= 0) {
+                    strecke = streckeInput(this.selectedLog.getStrecke() + "");
+                    if (strecke >= 0) {
+                        pause = pauseInput(this.selectedLog.getPause() + "");
+                        if (pause >= 0.0) {
+                            hightmeter = hightmeterInput(this.selectedLog.getHightmeter() + "");
+                            if (hightmeter >= 0.0) {
+                                bemerkung = bemerkungInput(this.selectedLog.getBemerkung());
+                                if (bemerkung) {
+                                    datum = dateInput(this.selectedLog.getDatum());
+                                    if (datum != null) {
+                                        Log newLog = new Log(this.selectedLog.getId(), dauer, datum, strecke, this.neueBemerkung, hightmeter, pause);
+                                        this.logData.removeIf(ld -> ld.getId().equals(this.selectedLog.getId()));
+                                        this.logData.add(newLog);
 //                                tourAkt.get().delLog(this.selectedLog.getId());
 //                                tourAkt.get().addLog(newLog);
-                                this.tourData.removeIf(td -> td.getName().equals(this.selectedTour.getName()));
-                                this.tourData.add(tourAkt.get());
-                                TourPlaner.editLog(this.selectedTour.getName(), newLog);
+                                        this.tourData.removeIf(td -> td.getName().equals(this.selectedTour.getName()));
+                                        this.tourData.add(tourAkt.get());
+                                        TourPlaner.editLog(this.selectedTour.getName(), newLog);
+                                    }
+                                }
                             }
                         }
                     }
                 }
+            }else{
+                //Kein log
+                AlertHelper.warn(ConfigHelper.getLangIniString("achtung"), ConfigHelper.getLangIniString("keinlogselected"), ConfigHelper.getLangIniString("keinlogselectedmsg"));
             }
+        }else {
+            //Keine Tour
+                AlertHelper.warn(ConfigHelper.getLangIniString("achtung"), ConfigHelper.getLangIniString("keinetourselected"), ConfigHelper.getLangIniString("deltournoselectmsg"));
         }
         this.neueBemerkung = null;
     }
@@ -407,7 +418,7 @@ public class ViewModel {
 
     public ObservableList<Log> getLogData() {
         logData.removeAll();
-        logData.addAll(TourPlaner.getLogs(this.selectedTour.getName()));
+        if(this.selectedLog != null) logData.addAll(TourPlaner.getLogs(this.selectedTour.getName()));
         return logData;
     }