Removes hardcoded defaulting to US number from the code.

This commit is contained in:
damencho 2019-05-03 10:50:58 +01:00 committed by Дамян Минков
parent 0ec4e6a805
commit 384f0d4317
1 changed files with 2 additions and 16 deletions

View File

@ -518,18 +518,12 @@ export function getDialInfoPageURL(
*/ */
export function _getDefaultPhoneNumber( export function _getDefaultPhoneNumber(
dialInNumbers: Object): ?string { dialInNumbers: Object): ?string {
const defValueForDefaultCountry = 'US';
if (Array.isArray(dialInNumbers)) { if (Array.isArray(dialInNumbers)) {
// new syntax follows // new syntax follows
// find the default country inside dialInNumbers, US one // find the default country inside dialInNumbers, US one
// or return the first one // or return the first one
let defaultNumber = dialInNumbers.find(number => number.default); const defaultNumber = dialInNumbers.find(number => number.default);
if (!defaultNumber) {
defaultNumber = dialInNumbers.find(({ countryCode }) =>
countryCode === defValueForDefaultCountry);
}
if (defaultNumber) { if (defaultNumber) {
return defaultNumber.formattedNumber; return defaultNumber.formattedNumber;
@ -539,18 +533,10 @@ export function _getDefaultPhoneNumber(
? dialInNumbers[0].formattedNumber : null; ? dialInNumbers[0].formattedNumber : null;
} }
const { const { numbers } = dialInNumbers;
defaultCountry = defValueForDefaultCountry,
numbers } = dialInNumbers;
if (numbers && Object.keys(numbers).length > 0) { if (numbers && Object.keys(numbers).length > 0) {
// deprecated and will be removed // deprecated and will be removed
const defaultNumbers = numbers[defaultCountry];
if (defaultNumbers) {
return defaultNumbers[0];
}
const firstRegion = Object.keys(numbers)[0]; const firstRegion = Object.keys(numbers)[0];
return firstRegion && numbers[firstRegion][0]; return firstRegion && numbers[firstRegion][0];