Changeset 13969 in josm for trunk/src

Timestamp:

2018-06-23T11:59:58+02:00 (6 years ago)

Author:

Don-vip

Message:

fix #16417 - Downloading primitive with id "0" causes IllegalArgumentException

Location:

trunk/src/org/openstreetmap/josm

Files:

• data/osm/SimplePrimitiveId.java (modified) (1 diff)
• gui/widgets/OsmIdTextField.java (modified) (1 diff)

trunk/src/org/openstreetmap/josm/data/osm/SimplePrimitiveId.java

@@ -133 +133 @@
             }
             for (long i = Long.parseLong(firstId); i <= lastIdParsed; i++) {
+                if (i > 0) {
+                    ids.add(new SimplePrimitiveId(i, type));
+                }
+            }
+        } else {
+            long i = Long.parseLong(firstId);
+            if (i > 0) {
                 ids.add(new SimplePrimitiveId(i, type));
             }
-        } else {
-            ids.add(new SimplePrimitiveId(Long.parseLong(firstId), type));
         }
         return ids;

trunk/src/org/openstreetmap/josm/gui/widgets/OsmIdTextField.java

@@ -129 +129 @@
                 }
             }
-            return true;
+            return !ids.isEmpty();
         }
     }