Changeset 670 in josm

Timestamp:

Jun 30, 2008 8:16:11 PM (5 years ago)

Author:

framm

Message:

• patch to fix the unnecessary refresh of download dialog values. fixes #969. patch submitted by Dirk Stoecker <openstreetmap@…>.

File:

• trunk/src/org/openstreetmap/josm/gui/download/BoundingBoxSelection.java (modified) (3 diffs)

trunk/src/org/openstreetmap/josm/gui/download/BoundingBoxSelection.java

@@ -41 +41 @@
                         new JTextField(11) };
         final JTextArea osmUrl = new JTextArea();
+        String oldUrl;
         
         final JLabel sizeCheck = new JLabel();
@@ -82 +83 @@
                         public void removeUpdate(DocumentEvent e) { dowork(); }
                         private void dowork() {
-                                Bounds b = osmurl2bounds(osmUrl.getText());
-                                if (b != null) {
-                                        gui.minlon = b.min.lon();
-                                        gui.minlat = b.min.lat();
-                                        gui.maxlon = b.max.lon();
-                                        gui.maxlat = b.max.lat();
-                                        gui.boundingBoxChanged(BoundingBoxSelection.this);
-                                        updateBboxFields(gui);
-                                        updateSizeCheck(gui);
+                                if(!oldUrl.equals(osmUrl.getText()))
+                                {
+                                        Bounds b = osmurl2bounds(osmUrl.getText());
+                                        if (b != null) {
+                                                gui.minlon = b.min.lon();
+                                                gui.minlat = b.min.lat();
+                                                gui.maxlon = b.max.lon();
+                                                gui.maxlat = b.max.lat();
+                                                gui.boundingBoxChanged(BoundingBoxSelection.this);
+                                                updateBboxFields(gui);
+                                                updateSizeCheck(gui);
+                                        }
                                 }
                         }
@@ -163 +167 @@
                         zoom++;
                 }
-                osmUrl.setText("http://www.openstreetmap.org/index.html?mlat="+lat+"&mlon="+lon+"&zoom="+zoom);
+                // setting old URL prevents refresh based on this URL
+                oldUrl = "http://www.openstreetmap.org/index.html?mlat="+lat+"&mlon="+lon+"&zoom="+zoom;
+                osmUrl.setText(oldUrl);
         }