Kontrolio\Rules\Core\Email::isValid()

Complexity

Conditions	6
Paths	5

Size

Total Lines	17
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    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();
    }