Modify

Opened 15 years ago

Closed 15 years ago

#4606 closed enhancement (fixed)

merging a new and an already existing node

Reported by: anonymous Owned by: team
Priority: normal Milestone:
Component: Core Version: latest
Keywords: merge new node Cc:

Description

If you merge a newly created node to an existing node, the existing node should not be deleted.

node(ID:-1) + node(ID:1234) --merge--> node(ID:1234)

Attachments (0)

Change History (9)

comment:1 by mjulius, 15 years ago

Since r2341 Merge Nodes always merges onto the last selected node.

comment:2 by vsandre, 15 years ago

Please introduce it anyway. A little bit lesser database flooding.

BTW I checked the new behaviour out, after reporting this bug. Nice feature.

comment:3 by Gubaer, 15 years ago

Type: defectenhancement

Trying to reformulate:

  • When a ordered list of nodes [node_1,node_2,..., node_n] (in the order of selection, first selected node first, last selected node last) are merged using Tools-Merge Nodes one of the nodes becomes the target node. The other nodes are deleted.
  • Current situation: the last node node_n becomes the target node
  • Proposed enhancement in this ticket: select the last non-new node as target node. If no such node exists, still select the last node.

in reply to:  3 comment:4 by mjulius, 15 years ago

Replying to Gubaer:

  • Proposed enhancement in this ticket: select the last non-new node as target node. If no such node exists, still select the last node.

And the target node should get the coordinates of the last selected node. This was the point of r2341.

comment:5 by Gubaer, 15 years ago

And the target node should get the coordinates of the last selected node. This was the point of r2341.

Right. I give it another try:

  • When an ordered list of nodes [node_1,node_2,..., node_n] (in the order of selection, first selected node first, last selected node last) are merged using Tools->Merge Nodes one of the nodes becomes the survinging node. The other nodes are deleted.
  • Current situation: the last node node_n becomes the survinging node
  • Proposed enhancement in this ticket: select the last non-new node as surviving node. If no such node exists, select the last selected node. The surviving node is always assigned the position of the last selected node.

in reply to:  5 comment:6 by vsandre, 15 years ago

Replying to Gubaer:

And the target node should get the coordinates of the last selected node. This was the point of r2341.

Right. I give it another try:

  • When an ordered list of nodes [node_1,node_2,..., node_n] (in the order of selection, first selected node first, last selected node last) are merged using Tools->Merge Nodes one of the nodes becomes the survinging node. The other nodes are deleted.
  • Current situation: the last node node_n becomes the survinging node
  • Proposed enhancement in this ticket: select the last non-new node as surviving node. If no such node exists, select the last selected node. The surviving node is always assigned the position of the last selected node.

+1

comment:7 by skyper, 15 years ago

Keywords: merge new node added

If JOSM could revert changes on server I think it would be data the best to keep the lowest id and use the last node as (new) position.

in reply to:  7 comment:8 by skyper, 15 years ago

Replying to skyper:

If JOSM could revert changes on server I think it would be data the best to keep the lowest id and use the last node as (new) position.

comment:9 by bastiK, 15 years ago

Resolution: fixed
Status: newclosed

(In [3134]) fixed #4606 - merging a new and an already existing node (better selection of "surviving" node)

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain team.
as The resolution will be set.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.