Kontrolio\Rules\Core\Email

Complexity

Total Complexity

11

Size/Duplication

Total Lines	76
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	checkHost()	0	3	3
A	isValid()	0	17	6
A	checkMX()	0	3	1

<?php

namespace Kontrolio\Rules\Core;

use Kontrolio\Rules\AbstractRule;

/**
 * Email validation rule.
 *
 * @package Kontrolio\Rules\Core
 */
class Email extends AbstractRule
{
    /**
     * Whether to check MX record.
     *
     * @var bool
     */
    private $mx;

    /**
     * Whether to check host.
     *
     * @var bool
     */
    private $host;

    /**
     * Email validation rule constructor.
     *
     * @param bool $mx
     * @param bool $host
     */
    public function __construct($mx = false, $host = false)
    {
        $this->mx = $mx;
        $this->host = $host;
    }

    /**
     * Validates input.
     *
     * @param mixed $input
     *
     * @return bool
     */
    public function isValid($input = null)
    {
        if (!preg_match('/^.+\@\S+\.\S+$/', $input)) {

It seems like $input can also be of type null; however, parameter $subject of preg_match() does only seem to accept string, maybe add an additional type check?

            return false;
        }

        $host = substr($input, strrpos($input, '@') + 1);

It seems like $input can also be of type null; however, parameter $haystack of strrpos() does only seem to accept string, maybe add an additional type check?

It seems like $input can also be of type null; however, parameter $string of substr() does only seem to accept string, maybe add an additional type check?

        if ($this->mx && !$this->checkMX($host)) {
            $this->violations[] = 'mx';
        }

        if ($this->host && !$this->checkHost($host)) {
            $this->violations[] = 'host';
        }

        return !$this->hasViolations();
    }

    /**
     * Checks DNS records for MX type.
     *
     * @param string $host
     *
     * @return bool
     */
    private function checkMX($host)
    {
        return checkdnsrr($host, 'MX');
    }

    /**
     * Check if one of MX, A or AAAA DNS RR exists.
     *
     * @param string $host
     *
     * @return bool
     */
    private function checkHost($host)
    {
        return $this->checkMX($host) || (checkdnsrr($host, 'A') || checkdnsrr($host, 'AAAA'));
    }
}