Modify

Opened 12 years ago

Closed 12 years ago

#3786 closed defect (fixed)

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

Reported by: avarab@… Owned by: team
Priority: major Milestone:
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 avarab@… 12 years ago.
Way with duplicate nodes
way-node-count.patch (704 bytes) - added by Daeron 12 years ago.

Download all attachments as: .zip

Change History (6)

Changed 12 years ago by avarab@…

Attachment: dup.xml added

Way with duplicate nodes

comment:1 Changed 12 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 12 years ago by avarab@…

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 12 years ago by Daeron

Attachment: way-node-count.patch added

comment:3 Changed 12 years ago by Daeron

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

comment:4 Changed 12 years ago by Gubaer

Resolution: fixed
Status: newclosed

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

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.

Add Comment


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

 
Note: See TracTickets for help on using tickets.