[openstreetmap/openstreetmap-website] Properly mark `tel:` link on multiple phone numbers in the POI (#2069)

Nutchanon Wetchasit notifications at github.com
Sun Nov 25 16:58:23 UTC 2018


For further testing, here are Overpass POI search URLs for each kind of phone/fax number that are affected by semicolon separator issue...

- Phone numbers:
	- `phone` <small>(~10.9k nodes, ~4.5k ways, ~420 relations)</small>: [Turbo](https://overpass-turbo.eu/?Q=%28node%5B%7E%22%5Ephone%28%3A.%2B%29%3F%24%22%7E%22%3B%22%5D%3B%28way%5B%7E%22%5Ephone%28%3A.%2B%29%3F%24%22%7E%22%3B%22%5D%3B%28._%3B%3E%3B%29%3B%29%3B%28relation%5B%7E%22%5Ephone%28%3A.%2B%29%3F%24%22%7E%22%3B%22%5D%3B%28._%3B%3E%3B%29%3B%29%3B%29-%3E._%3Bout%3B&R) | [Turbo+BBox](https://overpass-turbo.eu/?Q=%28node%5B%7E%22%5Ephone%28%3A.%2B%29%3F%24%22%7E%22%3B%22%5D%28%7B%7Bbbox%7D%7D%29%3B%28way%5B%7E%22%5Ephone%28%3A.%2B%29%3F%24%22%7E%22%3B%22%5D%28%7B%7Bbbox%7D%7D%29%3B%28._%3B%3E%3B%29%3B%29%3B%28relation%5B%7E%22%5Ephone%28%3A.%2B%29%3F%24%22%7E%22%3B%22%5D%28%7B%7Bbbox%7D%7D%29%3B%28._%3B%3E%3B%29%3B%29%3B%29-%3E._%3Bout%3B) | [Raw](https://overpass-api.de/api/interpreter?data=%28node%5B%7E%22%5Ephone%28%3A.%2B%29%3F%24%22%7E%22%3B%22%5D%3B%28way%5B%7E%22%5Ephone%28%3A.%2B%29%3F%24%22%7E%22%3B%22%5D%3B%28._%3B%3E%3B%29%3B%29%3B%28relation%5B%7E%22%5Ephone%28%3A.%2B%29%3F%24%22%7E%22%3B%22%5D%3B%28._%3B%3E%3B%29%3B%29%3B%29-%3E._%3Bout%3B)
	- `contact:phone` <small>(~4.8k nodes, ~1.7k ways, ~120 relations)</small>: [Turbo](https://overpass-turbo.eu/?Q=%28node%5B%7E%22%5Econtact%3Aphone%28%3A.%2B%29%3F%24%22%7E%22%3B%22%5D%3B%28way%5B%7E%22%5Econtact%3Aphone%28%3A.%2B%29%3F%24%22%7E%22%3B%22%5D%3B%28._%3B%3E%3B%29%3B%29%3B%28relation%5B%7E%22%5Econtact%3Aphone%28%3A.%2B%29%3F%24%22%7E%22%3B%22%5D%3B%28._%3B%3E%3B%29%3B%29%3B%29-%3E._%3Bout%3B&R) | [Turbo+BBox](https://overpass-turbo.eu/?Q=%28node%5B%7E%22%5Econtact%3Aphone%28%3A.%2B%29%3F%24%22%7E%22%3B%22%5D%28%7B%7Bbbox%7D%7D%29%3B%28way%5B%7E%22%5Econtact%3Aphone%28%3A.%2B%29%3F%24%22%7E%22%3B%22%5D%28%7B%7Bbbox%7D%7D%29%3B%28._%3B%3E%3B%29%3B%29%3B%28relation%5B%7E%22%5Econtact%3Aphone%28%3A.%2B%29%3F%24%22%7E%22%3B%22%5D%28%7B%7Bbbox%7D%7D%29%3B%28._%3B%3E%3B%29%3B%29%3B%29-%3E._%3Bout%3B) | [Raw](https://overpass-api.de/api/interpreter?data=%28node%5B%7E%22%5Econtact%3Aphone%28%3A.%2B%29%3F%24%22%7E%22%3B%22%5D%3B%28way%5B%7E%22%5Econtact%3Aphone%28%3A.%2B%29%3F%24%22%7E%22%3B%22%5D%3B%28._%3B%3E%3B%29%3B%29%3B%28relation%5B%7E%22%5Econtact%3Aphone%28%3A.%2B%29%3F%24%22%7E%22%3B%22%5D%3B%28._%3B%3E%3B%29%3B%29%3B%29-%3E._%3Bout%3B)
	- Both kind <small>(~15.7k nodes, ~6.2k ways, ~540 relations)</small>: [Turbo](https://overpass-turbo.eu/?Q=%28node%5B%7E%22%5E%28contact%3A%29%3Fphone%28%3A.%2B%29%3F%24%22%7E%22%3B%22%5D%3B%28way%5B%7E%22%5E%28contact%3A%29%3Fphone%28%3A.%2B%29%3F%24%22%7E%22%3B%22%5D%3B%28._%3B%3E%3B%29%3B%29%3B%28relation%5B%7E%22%5E%28contact%3A%29%3Fphone%28%3A.%2B%29%3F%24%22%7E%22%3B%22%5D%3B%28._%3B%3E%3B%29%3B%29%3B%29-%3E._%3Bout%3B&R) | [Turbo+BBox](https://overpass-turbo.eu/?Q=%28node%5B%7E%22%5E%28contact%3A%29%3Fphone%28%3A.%2B%29%3F%24%22%7E%22%3B%22%5D%28%7B%7Bbbox%7D%7D%29%3B%28way%5B%7E%22%5E%28contact%3A%29%3Fphone%28%3A.%2B%29%3F%24%22%7E%22%3B%22%5D%28%7B%7Bbbox%7D%7D%29%3B%28._%3B%3E%3B%29%3B%29%3B%28relation%5B%7E%22%5E%28contact%3A%29%3Fphone%28%3A.%2B%29%3F%24%22%7E%22%3B%22%5D%28%7B%7Bbbox%7D%7D%29%3B%28._%3B%3E%3B%29%3B%29%3B%29-%3E._%3Bout%3B) | [Raw](https://overpass-api.de/api/interpreter?data=%28node%5B%7E%22%5E%28contact%3A%29%3Fphone%28%3A.%2B%29%3F%24%22%7E%22%3B%22%5D%3B%28way%5B%7E%22%5E%28contact%3A%29%3Fphone%28%3A.%2B%29%3F%24%22%7E%22%3B%22%5D%3B%28._%3B%3E%3B%29%3B%29%3B%28relation%5B%7E%22%5E%28contact%3A%29%3Fphone%28%3A.%2B%29%3F%24%22%7E%22%3B%22%5D%3B%28._%3B%3E%3B%29%3B%29%3B%29-%3E._%3Bout%3B)
- Fax numbers:
	- `fax` <small>(~70 nodes, ~75 ways, ~10 relations)</small>: [Turbo](https://overpass-turbo.eu/?Q=%28node%5B%7E%22%5Efax%28%3A.%2B%29%3F%24%22%7E%22%3B%22%5D%3B%28way%5B%7E%22%5Efax%28%3A.%2B%29%3F%24%22%7E%22%3B%22%5D%3B%28._%3B%3E%3B%29%3B%29%3B%28relation%5B%7E%22%5Efax%28%3A.%2B%29%3F%24%22%7E%22%3B%22%5D%3B%28._%3B%3E%3B%29%3B%29%3B%29-%3E._%3Bout%3B&R) | [Turbo+BBox](https://overpass-turbo.eu/?Q=%28node%5B%7E%22%5Efax%28%3A.%2B%29%3F%24%22%7E%22%3B%22%5D%28%7B%7Bbbox%7D%7D%29%3B%28way%5B%7E%22%5Efax%28%3A.%2B%29%3F%24%22%7E%22%3B%22%5D%28%7B%7Bbbox%7D%7D%29%3B%28._%3B%3E%3B%29%3B%29%3B%28relation%5B%7E%22%5Efax%28%3A.%2B%29%3F%24%22%7E%22%3B%22%5D%28%7B%7Bbbox%7D%7D%29%3B%28._%3B%3E%3B%29%3B%29%3B%29-%3E._%3Bout%3B) | [Raw](https://overpass-api.de/api/interpreter?data=%28node%5B%7E%22%5Efax%28%3A.%2B%29%3F%24%22%7E%22%3B%22%5D%3B%28way%5B%7E%22%5Efax%28%3A.%2B%29%3F%24%22%7E%22%3B%22%5D%3B%28._%3B%3E%3B%29%3B%29%3B%28relation%5B%7E%22%5Efax%28%3A.%2B%29%3F%24%22%7E%22%3B%22%5D%3B%28._%3B%3E%3B%29%3B%29%3B%29-%3E._%3Bout%3B)
	- `contact:fax` <small>(~30 nodes, ~25 ways, ~5 relations)</small>: [Turbo](https://overpass-turbo.eu/?Q=%28node%5B%7E%22%5Econtact%3Afax%28%3A.%2B%29%3F%24%22%7E%22%3B%22%5D%3B%28way%5B%7E%22%5Econtact%3Afax%28%3A.%2B%29%3F%24%22%7E%22%3B%22%5D%3B%28._%3B%3E%3B%29%3B%29%3B%28relation%5B%7E%22%5Econtact%3Afax%28%3A.%2B%29%3F%24%22%7E%22%3B%22%5D%3B%28._%3B%3E%3B%29%3B%29%3B%29-%3E._%3Bout%3B&R) | [Turbo+BBox](https://overpass-turbo.eu/?Q=%28node%5B%7E%22%5Econtact%3Afax%28%3A.%2B%29%3F%24%22%7E%22%3B%22%5D%28%7B%7Bbbox%7D%7D%29%3B%28way%5B%7E%22%5Econtact%3Afax%28%3A.%2B%29%3F%24%22%7E%22%3B%22%5D%28%7B%7Bbbox%7D%7D%29%3B%28._%3B%3E%3B%29%3B%29%3B%28relation%5B%7E%22%5Econtact%3Afax%28%3A.%2B%29%3F%24%22%7E%22%3B%22%5D%28%7B%7Bbbox%7D%7D%29%3B%28._%3B%3E%3B%29%3B%29%3B%29-%3E._%3Bout%3B) | [Raw](https://overpass-api.de/api/interpreter?data=%28node%5B%7E%22%5Econtact%3Afax%28%3A.%2B%29%3F%24%22%7E%22%3B%22%5D%3B%28way%5B%7E%22%5Econtact%3Afax%28%3A.%2B%29%3F%24%22%7E%22%3B%22%5D%3B%28._%3B%3E%3B%29%3B%29%3B%28relation%5B%7E%22%5Econtact%3Afax%28%3A.%2B%29%3F%24%22%7E%22%3B%22%5D%3B%28._%3B%3E%3B%29%3B%29%3B%29-%3E._%3Bout%3B)
	- Both kind  <small>(~100 nodes, ~100 ways, ~15 relations)</small>: [Turbo](https://overpass-turbo.eu/?Q=%28node%5B%7E%22%5E%28contact%3A%29%3Ffax%28%3A.%2B%29%3F%24%22%7E%22%3B%22%5D%3B%28way%5B%7E%22%5E%28contact%3A%29%3Ffax%28%3A.%2B%29%3F%24%22%7E%22%3B%22%5D%3B%28._%3B%3E%3B%29%3B%29%3B%28relation%5B%7E%22%5E%28contact%3A%29%3Ffax%28%3A.%2B%29%3F%24%22%7E%22%3B%22%5D%3B%28._%3B%3E%3B%29%3B%29%3B%29-%3E._%3Bout%3B&R) | [Turbo+BBox](https://overpass-turbo.eu/?Q=%28node%5B%7E%22%5E%28contact%3A%29%3Ffax%28%3A.%2B%29%3F%24%22%7E%22%3B%22%5D%28%7B%7Bbbox%7D%7D%29%3B%28way%5B%7E%22%5E%28contact%3A%29%3Ffax%28%3A.%2B%29%3F%24%22%7E%22%3B%22%5D%28%7B%7Bbbox%7D%7D%29%3B%28._%3B%3E%3B%29%3B%29%3B%28relation%5B%7E%22%5E%28contact%3A%29%3Ffax%28%3A.%2B%29%3F%24%22%7E%22%3B%22%5D%28%7B%7Bbbox%7D%7D%29%3B%28._%3B%3E%3B%29%3B%29%3B%29-%3E._%3Bout%3B) | [Raw](https://overpass-api.de/api/interpreter?data=%28node%5B%7E%22%5E%28contact%3A%29%3Ffax%28%3A.%2B%29%3F%24%22%7E%22%3B%22%5D%3B%28way%5B%7E%22%5E%28contact%3A%29%3Ffax%28%3A.%2B%29%3F%24%22%7E%22%3B%22%5D%3B%28._%3B%3E%3B%29%3B%29%3B%28relation%5B%7E%22%5E%28contact%3A%29%3Ffax%28%3A.%2B%29%3F%24%22%7E%22%3B%22%5D%3B%28._%3B%3E%3B%29%3B%29%3B%29-%3E._%3Bout%3B)

**Legends:** _Turbo_ for instant global search, _Turbo+BBox_ for specific-area search, _Raw_ for global search exported in .OSM XML format.


-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/issues/2069#issuecomment-441455024
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20181125/c71a3292/attachment-0001.html>


More information about the rails-dev mailing list