Opened 5 years ago

Last modified 5 years ago

#19581 closed defect

Fixing duplicated nodes does not work — at Version 1

Reported by: skyper Owned by: team
Priority: normal Milestone: 20.11
Component: Core validator Version:
Keywords: template_report fix duplicate nodes Cc:

Description (last modified by skyper)

Edit: Oh, the fix does not work at all but selection issue below is strange.

What steps will reproduce the problem?

  1. Have two areas with Mixed type duplicated nodes (josm_Mixed_type_duplicated_nodes_example.osm.xz)
  2. Run Validator
  3. Take a look at the number of duplicated nodes reported
  4. Select the nodes using select button in validator panel
  5. Take a look at the selection in selection list panel (2 nodes)
  6. Click on fix button in validator panel
  7. Take a look at the selection in selection list panel (2 nodes)
  8. Fix all reported duplicated nodes
  9. Run validator

What is the expected result?

Nodes are merge with the fix and only one node should be listed in the selection list panel.

What happens instead?

Displayed selection in selection list panel does not change. Still both nodes.
Nodes are not merged.

Please provide any additional information below. Attach a screenshot if possible.

I tried to select the younger one (higher id) which should be deleted and it works but deleting the node deletes both, so I think it is hopefully only a problem of updating the display of the selection list there could be even more hidden problems.

Or did I find a critical bug?

Relative:URL: ^/trunk
Repository:UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
Last:Changed Date: 2020-07-25 14:35:24 +0200 (Sat, 25 Jul 2020)
Revision:16808
Build-Date:2020-07-26 01:30:54
URL:https://josm.openstreetmap.de/svn/trunk

Change History (2)

by skyper, 5 years ago

example file

comment:1 by skyper, 5 years ago

Description: modified (diff)
Summary: Wrong selection after merging nodes using "fix" buttonFixing duplicated nodes does not work

Fix does not work at all.

Note: See TracTickets for help on using tickets.