From c98856a634c087e30aa790a47524c3f1e9150dfc Mon Sep 17 00:00:00 2001
From: ondrejm <ondrej.mihalyi@gmail.com>
Date: Fri, 15 Apr 2016 11:49:23 +0200
Subject: [PATCH 1/2] Export file dialog shows directories. Small refactoring.

---
 .../perf/gcviewer/ctrl/action/Export.java     | 50 +++++++++----------
 .../view/util/ExtensionFileFilter.java        |  2 +-
 2 files changed, 26 insertions(+), 26 deletions(-)

diff --git a/src/main/java/com/tagtraum/perf/gcviewer/ctrl/action/Export.java b/src/main/java/com/tagtraum/perf/gcviewer/ctrl/action/Export.java
index 2dc5cf2c..4f3eedaf 100644
--- a/src/main/java/com/tagtraum/perf/gcviewer/ctrl/action/Export.java
+++ b/src/main/java/com/tagtraum/perf/gcviewer/ctrl/action/Export.java
@@ -19,6 +19,7 @@
 import com.tagtraum.perf.gcviewer.view.ChartPanelView;
 import com.tagtraum.perf.gcviewer.view.GCDocument;
 import com.tagtraum.perf.gcviewer.view.GCViewerGui;
+import com.tagtraum.perf.gcviewer.view.util.ExtensionFileFilter;
 import com.tagtraum.perf.gcviewer.view.util.ImageHelper;
 
 /**
@@ -27,6 +28,7 @@
  * Time: 2:01:07 PM
  */
 public class Export extends AbstractAction {
+
     private GCViewerGui gcViewer;
     private JFileChooser saveDialog;
 
@@ -43,14 +45,12 @@ public Export(final GCViewerGui gcViewer) {
         saveDialog = new JFileChooser();
         saveDialog.setDialogTitle(LocalisationHelper.getString("fileexport_dialog_title"));
         saveDialog.removeChoosableFileFilter(saveDialog.getAcceptAllFileFilter());
-        saveDialog.addChoosableFileFilter(new ExtensionFileFilter(".csv", LocalisationHelper.getString("fileexport_dialog_csv"), DataWriterType.CSV));
-        saveDialog.addChoosableFileFilter(new ExtensionFileFilter(".csv", LocalisationHelper.getString("fileexport_dialog_csv_ts"), DataWriterType.CSV_TS));
-        saveDialog.addChoosableFileFilter(new ExtensionFileFilter(".txt", LocalisationHelper.getString("fileexport_dialog_txt"), DataWriterType.PLAIN));
-        saveDialog.addChoosableFileFilter(new ExtensionFileFilter(".simple.log", LocalisationHelper.getString("fileexport_dialog_simplelog"), DataWriterType.SIMPLE));
-        saveDialog.addChoosableFileFilter(new ExtensionFileFilter(".csv", LocalisationHelper.getString("fileexport_dialog_summarylog"), DataWriterType.SUMMARY));
-        saveDialog.addChoosableFileFilter(new ExtensionFileFilter(".png", LocalisationHelper.getString("fileexport_dialog_png"), DataWriterType.PNG));
+        for (ExportExtensionFileFilter filter : ExportExtensionFileFilter.EXT_FILE_FILTERS) {
+            saveDialog.addChoosableFileFilter(filter);
         }
+    }
 
+    @Override
     public void actionPerformed(final ActionEvent e) {
         final GCDocument gcDocument = gcViewer.getSelectedGCDocument();
         for (int i=0; i<gcDocument.getChartPanelViewCount(); i++) {
@@ -60,10 +60,10 @@ public void actionPerformed(final ActionEvent e) {
             saveDialog.setSelectedFile(file);
             final int val = saveDialog.showSaveDialog(gcViewer);
             if (val == JFileChooser.APPROVE_OPTION) {
-                ExtensionFileFilter fileFilter = (ExtensionFileFilter) saveDialog.getFileFilter();
+                ExportExtensionFileFilter fileFilter = (ExportExtensionFileFilter) saveDialog.getFileFilter();
                 // On OS/X if you don't select one of the filters and just press "Save" the filter may be null. Use the CSV one then
                 if (fileFilter==null) {
-                    fileFilter = (ExtensionFileFilter) saveDialog.getChoosableFileFilters()[0];
+                    fileFilter = (ExportExtensionFileFilter) saveDialog.getChoosableFileFilters()[0];
                 }
                 exportFile(chartPanelView.getGCResource().getModel(),
                         saveDialog.getSelectedFile(), 
@@ -96,31 +96,31 @@ public void exportFile(final GCModel model, File file, final String extension, f
         }
     }
 
-    private static class ExtensionFileFilter extends FileFilter {
-        private String extension;
-        private String description;
-        private DataWriterType dataWriterType;
+    private static class ExportExtensionFileFilter extends ExtensionFileFilter {
+
+        public static final ExportExtensionFileFilter[] EXT_FILE_FILTERS = {
+            new ExportExtensionFileFilter(".csv", LocalisationHelper.getString("fileexport_dialog_csv"), DataWriterType.CSV),
+            new ExportExtensionFileFilter(".csv", LocalisationHelper.getString("fileexport_dialog_csv_ts"), DataWriterType.CSV_TS),
+            new ExportExtensionFileFilter(".txt", LocalisationHelper.getString("fileexport_dialog_txt"), DataWriterType.PLAIN),
+            new ExportExtensionFileFilter(".simple.log", LocalisationHelper.getString("fileexport_dialog_simplelog"), DataWriterType.SIMPLE),
+            new ExportExtensionFileFilter(".csv", LocalisationHelper.getString("fileexport_dialog_summarylog"), DataWriterType.SUMMARY),
+            new ExportExtensionFileFilter(".png", LocalisationHelper.getString("fileexport_dialog_png"), DataWriterType.PNG)        
+        };
+
+        private final String description;
+        private final DataWriterType dataWriterType;
 
-        public ExtensionFileFilter(final String extension, final String description, final DataWriterType dataWriterType) {
-            this.extension = extension.toLowerCase();
+        public ExportExtensionFileFilter(final String extension, final String description, final DataWriterType dataWriterType) {
+            super(extension.toLowerCase());
             this.description = description;
             this.dataWriterType = dataWriterType;
         }
 
-        public boolean accept(final File file) {
-        	// TODO refactor
-        	try {
-                return file.toString().toLowerCase().endsWith(extension);
-        	}
-        	catch (NullPointerException e) {
-        		return false;
-        	}
-        }
-
         public String getExtension() {
-            return extension;
+            return super.extension;
         }
 
+        @Override
         public String getDescription() {
             return description;
         }
diff --git a/src/main/java/com/tagtraum/perf/gcviewer/view/util/ExtensionFileFilter.java b/src/main/java/com/tagtraum/perf/gcviewer/view/util/ExtensionFileFilter.java
index 5e9b5e15..6be46604 100644
--- a/src/main/java/com/tagtraum/perf/gcviewer/view/util/ExtensionFileFilter.java
+++ b/src/main/java/com/tagtraum/perf/gcviewer/view/util/ExtensionFileFilter.java
@@ -15,7 +15,7 @@ public class ExtensionFileFilter extends FileFilter {
         new ExtensionFileFilter("txt.gz"), new ExtensionFileFilter("gc.gz"), new ExtensionFileFilter("log.gz")
     };
 
-    private final String extension;
+    protected final String extension;
 
     public ExtensionFileFilter(String extension) {
         this.extension = extension;

From 3079c554699f26137aeaa2dbc060c2145fe0ac43 Mon Sep 17 00:00:00 2001
From: ondrejm <ondrej.mihalyi@gmail.com>
Date: Sun, 17 Apr 2016 16:05:17 +0200
Subject: [PATCH 2/2] Fix of filtering on file extension in export dialog.

---
 .../tagtraum/perf/gcviewer/ctrl/action/Export.java   | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/src/main/java/com/tagtraum/perf/gcviewer/ctrl/action/Export.java b/src/main/java/com/tagtraum/perf/gcviewer/ctrl/action/Export.java
index 4f3eedaf..4ad89915 100644
--- a/src/main/java/com/tagtraum/perf/gcviewer/ctrl/action/Export.java
+++ b/src/main/java/com/tagtraum/perf/gcviewer/ctrl/action/Export.java
@@ -99,12 +99,12 @@ public void exportFile(final GCModel model, File file, final String extension, f
     private static class ExportExtensionFileFilter extends ExtensionFileFilter {
 
         public static final ExportExtensionFileFilter[] EXT_FILE_FILTERS = {
-            new ExportExtensionFileFilter(".csv", LocalisationHelper.getString("fileexport_dialog_csv"), DataWriterType.CSV),
-            new ExportExtensionFileFilter(".csv", LocalisationHelper.getString("fileexport_dialog_csv_ts"), DataWriterType.CSV_TS),
-            new ExportExtensionFileFilter(".txt", LocalisationHelper.getString("fileexport_dialog_txt"), DataWriterType.PLAIN),
-            new ExportExtensionFileFilter(".simple.log", LocalisationHelper.getString("fileexport_dialog_simplelog"), DataWriterType.SIMPLE),
-            new ExportExtensionFileFilter(".csv", LocalisationHelper.getString("fileexport_dialog_summarylog"), DataWriterType.SUMMARY),
-            new ExportExtensionFileFilter(".png", LocalisationHelper.getString("fileexport_dialog_png"), DataWriterType.PNG)        
+            new ExportExtensionFileFilter("csv", LocalisationHelper.getString("fileexport_dialog_csv"), DataWriterType.CSV),
+            new ExportExtensionFileFilter("csv", LocalisationHelper.getString("fileexport_dialog_csv_ts"), DataWriterType.CSV_TS),
+            new ExportExtensionFileFilter("txt", LocalisationHelper.getString("fileexport_dialog_txt"), DataWriterType.PLAIN),
+            new ExportExtensionFileFilter("simple.log", LocalisationHelper.getString("fileexport_dialog_simplelog"), DataWriterType.SIMPLE),
+            new ExportExtensionFileFilter("csv", LocalisationHelper.getString("fileexport_dialog_summarylog"), DataWriterType.SUMMARY),
+            new ExportExtensionFileFilter("png", LocalisationHelper.getString("fileexport_dialog_png"), DataWriterType.PNG)        
         };
 
         private final String description;