Prevent modified status even if no real modification has been done

[anonymous]

Sometimes there are duplicate nodes in the map, however, if I merge two duplicated nodes (that are not used by anything else, like POIs), this should result in one of the nodes being deleted (and the other kept), however, the result is that one of the nodes is deleted and the other is flagged as modified, so it gets uploaded to server even when it have not changed at all.

This happend both when merging the nodes manually and when merging through validator ("fix" button on listing of duplicated nodes)

Attached is example of JOSM behavior:

dupnodefix-before.osm - situation before doing anything. Two identical nodes at same position
dupnodefix-after.osm - situation after fixing (merging nodes) - note that one of the nodes have action='delete', second have action='modify'

The result should be that only one of the nodes gets deleted and the other is kept intact.

[anonymous]

before merging

[anonymous]

after merging

[stoecker]

Actually that is not restricted to nodes, but to all objects in JOSM. The node is actually changed, as it contains the joined tags of both nodes.

JOSM misses a way to detect changes which actually do not change the object at all. This also applies to changes which have been done and later reversed.

[anonymous]

While this can happen also in other ways (some value changed to something and later back), the problem when merging nodes is probably the most common, as nodes usually have same values when merging. Therefore, the merge could check if keys are the same and if they are, donot modify the node (and just drop the others).

I've done small patch that does that, attaching it here...

[anonymous]

Patch to not modify the node while merging if it is not modified

[Gubaer]

(In [2064]) applied #3384: patch by anonymous: Prevent modified status even if no real modification has been done)

[Gubaer]

mergenodes.patch applied in r2064, but reopening the ticket which has a much broader scope.

[anonymous]

A note: Use "see #xxxx" instead of "fixed #xxxx" or "applied #xxxx" to reference a bug without closing :-)

[Don-vip]

Ticket #3804 has been marked as a duplicate of this ticket.

[skyper]

So the only way for ways to prevent a new version is to purge it, atm.

[stoecker]

Ticket #13054 has been marked as a duplicate of this ticket.