From 134d7fedc8dc84ea5372ae5b8ab8fc1a759f574d Mon Sep 17 00:00:00 2001
From: Andreas Billmann <andreas.billmann@slab.de>
Date: Tue, 12 Jan 2016 21:58:27 +0100
Subject: [PATCH] enable save menu items only when csv file is loaded

---
 .../javafx/smartcsv/fx/SmartCSVController.java     | 14 ++++++++++++++
 .../ninja/javafx/smartcsv/fx/smartcsv.fxml         |  4 ++--
 2 files changed, 16 insertions(+), 2 deletions(-)

diff --git a/src/main/java/ninja/javafx/smartcsv/fx/SmartCSVController.java b/src/main/java/ninja/javafx/smartcsv/fx/SmartCSVController.java
index 8006548..93e891c 100644
--- a/src/main/java/ninja/javafx/smartcsv/fx/SmartCSVController.java
+++ b/src/main/java/ninja/javafx/smartcsv/fx/SmartCSVController.java
@@ -134,6 +134,12 @@ public class SmartCSVController extends FXMLController {
     @FXML
     private AnchorPane tableWrapper;
 
+    @FXML
+    private MenuItem saveMenuItem;
+
+    @FXML
+    private MenuItem saveAsMenuItem;
+
 
     ////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
     // members
@@ -190,6 +196,7 @@ public class SmartCSVController extends FXMLController {
                 "*.csv",
                 "Open CSV",
                 currentCsvFile);
+        enableSaveMenuItems();
         setCsvFileName();
     }
 
@@ -271,6 +278,13 @@ public class SmartCSVController extends FXMLController {
     // private methods
     ////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
 
+    private void enableSaveMenuItems() {
+        if (currentCsvFile != null) {
+            saveMenuItem.setDisable(false);
+            saveAsMenuItem.setDisable(false);
+        }
+    }
+
     private void setCsvFileName() {
         if (currentCsvFile != null) {
             csvName.setText(currentCsvFile.getName());
diff --git a/src/main/resources/ninja/javafx/smartcsv/fx/smartcsv.fxml b/src/main/resources/ninja/javafx/smartcsv/fx/smartcsv.fxml
index a54c1ba..c50ea75 100644
--- a/src/main/resources/ninja/javafx/smartcsv/fx/smartcsv.fxml
+++ b/src/main/resources/ninja/javafx/smartcsv/fx/smartcsv.fxml
@@ -29,12 +29,12 @@
                                     </graphic>
                                 </MenuItem>
                                 <SeparatorMenuItem mnemonicParsing="false"/>
-                                <MenuItem mnemonicParsing="false" onAction="#saveCsv" text="%menu.save">
+                                <MenuItem fx:id="saveMenuItem" mnemonicParsing="false" onAction="#saveCsv" text="%menu.save" disable="true">
                                     <graphic>
                                         <FontAwesomeIconView styleClass="save-icon"/>
                                     </graphic>
                                 </MenuItem>
-                                <MenuItem mnemonicParsing="false" onAction="#saveAsCsv" text="%menu.save.as">
+                                <MenuItem fx:id="saveAsMenuItem" mnemonicParsing="false" onAction="#saveAsCsv" text="%menu.save.as" disable="true">
                                     <graphic>
                                         <FontAwesomeIconView styleClass="save-icon"/>
                                     </graphic>