DrHyde / perl-modules-Number-Phone

Number::Phone and friends
24 stars 31 forks source link

Unintentional UK mode #47

Closed mjg17 closed 8 years ago

mjg17 commented 8 years ago

In adding the Åland Islands to Finland, I discovered a bug: any use of Number::Phone or of a stub country based on Number::Phone::StubCountry sets UK mode for Number::Phone::Country (and calling phone2country on a code which has multiple resolutions will cause stub country modules to be loaded).

The first commit is a failing test; the second is a fix.

The Åland Islands - AX - will be coming in a second data update patch at some stage.

(Test 41 is failing for me on master and after this patch, on subtests 97 and 117. I haven't investigated.)

DrHyde commented 8 years ago

I fixed those failing tests so you might want to merge master into your branch