[PATCH] Ignore NodesWithSameName with unique ref

[Marko Mäkelä <marko.makela@…>]

There tend to be many bus or tram stops by the same name. When all the equally named nodes carry unique ref attributes, NodesWithSameName should not complain. This would reduce noise (false positives).

A further enhancement would be to somehow take operator=* into account, but this could be tricky, since both operator and ref are optional. It might also be useful to take the node type into account and not complain if bus, train or tram stops or parking places or rental stations all share the same street or place name.

[Marko Mäkelä <marko.makela@…>]

Patch to ignore same-name nodes with unique ref

[stoecker]

Actually I think that check should be dropped totally. It was invented for the GeoNames import and is probably outdated a lot nowadays.

[Marko Mäkelä <marko.makela@…>]

Replying to stoecker:

Actually I think that check should be dropped totally. It was invented for the GeoNames import and is probably outdated a lot nowadays.

I think that the check still is somewhat useful, because it can highlight bus stops with missing ref. Having validator generate a separate list of these “search results” could be a little nicer than repeatedly searching for “highway=bus_stop ref=”. Also, checking for duplicate place=* or shop=* names is still useful.
(I think that shops should be tagged not just name='Chain' but name='Chain Location'. When the location is part of the shop name (and it usually is if you look at the sales slip), it will be easier to read the list of nearby POIs in a navigator.)

[Marko Mäkelä <marko.makela@…>]

I realized that I do have commit access to the OpenStreetMap repository, where the JOSM plugins are stored. Because I am convinced that this patch is not breaking anything and it is silencing bogus warnings for me, I took the liberty of committing the patch.