Properly process alt_name tag when there are multiple entries separated by semicolon

[SiarheiFedartsou]

👋 I did some awesome work for the Pelias project and would love for everyone to have a look at it and provide feedback.

---

Here's the reason for this change 🚀

OSM's alt_name tag may contain multiple entries which are separated by semicolon (;) that's why I propose to split by it and use each entry as separate alias name.
See: https://wiki.openstreetmap.org/wiki/Key:alt_name#Examples
Example: https://www.openstreetmap.org/way/31151623

---

Here's what actually got changed 👏

Just additional split call for alt_name tags :)

---

Here's how others can test the changes 👀

I added new test (which is almost copy-paste of one of existing ones though, but couldn't come up with better solution)

[SiarheiFedartsou]

Tbh at glance it seems to be better to "bake" the fact that we need to split by something to NAME_SCHEMA, but it seems to be a bit of over-engineering for now (but let me know :) )

[missinglink]

We're doing something similar already for addresses FYI:

openstreetmap/stream/address_extractor.js

Lines 48 to 50 in 75309fa

	// accept semi-colon delimited house numbers
	// ref: https://github.com/pelias/openstreetmap/issues/21
	var streetNumbers = (doc.getAddress('number') || '').split(';').map(Function.prototype.call, String.prototype.trim);

[missinglink]

Seems this day has come 😆
#51 (comment)

[missinglink]

FWIW I really hate this semicolon convention but it's here to stay 🤷‍♂️

[SiarheiFedartsou]

We're doing something similar already for addresses FYI:

Do you think I should change this PR somehow to be aligned with this implementation? :)

[SiarheiFedartsou]

Btw it seems alt_name:XXX (e.g. alt_name:en or alt_name:th) are not supported, or I am missing something?

[missinglink]

Btw it seems alt_name:XXX (e.g. alt_name:en or alt_name:th) are not supported, or I am missing something?

For alt specifically? If so let's please open a new issue so we can discuss separately

[SiarheiFedartsou]

Btw it seems alt_name:XXX (e.g. alt_name:en or alt_name:th) are not supported, or I am missing something?

For alt specifically? If so let's please open a new issue so we can discuss separately

Yep, please see #583