Changeset 16328 in josm for trunk/src/org/openstreetmap/josm/io

Timestamp:

2020-04-17T21:27:35+02:00 (4 years ago)

Author:

simon04

Message:

AddTagsDialog: use TextTagParser.readTagsByRegexp

Location:

trunk/src/org/openstreetmap/josm/io/remotecontrol

Files:

• AddTagsDialog.java (modified) (5 diffs)
• handler/LoadAndZoomHandler.java (modified) (1 diff)

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

@@ -11 +11 @@
 import java.awt.event.KeyEvent;
 import java.awt.event.MouseEvent;
-import java.util.Arrays;
 import java.util.Collection;
 import java.util.HashMap;
@@ -18 +17 @@
 import java.util.Map.Entry;
 import java.util.Set;
-import java.util.stream.Collectors;
 
 import javax.swing.AbstractAction;
@@ -38 +36 @@
 import org.openstreetmap.josm.gui.util.TableHelper;
 import org.openstreetmap.josm.tools.GBC;
+import org.openstreetmap.josm.tools.TextTagParser;
 
 /**
@@ -293 +292 @@
      * @since 15316
      */
-    public static String[][] parseUrlTagsToKeyValues(String urlSection) {
-        return Arrays.stream(urlSection.split("\\|"))
-                .map(String::trim)
-                .filter(tag -> !tag.isEmpty() && tag.contains("="))
-                .map(tag -> tag.split("\\s*=\\s*", 2))
-                .map(pair -> {
-                    pair[1] = pair.length < 2 ? "" : pair[1];
-                    return pair;
-                })
-                .collect(Collectors.toList()).toArray(new String[][] {});
+    public static Map<String, String> parseUrlTagsToKeyValues(String urlSection) {
+        return TextTagParser.readTagsByRegexp(urlSection, "\\|", "(.*?)=(.*?)", false);
     }
 
@@ -313 +304 @@
      */
     public static void addTags(String[][] keyValue, String sender, Collection<? extends OsmPrimitive> primitives) {
-        if (trustedSenders.contains(sender)) {
-            if (MainApplication.getLayerManager().getEditDataSet() != null) {
-                for (String[] row : keyValue) {
-                    UndoRedoHandler.getInstance().add(new ChangePropertyCommand(primitives, row[0], row[1]));
-                }
-            }
-        } else {
-            new AddTagsDialog(keyValue, sender, primitives).showDialog();
-        }
+        new AddTagsDialog(keyValue, sender, primitives).showDialog();
     }
 }

trunk/src/org/openstreetmap/josm/io/remotecontrol/handler/LoadAndZoomHandler.java

@@ -269 +269 @@
                 DataSet ds = MainApplication.getLayerManager().getEditDataSet();
                 if (ds != null) {
-                    for (String[] key : AddTagsDialog.parseUrlTagsToKeyValues(args.get("changeset_tags"))) {
-                        ds.addChangeSetTag(key[0], key[1]);
-                    }
+                    AddTagsDialog.parseUrlTagsToKeyValues(args.get("changeset_tags")).forEach(ds::addChangeSetTag);
                 }
             });