Ticket #11505: 0001-Load-a-range-of-OSM-objects-using-a-hyphen-as-a-sepa.patch

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

@@ -16 +16 @@
 
     public static final Pattern ID_PATTERN = Pattern.compile("((n(ode)?|w(ay)?|r(el(ation)?)?)[ /]?)(\\d+)");
 
+    public static final Pattern MULTIPLE_IDS_PATTERN = Pattern.compile("((n(ode)?|w(ay)?|r(el(ation)?)?)/?)(\\d+)(-(\\d+))?");
+
     public SimplePrimitiveId(long id, OsmPrimitiveType type) {
         this.id = id;
         this.type = type;
@@ -97 +99 @@
      */
     public static List<SimplePrimitiveId> fuzzyParse(String s) {
         final List<SimplePrimitiveId> ids = new ArrayList<>();
-        final Matcher m = ID_PATTERN.matcher(s);
+        final Matcher m = MULTIPLE_IDS_PATTERN.matcher(s);
         while (m.find()) {
             final char firstChar = s.charAt(m.start());
-            ids.add(new SimplePrimitiveId(Long.parseLong(m.group(m.groupCount())),
-                    firstChar == 'n'
+            if (null != m.group(m.groupCount())) {
+                for(long r = Long.parseLong(m.group(m.groupCount())) - 2; r <= Long.parseLong(m.group(m.groupCount())); r ++) {
+                    ids.add(new SimplePrimitiveId(r,
+                            firstChar == 'n'
                             ? OsmPrimitiveType.NODE
                             : firstChar == 'w'
                             ? OsmPrimitiveType.WAY
                             : OsmPrimitiveType.RELATION));
+                }
+            } else {
+                ids.add(new SimplePrimitiveId(Long.parseLong(m.group(m.groupCount())) - 2,
+                        firstChar == 'n'
+                        ? OsmPrimitiveType.NODE
+                        : firstChar == 'w'
+                        ? OsmPrimitiveType.WAY
+                        : OsmPrimitiveType.RELATION));
+            }
         }
         return ids;
     }

src/org/openstreetmap/josm/gui/dialogs/OsmIdSelectionDialog.java

@@ -93 +93 @@
         HtmlPanel help = new HtmlPanel(/* I18n: {0} and {1} contains example strings not meant for translation. {2}=n, {3}=w, {4}=r. */
                 tr("Object IDs can be separated by comma or space.<br/>"
                         + "Examples: {0}<br/>"
-                        + "In mixed mode, specify objects like this: {1}<br/>"
-                        + "({2} stands for <i>node</i>, {3} for <i>way</i>, and {4} for <i>relation</i>)",
+                        + "In mixed mode, specify objects like this: {1}<br/><br/>"
+                        + "Ranges of object IDs can also be catched, by a hyphen.<br/>"
+                        + "Examples: {2}<br/>"
+                        + "({3} stands for <i>node</i>, {4} for <i>way</i>, and {5} for <i>relation</i>)",
                         "<b>" + Utils.joinAsHtmlUnorderedList(Arrays.asList("1 2 5", "1,2,5")) + "</b>",
                         "<b>w123, n110, w12, r15</b>",
+                        "<b>" + Utils.joinAsHtmlUnorderedList(Arrays.asList("w1-5", "n1-7")) + "</b>",
                         "<b>n</b>", "<b>w</b>", "<b>r</b>"
                 ));
         help.setBorder(BorderFactory.createEtchedBorder(EtchedBorder.LOWERED));

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

@@ -122 +122 @@
                                 return false;
                             }
                         } catch (IllegalArgumentException ex2) {
-                            return false;
+                            try {
+                                ids.addAll(SimplePrimitiveId.fuzzyParse(s));
+                            } catch (IllegalArgumentException ex3) {
+                                return false;
+                            }
                         }
                     }
                 }

test/unit/org/openstreetmap/josm/data/osm/SimplePrimitiveIdTest.groovy

@@ -33 +33 @@
         assert SimplePrimitiveId.fromString("way 123") == new SimplePrimitiveId(123, OsmPrimitiveType.WAY)
         assert SimplePrimitiveId.fromString("relation 123") == new SimplePrimitiveId(123, OsmPrimitiveType.RELATION)
     }
+
+    void testMultipleIDs() {
+        assert SimplePrimitiveId.fuzzyParse("node/123-125").toString() == "[node 123, node 124, node 125]"
+        assert SimplePrimitiveId.fuzzyParse("n/123-125").toString() == "[node 123, node 124, node 125]"
+        assert SimplePrimitiveId.fuzzyParse("node123-125").toString() == "[node 123, node 124, node 125]"
+        assert SimplePrimitiveId.fuzzyParse("way/123-125").toString() == "[way 123, way 124, way 125]"
+        assert SimplePrimitiveId.fuzzyParse("w/123-125").toString() == "[way 123, way 124, way 125]"
+        assert SimplePrimitiveId.fuzzyParse("way123-125").toString() == "[way 123, way 124, way 125]"
+        assert SimplePrimitiveId.fuzzyParse("relation/123-125").toString() == "[relation 123, relation 124, relation 125]"
+        assert SimplePrimitiveId.fuzzyParse("r/123-125").toString() == "[relation 123, relation 124, relation 125]"
+        assert SimplePrimitiveId.fuzzyParse("relation123-125").toString() == "[relation 123, relation 124, relation 125]"
+    }
 }