Skip to content
This repository has been archived by the owner on Jul 24, 2021. It is now read-only.

Moving tagged nodes in the applet looses the tags #369

Closed
openstreetmap-trac opened this issue Jul 23, 2021 · 3 comments
Closed

Moving tagged nodes in the applet looses the tags #369

openstreetmap-trac opened this issue Jul 23, 2021 · 3 comments

Comments

@openstreetmap-trac
Copy link

Reporter: pere[at]hungry.com
[Submitted to the original trac issue database at 11.03pm, Sunday, 28th January 2007]

I ran into a problem with the applet the other day, and was able to verify it
today. I've tagged a few nodes in josm, and used the applet to move them
into the correct location. First I moved the royal castle in Oslo, and next it
was a cinema. When the node is moved in the applet, josm is no longer able
to find any tags on the node. I checked in the applet after the move if the
tags were still present. They were there. But when I looked at the same map
area using jsom later, the tags were gone.

This make it very dangerous to move nodes in the applet.

@openstreetmap-trac
Copy link
Author

Author: tom[at]compton.nu
[Added to the original trac issue at 11.55am, Wednesday, 7th February 2007]

If the output on the Java console accurately reflects the PUT request that is sent to the server then the problem is that the wrong format seems to be being used for the tags in the XML that is sent - see this example:

Trying to PUT xml "" to URL http://www.openstreetmap.org/api/0.3/node/18338210

@openstreetmap-trac
Copy link
Author

Author: writetodan[at]yahoo.com
[Added to the original trac issue at 12.56am, Friday, 23rd February 2007]

confirmed from code, .../client/Adapter.java, culprit:

   String xml = "<osm><node tags=\"" + node.getTags() + "\" lon=\""
     + node.coor.lon + "\" lat=\"" + node.coor.lat + "\" id=\""
     + node.id + "\" /></osm>";

... the tags format we're seeing in the request is default Hashtable toString() implementation. am checking in a load of other fixes, i'll add this one in as well once i have tested.

@openstreetmap-trac
Copy link
Author

Author: writetodan[at]yahoo.com
[Added to the original trac issue at 8.53pm, Wednesday, 14th March 2007]

fix deployed to www

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant