Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Potential issue with + #69

Closed
hikkyu opened this issue Feb 3, 2017 · 3 comments
Closed

Potential issue with + #69

hikkyu opened this issue Feb 3, 2017 · 3 comments

Comments

@hikkyu
Copy link

hikkyu commented Feb 3, 2017

I there!

If you put: 643123123 & FR
It output: ["+33643123123", "FRA"] All right!

But if you try this combinaison: +643123123 & FR
It output : ["+643123123", "FRA"], that is not the same number (or wrong number, I don't know).

Did i missed something ?

@adam0x01
Copy link
Contributor

adam0x01 commented Feb 10, 2017

If you provide the 2nd parameter (country), no need to add the + sign for the phone number.

var phone = require('phone');

phone("+33643123123", "FRA"); // [ '+33643123123', 'FRA' ]
phone("33643123123", "FRA");  // [ '+33643123123', 'FRA' ]
phone("643123123", "FR");     // [ '+33643123123', 'FRA' ]

@HiuKwok
Copy link
Contributor

HiuKwok commented Aug 27, 2018

Hi @adamliuxy ,
Does it mean once the plus sign is provided then the program itself would look for a country code prefix?
Let say by typing a HKG mobile ("+852 9122 5454", "HKG") would consider to be valid cuz plus sign is provided then it look for (852) and decide it's HKG cell phone number. But on the other case when user type ("+ 9122 5454", "HKG") it would consider to be invalid cuz country number (852) prefix is not provided?

And base on this should fill the number prefix on output if the country short form is provided?
Old case: ("+ 9122 5454", "HKG") -> Invalid
New case: ("+ 9122 5454", "HKG") -> Oh prefix (852) is absent but "HKG" is on -> Return ['+85291225454', 'HKG'] for output?

@Bossa573
Copy link
Member

@HiuKwok If a phone number is prefixed with a plus sign, it should be considered as there should be a country code provided i.e. for +91225454, the library should try to look up the country code 9, 91 and so on, and eventually no result could be found.

The country iso3 parameter provides extra information for formatting the phone number, but if the library adds the country code prefix according to the country iso3, ignoring the + sign, it is distorting the meaning of "the digits come after the plus sign", which supposedly should be followed by country code.

And for @hikkyu cases, please do not add the + sign before the phone number if you are not sure the digit followed by the + sign is the country code.

Thanks for the information and contribution, please reopen the ticket if needed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants