Ticket #21256: 21256.patch

File 21256.patch, 1.9 KB (added by taylor.smock, 4 years ago)

RemoteControl: Don't add empty changeset tags to the dataset

  • src/org/openstreetmap/josm/data/osm/DataSet.java

    diff --git a/src/org/openstreetmap/josm/data/osm/DataSet.java b/src/org/openstreetmap/josm/data/osm/DataSet.java
    index 9aead6d346..e9a4fb6157 100644
    a b public final class DataSet implements OsmData<OsmPrimitive, Node, Way, Relation>  
    387387     * @see #getChangeSetTags
    388388     */
    389389    public void addChangeSetTag(String k, String v) {
    390         this.changeSetTags.put(k, v);
     390        if (v != null) {
     391            this.changeSetTags.put(k, v);
     392        } else {
     393            this.changeSetTags.remove(k);
     394        }
    391395    }
    392396
    393397    @Override
  • src/org/openstreetmap/josm/io/remotecontrol/handler/LoadAndZoomHandler.java

    diff --git a/src/org/openstreetmap/josm/io/remotecontrol/handler/LoadAndZoomHandler.java b/src/org/openstreetmap/josm/io/remotecontrol/handler/LoadAndZoomHandler.java
    index 940eba1215..16cb3974d5 100644
    a b public class LoadAndZoomHandler extends RequestHandler {  
    254254                if (ds != null) {
    255255                    for (String tag : Arrays.asList("changeset_comment", "changeset_source", "changeset_hashtags")) {
    256256                        if (args.containsKey(tag)) {
    257                             ds.addChangeSetTag(tag.substring("changeset_".length()), args.get(tag));
     257                            final String tagKey = tag.substring("changeset_".length());
     258                            final String value = args.get(tag);
     259                            if (!Utils.isStripEmpty(value)) {
     260                                ds.addChangeSetTag(tagKey, value);
     261                            } else {
     262                                ds.addChangeSetTag(tagKey, null);
     263                            }
    258264                        }
    259265                    }
    260266                }