Changeset 13534 in josm for trunk/src/org/openstreetmap/josm/gui/io/DownloadPrimitivesWithReferrersTask.java

Timestamp:

2018-03-17T01:16:09+01:00 (6 years ago)

Author:

Don-vip

Message:

fix #16096 - allow only positive ids in DownloadPrimitivesWithReferrersTask constructor

File:

• trunk/src/org/openstreetmap/josm/gui/io/DownloadPrimitivesWithReferrersTask.java (modified) (2 diffs)

trunk/src/org/openstreetmap/josm/gui/io/DownloadPrimitivesWithReferrersTask.java

@@ -8 +8 @@
 import java.awt.GridBagLayout;
 import java.io.IOException;
+import java.text.MessageFormat;
 import java.util.ArrayList;
 import java.util.HashSet;
@@ -76 +77 @@
         this.full = full;
         this.newLayer = newLayer;
+        // Check we don't try to download new primitives
+        for (PrimitiveId primitiveId : ids) {
+            if (primitiveId.isNew()) {
+                throw new IllegalArgumentException(MessageFormat.format(
+                        "Cannot download new primitives (ID {0})", primitiveId.getUniqueId()));
+            }
+        }
         // All downloaded primitives are put in a tmpLayer
         tmpLayer = new OsmDataLayer(new DataSet(), newLayerName != null ? newLayerName : OsmDataLayer.createNewName(), null);