Digitonic\Validation\Services\AllowedRecipientsSanitiser

Complexity

Total Complexity

12

Size/Duplication

Total Lines	99
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

100%

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	getFormattedNumber()	0	22	3
A	getPhoneNumberObject()	0	15	4
A	sanitisePhoneNumber()	0	6	1
A	getDigits()	0	4	1
A	formatNumber()	0	11	2

<?php

namespace Digitonic\Validation\Services;

use libphonenumber\NumberParseException;
use libphonenumber\PhoneNumberUtil;
use libphonenumber\PhoneNumberFormat;

class AllowedRecipientsSanitiser
{
    protected $libPhoneNumber;

    public function __construct()
    {
        $this->libPhoneNumber = PhoneNumberUtil::getInstance();
    }

    /**
     * Get formatted phone number in E.164 standard without (+).
     * This method will return false if the number is not valid.
     *
     * @param string $phoneNumber
     *
     * @return mixed
     */
    public function getFormattedNumber(string $phoneNumber)
    {
        $phoneNumberObject = $this->getPhoneNumberObject($phoneNumber);

        if ($phoneNumberObject) {
            if (in_array(
                $this->libPhoneNumber->getRegionCodeForNumber($phoneNumberObject),
                config('digitonic.validation.allowed_mobile_origins')
            )) {
                // Format number to E.164 standard and remove '+'
                $formattedPhone = str_replace(
                    '+',
                    '',
                    $this->libPhoneNumber->format($phoneNumberObject, PhoneNumberFormat::E164)
                );

                return $formattedPhone;
            }
        }

        return false;
    }

    /**
     * Get phoneNumberObject.
     *
     * @param string $phoneNumber
     *
     * @return mixed
     */
    public function getPhoneNumberObject(string $phoneNumber)
    {
        try {
            if (!preg_match('/[A-Za-z]/', $phoneNumber)) {
                $phoneNumberObject = $this->libPhoneNumber->parse($this->sanitisePhoneNumber($phoneNumber), null);
                if ($this->libPhoneNumber->isPossibleNumber($phoneNumberObject)) {
                    return $phoneNumberObject;
                }
            }
        } catch (NumberParseException $exception) {
            // Do nothing
        }

        return false;
    }

    /**
     * @param string $phoneNumber
     * @return string
     */
    public function sanitisePhoneNumber(string $phoneNumber): string
    {
        $phoneNumber = $this->getDigits($phoneNumber);

        return $this->formatNumber($phoneNumber);
    }

    /**
     * @param string $phoneNumber
     * @return string
     */
    protected function getDigits(string $phoneNumber): string
    {
        return preg_replace('#\D+#', '', $phoneNumber);
    }

    /**
     * @param string $phoneNumber
     * @return string
     */
    protected function formatNumber(string $phoneNumber): string
    {
        // Format UK numbers starting with 07 or 7
        preg_match('/^0*(7\d{9})$/', $phoneNumber, $outputUkPhone);

        if ($outputUkPhone) {

The expression $outputUkPhone of type string[] is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

            return '+44' . $outputUkPhone[1];
        }

        return '+' . $phoneNumber;
    }
}