Modify

Opened 3 years ago

Closed 3 years ago

#4348 closed defect (fixed)

Validator wrongly merge duplicated nodes.

Reported by: Zbigniew Czernik <zibiczek@…> Owned by: team
Priority: normal Component: Core validator
Version: latest Keywords: node, duplicated, merge
Cc:

Description

Simple test scenario:

  1. I download any area from OSM and create a node.
  2. I send the node to JOSM (and the node has assigned ID)
  3. I select this node and copy it (CTRL + C)
  4. I create a new layer and paste this node here (CTRL + V)
  5. I merge both layers together.
  6. At this point, I have a duplicated node.
  7. I check by validator, it detects a duplicated node.
  8. I click on Fix.
  9. Validator removes the node with assigned ID and replace it with a new (without ID)
  10. JOSM wants to upload the new node.

The problem is trivial for a single node, but it looks otherwise when I prepare import and I have several thousand duplicated nodes.
Do not want to unnecessarily spam OSM database.

Tested on rev. 2813

Attachments (0)

Change History (3)

comment:1 Changed 3 years ago by Zibik

I have lost formatting in text :(

Simple test scenario:

  1. I download any area from OSM and create a node.
  2. I send the node to JOSM (and the node has assigned ID)
  3. I select this node and copy it (CTRL + C)
  4. I create a new layer and paste this node here (CTRL + V)
  5. I merge both layers together.
  6. At this point, I have a duplicated node.
  7. I check by validator, it detects a duplicated node.
  8. I click on Fix.
  9. Validator removes the node with assigned ID and replace it with a new (without ID)
  10. JOSM wants to upload the new node.

The problem is trivial for a single node, but it looks otherwise when I prepare import and I have several thousand duplicated nodes. Do not want to unnecessarily spam OSM database.

Tested on rev. 2813

comment:2 Changed 3 years ago by Zibik

Nobody wants to fix it?

comment:3 Changed 3 years ago by bastiK

  • Resolution set to fixed
  • Status changed from new to closed

Should be fixed in [3134]

Add Comment

Modify Ticket

Change Properties
<Author field>
Action
as closed .
as The resolution will be set. Next status will be 'closed'.
The resolution will be deleted. Next status will be 'reopened'.
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.