Basic::validate() - Code Metrics - Inspection of "Daily Inspection: Merge branch 'master' into FET-1..." - eventespresso/event-espresso-core - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Branch — FET-10766-extract-activation-d... (a650cc)

unknown

created 2017-12-16 21:13 UTC

Basic::validate() C

↳ Parent: Basic

Complexity

Conditions	10
Paths	10

Size

Total Lines	65
Code Lines	36

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	10
eloc	36
nc	10
nop	1
dl	0
loc	65
rs	6.2553
c	0
b	0
f	0

How to fix Long Method Complexity

<?php

namespace EventEspresso\core\domain\services\validation\email\strategies;

use EventEspresso\core\domain\services\validation\email\EmailValidationException;
use EventEspresso\core\domain\services\validation\email\EmailValidatorInterface;

defined('EVENT_ESPRESSO_VERSION') || exit('No direct script access allowed');



/**
 * Class Basic
 * Performs super basic email validation
 *
 * @package        Event Espresso
 * @author         Mike Nelson
 * @since          $VID:$
 */
class Basic implements EmailValidatorInterface
{

    /**
     * @param string $email_address
     * @return bool
     * @throws EmailValidationException
     */
    public function validate($email_address)
    {
        if (! preg_match('/^.+\@\S+$/', $email_address)) {
            // email not in correct {string}@{string} format
            throw new EmailValidationException(
                esc_html__('Email does not have the required @ sign.', 'event_espresso')
            );
        }
        $atIndex = $this->getAtIndex($email_address);
        $local = $this->getLocalPartOfEmail($email_address, $atIndex);
        $localLen = strlen($local);
        if ($localLen < 1) {
            //no local part
            throw new EmailValidationException(
                esc_html__('Email local-part (before the @) is required.', 'event_espresso')
            );
        }
        if ($localLen > 64) {
            // local part length exceeded
            throw new EmailValidationException(
                esc_html__('Email local-part (before the @) is too long.', 'event_espresso')
            );
        }
        if ($local[0] === '.') {
            // local part starts with '.'
            throw new EmailValidationException(
                esc_html__('Email local-part (before the @) must not begin with a period.', 'event_espresso')
            );
        }
        if ($local[$localLen - 1] === '.') {
            // local part starts or ends with '.'
            throw new EmailValidationException(
                esc_html__('Email local-part (before the @) must not end with a period.', 'event_espresso')
            );
        }
        if (preg_match('/\\.\\./', $local)) {
            // local part has two consecutive dots
            throw new EmailValidationException(
                esc_html__(
                    'Email local-part (before the @) must not have two consecutive periods.',
                    'event_espresso'
                )
            );
        }
        $domain = $this->getDomainPartOfEmail($email_address, $atIndex);
        $domainLen = strlen($domain);
        if ($domainLen < 1) {
            throw new EmailValidationException(
                esc_html__('Email domain (after the @) is required.', 'event_espresso')
            );
        }
        if ($domainLen > 255) {
            // domain part length exceeded
            throw new EmailValidationException(
                esc_html__('Email domain (after the @) is too long.', 'event_espresso')
            );
        }
        if (preg_match('/\\.\\./', $domain)) {
            // domain part has two consecutive dots
            throw new EmailValidationException(
                esc_html__('Email domain (after the @) must not have two consecutive periods.', 'event_espresso')
            );
        }
        return true;
    }



    /**
     * returns the location of the @ symbol
     *
     * @param string $email_address
     * @return bool|string
     */
    protected function getAtIndex($email_address)
    {
        return strrpos($email_address, '@');
    }



    /**
     * Gets the local part of the email
     *
     * @param string $email_address
     * @param bool|int $atIndex
     * @return bool|string
     */
    protected function getLocalPartOfEmail($email_address, $atIndex)
    {
        return substr($email_address, 0, $atIndex);
    }



    /**
     * Gets the domain part of the email
     *
     * @param string   $email_address
     * @param bool|int $atIndex
     * @return bool|string
     */
    protected function getDomainPartOfEmail($email_address, $atIndex)
    {
        return substr($email_address, $atIndex + 1);
    }
}
// End of file Basic.php
// Location: core\services\validation/Basic.php


1			<?php
2
3			namespace EventEspresso\core\domain\services\validation\email\strategies;
4
5			use EventEspresso\core\domain\services\validation\email\EmailValidationException;
6			use EventEspresso\core\domain\services\validation\email\EmailValidatorInterface;
7
8			defined('EVENT_ESPRESSO_VERSION') \|\| exit('No direct script access allowed');
9
10
11
12			/**
13			* Class Basic
14			* Performs super basic email validation
15			*
16			* @package Event Espresso
17			* @author Mike Nelson
18			* @since $VID:$
19			*/
20			class Basic implements EmailValidatorInterface
21			{
22
23			/**
24			* @param string $email_address
25			* @return bool
26			* @throws EmailValidationException
27			*/
28			public function validate($email_address)
29			{
30			if (! preg_match('/^.+\@\S+$/', $email_address)) {
31			// email not in correct {string}@{string} format
32			throw new EmailValidationException(
33			esc_html__('Email does not have the required @ sign.', 'event_espresso')
34			);
35			}
36			$atIndex = $this->getAtIndex($email_address);
37			$local = $this->getLocalPartOfEmail($email_address, $atIndex);
38			$localLen = strlen($local);
39			if ($localLen < 1) {
40			//no local part
41			throw new EmailValidationException(
42			esc_html__('Email local-part (before the @) is required.', 'event_espresso')
43			);
44			}
45			if ($localLen > 64) {
46			// local part length exceeded
47			throw new EmailValidationException(
48			esc_html__('Email local-part (before the @) is too long.', 'event_espresso')
49			);
50			}
51			if ($local[0] === '.') {
52			// local part starts with '.'
53			throw new EmailValidationException(
54			esc_html__('Email local-part (before the @) must not begin with a period.', 'event_espresso')
55			);
56			}
57			if ($local[$localLen - 1] === '.') {
58			// local part starts or ends with '.'
59			throw new EmailValidationException(
60			esc_html__('Email local-part (before the @) must not end with a period.', 'event_espresso')
61			);
62			}
63			if (preg_match('/\\.\\./', $local)) {
64			// local part has two consecutive dots
65			throw new EmailValidationException(
66			esc_html__(
67			'Email local-part (before the @) must not have two consecutive periods.',
68			'event_espresso'
69			)
70			);
71			}
72			$domain = $this->getDomainPartOfEmail($email_address, $atIndex);
73			$domainLen = strlen($domain);
74			if ($domainLen < 1) {
75			throw new EmailValidationException(
76			esc_html__('Email domain (after the @) is required.', 'event_espresso')
77			);
78			}
79			if ($domainLen > 255) {
80			// domain part length exceeded
81			throw new EmailValidationException(
82			esc_html__('Email domain (after the @) is too long.', 'event_espresso')
83			);
84			}
85			if (preg_match('/\\.\\./', $domain)) {
86			// domain part has two consecutive dots
87			throw new EmailValidationException(
88			esc_html__('Email domain (after the @) must not have two consecutive periods.', 'event_espresso')
89			);
90			}
91			return true;
92			}
93
94
95
96			/**
97			* returns the location of the @ symbol
98			*
99			* @param string $email_address
100			* @return bool\|string
101			*/
102			protected function getAtIndex($email_address)
103			{
104			return strrpos($email_address, '@');
105			}
106
107
108
109			/**
110			* Gets the local part of the email
111			*
112			* @param string $email_address
113			* @param bool\|int $atIndex
114			* @return bool\|string
115			*/
116			protected function getLocalPartOfEmail($email_address, $atIndex)
117			{
118			return substr($email_address, 0, $atIndex);
119			}
120
121
122
123			/**
124			* Gets the domain part of the email
125			*
126			* @param string $email_address
127			* @param bool\|int $atIndex
128			* @return bool\|string
129			*/
130			protected function getDomainPartOfEmail($email_address, $atIndex)
131			{
132			return substr($email_address, $atIndex + 1);
133			}
134			}
135			// End of file Basic.php
136			// Location: core\services\validation/Basic.php
137

eventespresso / event-espresso-core

Branch — FET-10766-extract-activation-d... (a650cc)

Basic::validate() C

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

Duplication Side-by-Side

Filter issues like