Changeset 15363 in josm for trunk/src/org/openstreetmap/josm/actions

Timestamp:

2019-09-19T16:30:27+02:00 (5 years ago)

Author:

Don-vip

Message:

fix #18050 - DownloadAlong: download as new layer

Location:

trunk/src/org/openstreetmap/josm/actions

Files:

• DownloadAlongAction.java (modified) (6 diffs)
• DownloadAlongWayAction.java (modified) (2 diffs)

trunk/src/org/openstreetmap/josm/actions/DownloadAlongAction.java

@@ -123 +123 @@
      * @param gpxDownload Set to true if GPX data should be downloaded
      * @param title the title string for the confirmation dialog
-     */
-    protected static void confirmAndDownloadAreas(Area a, double maxArea, boolean osmDownload, boolean gpxDownload, String title) {
+     * @param newLayer Set to true if all areas should be put into a single new layer
+     */
+    protected static void confirmAndDownloadAreas(Area a, double maxArea, boolean osmDownload, boolean gpxDownload, String title,
+            boolean newLayer) {
         List<Rectangle2D> toDownload = new ArrayList<>();
         addToDownload(a, a.getBounds(), toDownload, maxArea);
@@ -140 +142 @@
         final PleaseWaitProgressMonitor monitor = new PleaseWaitProgressMonitor(tr("Download data"));
         final Future<?> future = new DownloadTaskList(Config.getPref().getBoolean("download.along.zoom-after-download"))
-                .download(false, toDownload, osmDownload, gpxDownload, monitor);
+                .download(newLayer, toDownload, osmDownload, gpxDownload, monitor);
         waitFuture(future, monitor);
     }
@@ -179 +181 @@
      * @param panel the panel that was displayed to the user and now contains his selections
      * @param confirmTitle the title to display in the confirmation panel
+     * @param newLayer Set to true if all areas should be put into a single new layer
      * @return the task or null if canceled by user
      */
-    protected PleaseWaitRunnable createCalcTask(Path2D alongPath, DownloadAlongPanel panel, String confirmTitle) {
+    protected PleaseWaitRunnable createCalcTask(Path2D alongPath, DownloadAlongPanel panel, String confirmTitle, boolean newLayer) {
         /*
          * Find the average latitude for the data we're contemplating, so we can know how many
@@ -221 +224 @@
 
             private final Path2D downloadPath = new Path2D.Double();
+            private final boolean newLayer;
             private boolean cancel;
             private int ticks;
             private final Rectangle2D r = new Rectangle2D.Double();
 
-            CalculateDownloadArea() {
+            CalculateDownloadArea(boolean newLayer) {
                 super(tr("Calculating Download Area"), displayProgress ? null : NullProgressMonitor.INSTANCE, false);
+                this.newLayer = newLayer;
             }
 
@@ -245 +250 @@
                 }
                 confirmAndDownloadAreas(new Area(downloadPath), maxArea, panel.isDownloadOsmData(), panel.isDownloadGpxData(),
-                        confirmTitle);
+                        confirmTitle, newLayer);
             }
 
@@ -291 +296 @@
         }
 
-        return new CalculateDownloadArea();
+        return new CalculateDownloadArea(newLayer);
     }
 

trunk/src/org/openstreetmap/josm/actions/DownloadAlongWayAction.java

@@ -54 +54 @@
                 PREF_DOWNLOAD_ALONG_WAY_DISTANCE, PREF_DOWNLOAD_ALONG_WAY_AREA, null);
 
-        if (0 != panel.showInDownloadDialog(tr("Download from OSM along selected ways"), HelpUtil.ht("/Tools/DownloadAlong"))) {
+        int ret = panel.showInDownloadDialog(tr("Download from OSM along selected ways"), HelpUtil.ht("/Tools/DownloadAlong"));
+        if (0 != ret && 1 != ret) {
             return null;
         }
@@ -71 +72 @@
             }
         }
-        return createCalcTask(alongPath, panel, tr("Download from OSM along selected ways"));
-
+        return createCalcTask(alongPath, panel, tr("Download from OSM along selected ways"), 1 == ret);
     }