Changeset 8194 in josm for trunk/src/org/openstreetmap

Timestamp:

2015-04-14T23:10:19+02:00 (10 years ago)

Author:

simon04

Message:

Download objects: prefill ids from copied objects

The syntax from CopyAction is e.g. node 123 and has not been parsed yet.

Location:

trunk/src/org/openstreetmap/josm

Files:

• actions/CopyAction.java (modified) (2 diffs)
• data/osm/SimplePrimitiveId.java (modified) (1 diff)

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

@@ -57 +57 @@
     public static void copy(OsmDataLayer source, Collection<OsmPrimitive> primitives) {
         /* copy ids to the clipboard */
+        String ids = getCopyString(primitives);
+        Utils.copyToClipboard(ids);
+
+        Main.pasteBuffer.makeCopy(primitives);
+        Main.pasteSource = source;
+    }
+
+    public static String getCopyString(Collection<? extends OsmPrimitive> primitives) {
         StringBuilder idsBuilder = new StringBuilder();
         for (OsmPrimitive p : primitives) {
@@ -62 +70 @@
             idsBuilder.append(p.getId()).append(",");
         }
-        String ids = idsBuilder.substring(0, idsBuilder.length() - 1);
-        Utils.copyToClipboard(ids);
-
-        Main.pasteBuffer.makeCopy(primitives);
-        Main.pasteSource = source;
+        return idsBuilder.substring(0, idsBuilder.length() - 1);
     }
 

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

@@ -12 +12 @@
     private final OsmPrimitiveType type;
 
-    public static final Pattern ID_PATTERN = Pattern.compile("((n(ode)?|w(ay)?|r(el(ation)?)?)/?)(\\d+)");
+    public static final Pattern ID_PATTERN = Pattern.compile("((n(ode)?|w(ay)?|r(el(ation)?)?)[ /]?)(\\d+)");
 
     public SimplePrimitiveId(long id, OsmPrimitiveType type) {