Opened 6 years ago

Last modified 6 years ago

#19269 closed defect

Adding tags via remote control broken — at Version 1

Reported by: johsin18 Owned by: simon04
Priority: major Milestone: 20.05
Component: Core remotecontrol Version: latest
Keywords: regression Cc: simon04

Description (last modified by simon04)

Adding tags via remote control is broken, e.g. in revision r16465.

When calling e.g.

/add_node?lon=13.3&lat=53.2&addtags=natural=tree|name=%20%20%20==Great%20Oak==

as mentioned in the help, the node is created at the right location, but the tags are not set, the tag dialog does not open.

This was broken by revision r16328, where AddTagsDialog.parseUrlTagsToKeyValues was changed to return Map<String, String> instead of String[][]. This makes AddTagsDialog.addTags(final Map<String, String> args,... call itself recursively, instead of calling AddTagsDialog.addTags(String[][] keyValue,... which would open the tags dialog.

While it is a good idea to use a Map for key/value paris principle, a Map does not keep the original order, as the formerly used array did. The order might well-chosen, but would get lost in the current state of the code.

Change History (1)

comment:1 by simon04, 6 years ago

Description: modified (diff)
Milestone: 20.05
Owner: changed from team to simon04
Status: newassigned
Note: See TracTickets for help on using tickets.