Hello,

to fix bug http://josm.openstreetmap.de/ticket/836

I need to add osm/gpx file extension seperation (first three files). To 
make more understandable, that a GPX is written instead of the normal data 
I introduced a dialog title (other files).

Objections to this? If not I will check it in next days.

Index: src/org/openstreetmap/josm/actions/SaveActionBase.java
===================================================================
--- src/org/openstreetmap/josm/actions/SaveActionBase.java      (Revision 681)
+++ src/org/openstreetmap/josm/actions/SaveActionBase.java      (Arbeitskopie)
@@ -86,8 +86,8 @@
                return true;
        }

-       public static File openFileDialog() {
-               JFileChooser fc = createAndOpenFileChooser(false, false);
+       public static File openFileDialog(Layer layer) {
+               JFileChooser fc = createAndOpenFileChooser(false, false, layer 
instanceof GpxLayer ? tr("Save GPX file") : tr("Save OSM file"));
                if (fc == null)
                        return null;

@@ -98,6 +98,8 @@
                        FileFilter ff = fc.getFileFilter();
                        if (ff instanceof ExtensionFileFilter)
                                fn += "." + 
((ExtensionFileFilter)ff).defaultExtension;
+                       else if (layer instanceof GpxLayer)
+                               fn += ".gpx";
                        else
                                fn += ".osm";
                        file = new File(fn);
Index: src/org/openstreetmap/josm/actions/SaveAction.java
===================================================================
--- src/org/openstreetmap/josm/actions/SaveAction.java  (Revision 681)
+++ src/org/openstreetmap/josm/actions/SaveAction.java  (Arbeitskopie)
@@ -39,6 +39,6 @@
                                return f;
                        }
                }
-               return openFileDialog();
+               return openFileDialog(layer);
        }
  }
Index: src/org/openstreetmap/josm/actions/SaveAsAction.java
===================================================================
--- src/org/openstreetmap/josm/actions/SaveAsAction.java        (Revision 681)
+++ src/org/openstreetmap/josm/actions/SaveAsAction.java        (Arbeitskopie)
@@ -25,6 +25,6 @@
        }

        @Override protected File getFile(Layer layer) {
-               return openFileDialog();
+               return openFileDialog(layer);
        }
  }
Index: src/org/openstreetmap/josm/actions/DiskAccessAction.java
===================================================================
--- src/org/openstreetmap/josm/actions/DiskAccessAction.java    (Revision 681)
+++ src/org/openstreetmap/josm/actions/DiskAccessAction.java    (Arbeitskopie)
@@ -19,11 +19,14 @@
                super(name, iconName, tooltip, shortCut, modifiers, true);
        }

-       protected static JFileChooser createAndOpenFileChooser(boolean open, 
boolean multiple) {
+       protected static JFileChooser createAndOpenFileChooser(boolean open, 
boolean multiple, String title) {
                String curDir = Main.pref.get("lastDirectory");
                if (curDir.equals(""))
                        curDir = ".";
                JFileChooser fc = new JFileChooser(new File(curDir));
+               if(title != null)
+                       fc.setDialogTitle(title);
+
                fc.setMultiSelectionEnabled(multiple);
                for (int i = 0; i < ExtensionFileFilter.filters.length; ++i)
                        
fc.addChoosableFileFilter(ExtensionFileFilter.filters[i]);
Index: src/org/openstreetmap/josm/actions/GpxExportAction.java
===================================================================
--- src/org/openstreetmap/josm/actions/GpxExportAction.java     (Revision 681)
+++ src/org/openstreetmap/josm/actions/GpxExportAction.java     (Arbeitskopie)
@@ -54,7 +54,7 @@
                        return;
                }

-               JFileChooser fc = createAndOpenFileChooser(false, false);
+               JFileChooser fc = createAndOpenFileChooser(false, false, null);
                if (fc == null)
                        return;
                File file = fc.getSelectedFile();
Index: src/org/openstreetmap/josm/actions/OpenAction.java
===================================================================
--- src/org/openstreetmap/josm/actions/OpenAction.java  (Revision 681)
+++ src/org/openstreetmap/josm/actions/OpenAction.java  (Arbeitskopie)
@@ -40,7 +40,7 @@
        }

        public void actionPerformed(ActionEvent e) {
-               JFileChooser fc = createAndOpenFileChooser(true, true);
+               JFileChooser fc = createAndOpenFileChooser(true, true, null);
                if (fc == null)
                        return;
                File[] files = fc.getSelectedFiles();

Ciao
-- 
http://www.dstoecker.eu/ (PGP key available)

_______________________________________________
josm-dev mailing list
[EMAIL PROTECTED]
http://lists.openstreetmap.org/cgi-bin/mailman/listinfo/josm-dev

Reply via email to