Modify ↓

Opened 12 years ago

Closed 12 years ago

#8546 closed defect (fixed)

Validator detecting only one of multible "crossing buildings" or "building inside building"

Reported by: anonymous Owned by: team
Priority: major Milestone:
Component: Core validator Version: latest
Keywords: Cc:

Description

Draw for example 4 overlapping buildings.
Select all - shift-f
Results: Warnings -> Crossing Buildings (1)

tested with version: 5802 but also seen with earlier versions (5899)

Version 5759 shows all "Crossing Buildings" ... as expected.

Attachments (2)

building_cross_valid_bug.osm​ (4.9 KB ) - added by anonymous 12 years ago.
Snapshot_josm.png​ (51.8 KB ) - added by anonymous 12 years ago.
Snapshot: Josm validator

Download all attachments as: .zip

Change History (7)

comment:1 by Don-vip, 12 years ago

Owner: changed from team to anonymous
Status: new → needinfo

Please attach a test file.

by anonymous, 12 years ago

Attachment: building_cross_valid_bug.osm​ added

by anonymous, 12 years ago

Attachment: Snapshot_josm.png​ added

Snapshot: Josm validator

comment:2 by anonymous, 12 years ago

Status: needinfo → new

comment:3 by anonymous, 12 years ago

Owner: changed from anonymous to team

comment:4 by Don-vip, 12 years ago

Priority: normal → major

Confirmed. Regression introduced between r5779 and r5787 (maybe r5783).

comment:5 by Don-vip, 12 years ago

Resolution: → fixed
Status: new → closed

In 5822/josm:

fix #8546 - Validator do not detect errors named identically (partial revert of r5783, reopens #8517). This is due to TreeMap that uses compareTo() instead of equals() (see Javadoc)

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 .
Attachments ↑ Description ↑
 
Note: See TracTickets for help on using tickets.