Changeset 18017 in josm

Timestamp:

2021-07-14T16:37:05+02:00 (3 years ago)

Author:

Don-vip

Message:

fix #21034 - remember download OSM bounds from DownloadOsmTask instead of DownloadDialog

Location:

trunk/src/org/openstreetmap/josm

Files:

• actions/downloadtasks/DownloadOsmTask.java (modified) (3 diffs)
• gui/download/DownloadDialog.java (modified) (1 diff)

trunk/src/org/openstreetmap/josm/actions/downloadtasks/DownloadOsmTask.java

@@ -45 +45 @@
 import org.openstreetmap.josm.io.OverpassDownloadReader;
 import org.openstreetmap.josm.io.UrlPatterns.OsmUrlPattern;
+import org.openstreetmap.josm.spi.preferences.Config;
 import org.openstreetmap.josm.tools.Logging;
 import org.openstreetmap.josm.tools.Utils;
@@ -214 +215 @@
     }
 
+    protected final void rememberDownloadedBounds(Bounds bounds) {
+        if (bounds != null) {
+            Config.getPref().put("osm-download.bounds", bounds.encodeAsString(";"));
+        }
+    }
+
     /**
      * Superclass of internal download task.
@@ -506 +513 @@
             }
 
+            rememberDownloadedBounds(currentBounds);
             rememberDownloadedData(dataSet);
             loadData(newLayerName, currentBounds);

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

@@ -440 +440 @@
         downloadSourcesTab.getSelectedPanel().ifPresent(panel -> DOWNLOAD_SOURCE_TAB.put(panel.getSimpleName()));
         DOWNLOAD_ZOOMTODATA.put(cbZoomToDownloadedData.isSelected());
-        if (currentBounds != null) {
-            Config.getPref().put("osm-download.bounds", currentBounds.encodeAsString(";"));
-        }
     }