NameValidator::isValidNameError() - Code Metrics - Inspection of "Add initial support for schema validation (#118)" - digiaonline/graphql-php - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 8bc3a9...d5f8d6 )

by Christoffer

created 2018-03-16 23:10 UTC

NameValidator::isValidNameError() B

↳ Parent: NameValidator

Complexity

Conditions	7
Paths	3

Size

Total Lines	19
Code Lines	10

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	19
rs	8.2222
c	0
b	0
f	0
cc	7
eloc	10
nc	3
nop	2

<?php

namespace Digia\GraphQL\Util;

use Digia\GraphQL\Error\InvariantException;
use Digia\GraphQL\Error\ValidationException;
use Digia\GraphQL\Language\Node\NodeInterface;

class NameValidator
{
    /**
     * Upholds the spec rules about naming.
     *
     * @param string $name
     * @return string
     * @throws InvariantException
     * @throws ValidationException
     */
    function assertInvalidName(string $name): string

    {
        $error = $this->isValidNameError($name);

        if ($error) {
            throw $error;
        }

        return $name;
    }

    /**
     * Returns an Error if a name is invalid.
     *
     * @param string     $name
     * @param mixed|null $node
     * @return ValidationException
     * @throws InvariantException
     */
    function isValidNameError(string $name, $node = null): ?ValidationException

    {
        invariant(\is_string($name), 'Expected string');

        if (\strlen($name) > 1 && $name{0} === '_' && $name{1} === '_') {
            return new ValidationException(
                sprintf('Name "%s" must not begin with "__", which is reserved by GraphQL introspection.', $name),
                $node instanceof NodeInterface ? [$node] : null
            );
        }

        if (preg_match("/^[_a-zA-Z][_a-zA-Z0-9]*$/", $name) === 0) {
            return new ValidationException(
                sprintf('Names must match /^[_a-zA-Z][_a-zA-Z0-9]*$/ but "%s" does not.', $name),
                $node instanceof NodeInterface ? [$node] : null
            );
        }

        return null;
    }
}


1			<?php
2
3			namespace Digia\GraphQL\Util;
4
5			use Digia\GraphQL\Error\InvariantException;
6			use Digia\GraphQL\Error\ValidationException;
7			use Digia\GraphQL\Language\Node\NodeInterface;
8
9			class NameValidator
10			{
11			/**
12			* Upholds the spec rules about naming.
13			*
14			* @param string $name
15			* @return string
16			* @throws InvariantException
17			* @throws ValidationException
18			*/
19			function assertInvalidName(string $name): string
			0 ignored issues – show Best Practice introduced 2018-03-16 21:45 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history...
20			{
21			$error = $this->isValidNameError($name);
22
23			if ($error) {
24			throw $error;
25			}
26
27			return $name;
28			}
29
30			/**
31			* Returns an Error if a name is invalid.
32			*
33			* @param string $name
34			* @param mixed\|null $node
35			* @return ValidationException
36			* @throws InvariantException
37			*/
38			function isValidNameError(string $name, $node = null): ?ValidationException
			0 ignored issues – show Best Practice introduced 2018-03-16 21:45 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history...
39			{
40			invariant(\is_string($name), 'Expected string');
41
42			if (\strlen($name) > 1 && $name{0} === '_' && $name{1} === '_') {
43			return new ValidationException(
44			sprintf('Name "%s" must not begin with "__", which is reserved by GraphQL introspection.', $name),
45			$node instanceof NodeInterface ? [$node] : null
46			);
47			}
48
49			if (preg_match("/^[_a-zA-Z][_a-zA-Z0-9]*$/", $name) === 0) {
50			return new ValidationException(
51			sprintf('Names must match /^[_a-zA-Z][_a-zA-Z0-9]*$/ but "%s" does not.', $name),
52			$node instanceof NodeInterface ? [$node] : null
53			);
54			}
55
56			return null;
57			}
58			}
59

digiaonline / graphql-php

Push — master ( 8bc3a9...d5f8d6 )

NameValidator::isValidNameError() B

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like