PhoneValidator::isValidNumber() - Code Metrics - Inspection of "Add 'LENIENT' parameter for less strict validation" - Propaganistas/Laravel-Phone - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#22)

by Ryan

created 2015-12-28 01:01 UTC

PhoneValidator::isValidNumber() C

↳ Parent: PhoneValidator

Complexity

Conditions	7
Paths	9

Size

Total Lines	32
Code Lines	11

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	9
CRAP Score	7

Importance

Changes	3
Bugs	0	Features	1

Metric	Value
c	3
b	0
f	1
dl	0
loc	32
ccs	9
cts	9
cp	1
rs	6.7273
cc	7
eloc	11
nc	9
nop	2
crap	7

<?php namespace Propaganistas\LaravelPhone;

use libphonenumber\NumberParseException;
use libphonenumber\PhoneNumberType;
use libphonenumber\PhoneNumberUtil;
use Propaganistas\LaravelPhone\Exceptions\NoValidCountryFoundException;
use Propaganistas\LaravelPhone\Exceptions\InvalidParameterException;

class PhoneValidator
{

    /**
     * @var \libphonenumber\PhoneNumberUtil
     */
    protected $lib;

    /**
     * Whether the country should be auto-detected.
     *
     * @var bool
     */
    protected $autodetect = false;

    /**
     * Whether to allow lenient checking of numbers (i.e. landline without area codes)
     *
     * @var object
     */
    protected $lenient = false;

    /**
     * Countries to validate against.
     *
     * @var array
     */
    protected $countries = [];

    /**
     * Transformed phone number types to validate against.
     *
     * @var array
     */
    protected $types = [];

    /**
     * PhoneValidator constructor.
     */
    public function __construct(PhoneNumberUtil $lib)
    {
        $this->lib = $lib;
    }

    /**
     * Validates a phone number.
     *
     * @param  string $attribute
     * @param  mixed  $value
     * @param  array  $parameters
     * @param  object $validator
     * @return bool
     * @throws \Propaganistas\LaravelPhone\Exceptions\InvalidParameterException
     * @throws \Propaganistas\LaravelPhone\Exceptions\NoValidCountryFoundException
     */
    public function validatePhone($attribute, $value, array $parameters, $validator)
    {
        $this->assignParameters(array_map('strtoupper', $parameters));

        $this->checkCountries($attribute, $validator);

        // If autodetecting, let's first try without a country.
        // Otherwise use provided countries as default.
        if ($this->autodetect) {
            array_unshift($this->countries, null);
        }

        foreach ($this->countries as $country) {
            if ($this->isValidNumber($value, $country)) {
                return true;
            }
        }

        // All specified country validations have failed.
        return false;
    }

    /**
     * Parses the supplied validator parameters.
     *
     * @param array $parameters
     * @throws \Propaganistas\LaravelPhone\Exceptions\InvalidParameterException
     */
    protected function assignParameters(array $parameters)
    {
        $types = array();

        foreach ($parameters as $parameter) {
            if ($this->isPhoneCountry($parameter)) {
                $this->countries[] = $parameter;
            } elseif ($this->isPhoneType($parameter)) {
                $types[] = $parameter;
            } elseif ($parameter == 'AUTO') {
                $this->autodetect = true;
            } elseif ($parameter = 'LENIENT') {
                $this->lenient = true;

            } else {
                // Force developers to write proper code.
                throw new InvalidParameterException($parameter);
            }
        }

        $this->types = $this->parseTypes($types);

    }

    /**
     * Checks the detected countries. Overrides countries if a country field is present.
     * When using a country field, we should validate to false if country is empty so no exception
     * will be thrown.
     *
     * @param $attribute
     * @param $validator
     * @throws \Propaganistas\LaravelPhone\Exceptions\NoValidCountryFoundException
     */
    protected function checkCountries($attribute, $validator)
    {
        $data = $validator->getData();
        $countryField = $attribute . '_country';

        if (isset($data[$countryField])) {
            $this->countries = array($data[$countryField]);
        } elseif (!$this->autodetect && empty($this->countries)) {
            throw new NoValidCountryFoundException;
        }
    }

    /**
     * Performs the actual validation of the phone number.
     *
     * @param mixed $number
     * @param null  $country
     * @return bool
     */
    protected function isValidNumber($number, $country = null)
    {
        try {
            // Throws NumberParseException if not parsed correctly against the supplied country.
            // If no country was given, tries to discover the country code from the number itself.
            $phoneNumber = $this->lib->parse($number, $country);

            // For automatic detection, the number should have a country code.
            // Check if type is allowed.
            if ($phoneNumber->hasCountryCode() && (empty($this->types) || in_array($this->lib->getNumberType($phoneNumber), $this->types))) {

                // Lenient validation:
                if ($this->lenient) {
                    return $this->lib->isPossibleNumber($phoneNumber);
                }

                // Automatic detection:
                if ($this->autodetect) {
                    // Validate if the international phone number is valid for its contained country.
                    return $this->lib->isValidNumber($phoneNumber);
                }

                // Validate number against the specified country.
                return $this->lib->isValidNumberForRegion($phoneNumber, $country);
            }

        } catch (NumberParseException $e) {
            // Proceed to default validation error.
        }

        return false;
    }

    /**
     * Parses the supplied phone number types.
     *
     * @param array $types
     * @return array
     */
    protected function parseTypes(array $types)
    {
        // Transform types to their namespaced class constant.
        array_walk($types, function (&$type) {
            $type = constant('\libphonenumber\PhoneNumberType::' . $type);
        });

        // Add in the unsure number type if applicable.
        if (array_intersect([PhoneNumberType::FIXED_LINE, PhoneNumberType::MOBILE], $types)) {
            $types[] = PhoneNumberType::FIXED_LINE_OR_MOBILE;
        }

        return $types;
    }

    /**
     * Checks if the supplied string is a valid country code using some arbitrary country validation.
     * If using a package based on umpirsky/country-list, invalidate the option 'ZZ => Unknown or invalid region'.
     *
     * @param  string $country
     * @return bool
     */
    public function isPhoneCountry($country)
    {
        return (strlen($country) === 2 && ctype_alpha($country) && ctype_upper($country) && $country != 'ZZ');
    }

    /**
     * Checks if the supplied string is a valid phone number type.
     *
     * @param  string $type
     * @return bool
     */
    public function isPhoneType($type)
    {
        // Legacy support.
        $type = ($type == 'LANDLINE' ? 'FIXED_LINE' : $type);

        return defined('\libphonenumber\PhoneNumberType::' . $type);
    }

}


1		<?php namespace Propaganistas\LaravelPhone;
2
3		use libphonenumber\NumberParseException;
4		use libphonenumber\PhoneNumberType;
5		use libphonenumber\PhoneNumberUtil;
6		use Propaganistas\LaravelPhone\Exceptions\NoValidCountryFoundException;
7		use Propaganistas\LaravelPhone\Exceptions\InvalidParameterException;
8
9		class PhoneValidator
10		{
11
12		/**
13		* @var \libphonenumber\PhoneNumberUtil
14		*/
15		protected $lib;
16
17		/**
18		* Whether the country should be auto-detected.
19		*
20		* @var bool
21		*/
22		protected $autodetect = false;
23
24		/**
25		* Whether to allow lenient checking of numbers (i.e. landline without area codes)
26		*
27		* @var object
28		*/
29		protected $lenient = false;
30
31		/**
32		* Countries to validate against.
33		*
34		* @var array
35		*/
36		protected $countries = [];
37
38		/**
39		* Transformed phone number types to validate against.
40		*
41	21	* @var array
42		*/
43	21	protected $types = [];
44	21
45		/**
46		* PhoneValidator constructor.
47		*/
48		public function __construct(PhoneNumberUtil $lib)
49		{
50		$this->lib = $lib;
51		}
52
53		/**
54		* Validates a phone number.
55		*
56		* @param string $attribute
57	21	* @param mixed $value
58		* @param array $parameters
59	21	* @param object $validator
60		* @return bool
61	18	* @throws \Propaganistas\LaravelPhone\Exceptions\InvalidParameterException
62		* @throws \Propaganistas\LaravelPhone\Exceptions\NoValidCountryFoundException
63		*/
64		public function validatePhone($attribute, $value, array $parameters, $validator)
65	15	{
66	3	$this->assignParameters(array_map('strtoupper', $parameters));
67	2
68		$this->checkCountries($attribute, $validator);
69	15
70	15	// If autodetecting, let's first try without a country.
71	15	// Otherwise use provided countries as default.
72		if ($this->autodetect) {
73	10	array_unshift($this->countries, null);
74		}
75
76	15	foreach ($this->countries as $country) {
77		if ($this->isValidNumber($value, $country)) {
78		return true;
79		}
80		}
81
82		// All specified country validations have failed.
83		return false;
84		}
85	21
86		/**
87	21	* Parses the supplied validator parameters.
88		*
89	21	* @param array $parameters
90	15	* @throws \Propaganistas\LaravelPhone\Exceptions\InvalidParameterException
91	12	*/
92	14	protected function assignParameters(array $parameters)
93	9	{
94	10	$types = array();
95	3
96	2	foreach ($parameters as $parameter) {
97		if ($this->isPhoneCountry($parameter)) {
98	7	$this->countries[] = $parameter;
99		} elseif ($this->isPhoneType($parameter)) {
100	14	$types[] = $parameter;
101		} elseif ($parameter == 'AUTO') {
102	18	$this->autodetect = true;
103		} elseif ($parameter = 'LENIENT') {
104	18	$this->lenient = true;
		0 ignored issues – show Documentation Bug introduced 2015-12-28 01:25 UTC by Report Bug Copy Issue Report It seems like `true` of type `boolean` is incompatible with the declared type `object` of property `$lenient`. Our type inference engine has found an assignment to a property that is incompatible with the declared type of that property. Either this assignment is in error or the assigned type should be added to the documentation/type hint for that property.. Loading history...
105		} else {
106		// Force developers to write proper code.
107		throw new InvalidParameterException($parameter);
108		}
109		}
110
111		$this->types = $this->parseTypes($types);
112
113		}
114
115	18	/**
116		* Checks the detected countries. Overrides countries if a country field is present.
117	18	* When using a country field, we should validate to false if country is empty so no exception
118	18	* will be thrown.
119		*
120	18	* @param $attribute
121	6	* @param $validator
122	16	* @throws \Propaganistas\LaravelPhone\Exceptions\NoValidCountryFoundException
123	3	*/
124		protected function checkCountries($attribute, $validator)
125	15	{
126		$data = $validator->getData();
127		$countryField = $attribute . '_country';
128
129		if (isset($data[$countryField])) {
130		$this->countries = array($data[$countryField]);
131		} elseif (!$this->autodetect && empty($this->countries)) {
132		throw new NoValidCountryFoundException;
133		}
134	15	}
135
136		/**
137		* Performs the actual validation of the phone number.
138		*
139	15	* @param mixed $number
140		* @param null $country
141		* @return bool
142		*/
143	15	protected function isValidNumber($number, $country = null)
144		{
145		try {
146	15	// Throws NumberParseException if not parsed correctly against the supplied country.
147		// If no country was given, tries to discover the country code from the number itself.
148	3	$phoneNumber = $this->lib->parse($number, $country);
149
150		// For automatic detection, the number should have a country code.
151		// Check if type is allowed.
152	12	if ($phoneNumber->hasCountryCode() && (empty($this->types) \|\| in_array($this->lib->getNumberType($phoneNumber), $this->types))) {
153
154		// Lenient validation:
155	7	if ($this->lenient) {
156		return $this->lib->isPossibleNumber($phoneNumber);
157		}
158
159	9	// Automatic detection:
160		if ($this->autodetect) {
161		// Validate if the international phone number is valid for its contained country.
162		return $this->lib->isValidNumber($phoneNumber);
163		}
164
165		// Validate number against the specified country.
166		return $this->lib->isValidNumberForRegion($phoneNumber, $country);
167		}
168
169		} catch (NumberParseException $e) {
170		// Proceed to default validation error.
171	18	}
172	6
173	18	return false;
174		}
175
176	18	/**
177	6	* Parses the supplied phone number types.
178	4	*
179		* @param array $types
180	18	* @return array
181		*/
182		protected function parseTypes(array $types)
183		{
184		// Transform types to their namespaced class constant.
185		array_walk($types, function (&$type) {
186		$type = constant('\libphonenumber\PhoneNumberType::' . $type);
187		});
188
189		// Add in the unsure number type if applicable.
190	15	if (array_intersect([PhoneNumberType::FIXED_LINE, PhoneNumberType::MOBILE], $types)) {
191		$types[] = PhoneNumberType::FIXED_LINE_OR_MOBILE;
192	15	}
193
194		return $types;
195		}
196
197		/**
198		* Checks if the supplied string is a valid country code using some arbitrary country validation.
199		* If using a package based on umpirsky/country-list, invalidate the option 'ZZ => Unknown or invalid region'.
200		*
201	12	* @param string $country
202		* @return bool
203		*/
204	12	public function isPhoneCountry($country)
205		{
206	12	return (strlen($country) === 2 && ctype_alpha($country) && ctype_upper($country) && $country != 'ZZ');
207		}
208
209		/**
210		* Checks if the supplied string is a valid phone number type.
211		*
212		* @param string $type
213		* @return bool
214		*/
215		public function isPhoneType($type)
216		{
217		// Legacy support.
218		$type = ($type == 'LANDLINE' ? 'FIXED_LINE' : $type);
219
220		return defined('\libphonenumber\PhoneNumberType::' . $type);
221		}
222
223		}
224

Propaganistas / Laravel-Phone

Pull Request — master (#22)

PhoneValidator::isValidNumber() C

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like