Modify

Opened 4 years ago

Closed 4 years ago

#3786 closed defect (fixed)

[PATCH] JOSM doesn't count duplicate nodes in ways in node count

Reported by: avar Owned by: team
Priority: major Component: Core
Version: latest Keywords:
Cc: teemu.koskinen@…

Description

The selection dialog has e.g. "(2 nodes)" even when looking at a 4-node way like the one attached to this bug.

It's also impossible to edit these in any intelligent way but that's the subject for another bug.

Attachments (2)

dup.xml (774 bytes) - added by avar 4 years ago.
Way with duplicate nodes
way-node-count.patch (704 bytes) - added by Daeron 4 years ago.

Download all attachments as: .zip

Change History (6)

Changed 4 years ago by avar

Way with duplicate nodes

comment:1 follow-up: Changed 4 years ago by Gubaer

But why would you have identical nodes in sequence? This looks like an anomaly.

comment:2 in reply to: ↑ 1 Changed 4 years ago by avar

Replying to Gubaer:

But why would you have identical nodes in sequence? This looks like an anomaly.

It's an error in the data. But JOSM should report the actual number of nodes in a way regardless, not the number of unique node IDs in it.

Generally speaking there are several cases where the only thing to see these sort of errors is by using the validator plugin.

Changed 4 years ago by Daeron

comment:3 Changed 4 years ago by Daeron

  • Cc teemu.koskinen@… added
  • Summary changed from JOSM doesn't count duplicate nodes in ways in node count to [PATCH] JOSM doesn't count duplicate nodes in ways in node count

comment:4 Changed 4 years ago by Gubaer

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

(In [2385]) applied #3786: patch by Daeron: JOSM doesn't count duplicate nodes in ways in node count

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.