Modify

Opened 11 years ago

Closed 11 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 11 years ago.
Snapshot_josm.png (51.8 KB ) - added by anonymous 11 years ago.
Snapshot: Josm validator

Download all attachments as: .zip

Change History (7)

comment:1 by Don-vip, 11 years ago

Owner: changed from team to anonymous
Status: newneedinfo

Please attach a test file.

by anonymous, 11 years ago

by anonymous, 11 years ago

Attachment: Snapshot_josm.png added

Snapshot: Josm validator

comment:2 by anonymous, 11 years ago

Status: needinfonew

comment:3 by anonymous, 11 years ago

Owner: changed from anonymous to team

comment:4 by Don-vip, 11 years ago

Priority: normalmajor

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

comment:5 by Don-vip, 11 years ago

Resolution: fixed
Status: newclosed

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