Modify ↓

Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#16310 closed enhancement (fixed)

[PATCH] check for duplicate addresses

Reported by: Luzandro Owned by: team
Priority: normal Milestone: 18.06
Component: Core validator Version:
Keywords: address Cc:

Description

I've updated the address validator test, so that it checks for duplicate addresses that use addr:street/addr:place (the old test only checks associatedStreet relations).

Some considerations I've made:

  • POIs are ignored
  • units/flats are considered
  • there is a small amount of tolerance for what is seen as duplicate. For one whitespaces and dashes in street names are ignored, so that "Mozart-Gasse", "Mozart Gasse" and "Mozartgasse" are all seen as equal. And if at least one of the compared objects doesn't have addr:city and/or addr:postcode specified, distance is taken into account to decide if a warning should be generated and with what severity.

Attachments (1)

Addresses.java.patch​ (9.2 KB ) - added by Luzandro 6 years ago.

Download all attachments as: .zip

Change History (5)

comment:1 by Don-vip, 6 years ago

Keywords: address added
Milestone: → 18.06

by Luzandro, 6 years ago

Attachment: Addresses.java.patch​ added

comment:2 by Don-vip, 6 years ago

Resolution: → fixed
Status: new → closed

In 13968/josm:

fix #16310 - check for duplicate addresses (patch by Luzandro, modified)

comment:3 by Don-vip, 6 years ago

Great patch, thanks a lot! Sorry for the delay to apply it.

comment:4 by Don-vip, 6 years ago

In 13970/josm:

see #16310 - fix PMD warnings

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.