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

Timestamp:

2026-05-07T15:11:23+02:00 (4 days ago)

Author:

stoecker

Message:

fix #14490 - patch by ToniE - Support for escaping pipe character in remote control addtags parameters

Location:

trunk/src/org/openstreetmap/josm

Files:

• io/remotecontrol/AddTagsDialog.java (modified) (1 diff)
• tools/TextTagParser.java (modified) (5 diffs)

trunk/src/org/openstreetmap/josm/io/remotecontrol/AddTagsDialog.java

@@ -284 +284 @@
      */
     public static Map<String, String> parseUrlTagsToKeyValues(String urlSection) {
-        Map<String, String> tags = TextTagParser.readTagsByRegexp(urlSection, "\\|", "(.*?)=(.*?)", false);
+        Map<String, String> tags = TextTagParser.readTagsByRegexp(urlSection, "(?<!\\\\)\\|", "(.*?)=(.*?)", false, true);
         return tags == null ? Collections.emptyMap() : tags;
     }

trunk/src/org/openstreetmap/josm/tools/TextTagParser.java

@@ -47 +47 @@
      * @param tagRegex - each part is matched against this regex
      * @param unescapeTextInQuotes - if true, matched tag and value will be analyzed more thoroughly
+     * @param unescapePipe - if true, then replace all '\|' by '|' (issue #14490)
      * @return map of tags
+     * @since 19570 (parameter unescapePipe added)
      */
-    public static Map<String, String> readTagsByRegexp(String text, String splitRegex, String tagRegex, boolean unescapeTextInQuotes) {
-         String[] lines = text.split(splitRegex, -1);
-         Pattern p = Pattern.compile(tagRegex);
-         Map<String, String> tags = new LinkedHashMap<>();
-         String k;
-         String v;
-         for (String line: lines) {
+    public static Map<String, String> readTagsByRegexp(String text, String splitRegex, String tagRegex, 
+            boolean unescapeTextInQuotes, boolean unescapePipe) {
+        String[] lines = text.split(splitRegex, -1);
+        Pattern p = Pattern.compile(tagRegex);
+        Map<String, String> tags = new LinkedHashMap<>();
+        String k;
+        String v;
+        for (String line: lines) {
             if (line.trim().isEmpty()) continue; // skip empty lines
             Matcher m = p.matcher(line);
@@ -66 +69 @@
                      if (k == null || v == null) return null;
                  }
+                 if (unescapePipe) {
+                     k = k.replaceAll("\\\\\\|", "|");
+                     v = v.replaceAll("\\\\\\|", "|");
+                 }
                  tags.put(k, v);
             } else {
                 return null;
             }
-         }
-         if (!tags.isEmpty()) {
-            return tags;
-         } else {
-            return null;
-         }
+        }
+        if (!tags.isEmpty()) {
+           return tags;
+        } else {
+           return null;
+        }
     }
 
@@ -100 +107 @@
         // Format
         // tag1\tval1\ntag2\tval2\n
-        tags = readTagsByRegexp(buf, "[\\r\\n]+", ".*?([a-zA-Z0-9:_]+).*\\t(.*?)", false);
+        tags = readTagsByRegexp(buf, "[\\r\\n]+", ".*?([a-zA-Z0-9:_]+).*\\t(.*?)", false, false);
         // try "tag\tvalue\n" format
         if (tags != null) return tags;
@@ -109 +116 @@
         // a = "b=c" is OK
         // a = b=c  - this method of parsing fails intentionally
-        tags = readTagsByRegexp(buf, "[\\n\\t\\r]+", "(.*?)=(.*?)", true);
+        tags = readTagsByRegexp(buf, "[\\n\\t\\r]+", "(.*?)=(.*?)", true, false);
         // try format  t1=v1\n t2=v2\n ...
         if (tags != null) return tags;
@@ -119 +126 @@
             bufJson = bufJson.substring(1, bufJson.length()-1);
         tags = readTagsByRegexp(bufJson, "[\\s]*,[\\s]*",
-                "[\\s]*(\\\".*?[^\\\\]\\\")"+"[\\s]*:[\\s]*"+"(\\\".*?[^\\\\]\\\")[\\s]*", true);
+                "[\\s]*(\\\".*?[^\\\\]\\\")"+"[\\s]*:[\\s]*"+"(\\\".*?[^\\\\]\\\")[\\s]*", true, false);
         if (tags != null) return tags;