StandardResult::getInput() - Code Metrics - Inspection of "New Concrete API" - Respect/Validation - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#678)

by Henrique

created 2016-11-06 19:21 UTC

StandardResult::getInput() A

↳ Parent: StandardResult

Complexity

Conditions	1
Paths	1

Size

Total Lines	4
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	4
rs	10
c	0
b	0
f	0
cc	1
eloc	2
nc	1
nop	0

<?php

/*
 * This file is part of Respect/Validation.
 *
 * (c) Alexandre Gomes Gaigalas <[email protected]>
 *
 * For the full copyright and license information, please view the "LICENSE.md"
 * file that was distributed with this source code.
 */

namespace Respect\Validation;

/**
 * Class to handle validation.
 *
 * @author Henrique Moody <[email protected]>
 */
final class StandardResult implements Result
{
    /**
     * @var bool
     */
    private $isValid;

    /**
     * @var mixed
     */
    private $input;

    /**
     * @var Rule
     */
    private $rule;

    /**
     * @var Rule[]
     */
    private $children;

    /**
     * @var array
     */
    private $properties;

    /**
     * @param bool   $isValid
     * @param mixed  $input
     * @param Rule   $rule
     * @param array  $properties
     * @param Result $children...
/**
 * @param array $germany
 * @param array $ireland
 */
function finale($germany, $island) {
    return "2:1";
}
     */
    public function __construct($isValid, $input, Rule $rule, array $properties = [], Result ...$children)
    {
        $this->isValid = $isValid;
        $this->input = $input;
        $this->rule = $rule;
        $this->properties = $properties;
        $this->children = $children;

    }

    /**
     * {@inheritdoc}
     */
    public function isValid(): bool
    {
        return $this->isValid;
    }

    /**
     * {@inheritdoc}
     */
    public function getInput()
    {
        return $this->input;
    }

    /**
     * {@inheritdoc}
     */
    public function getRule(): Rule
    {
        return $this->rule;
    }

    /**
     * {@inheritdoc}
     */
    public function getChildren(): array
    {
        return $this->children;
    }

    /**
     * {@inheritdoc}
     */
    public function getProperties(): array
    {
        return $this->properties;
    }

    /**
     * {@inheritdoc}
     */
    public function with($isValid, array $properties = []): Result
    {
        return new self(
            $isValid,
            $this->getInput(),
            $this->getRule(),
            $properties + $this->getProperties(),
            ...$this->getChildren()
        );
    }
}


1			<?php
2
3			/*
4			* This file is part of Respect/Validation.
5			*
6			* (c) Alexandre Gomes Gaigalas <[email protected]>
7			*
8			* For the full copyright and license information, please view the "LICENSE.md"
9			* file that was distributed with this source code.
10			*/
11
12			namespace Respect\Validation;
13
14			/**
15			* Class to handle validation.
16			*
17			* @author Henrique Moody <[email protected]>
18			*/
19			final class StandardResult implements Result
20			{
21			/**
22			* @var bool
23			*/
24			private $isValid;
25
26			/**
27			* @var mixed
28			*/
29			private $input;
30
31			/**
32			* @var Rule
33			*/
34			private $rule;
35
36			/**
37			* @var Rule[]
38			*/
39			private $children;
40
41			/**
42			* @var array
43			*/
44			private $properties;
45
46			/**
47			* @param bool $isValid
48			* @param mixed $input
49			* @param Rule $rule
50			* @param array $properties
51			* @param Result $children...
			0 ignored issues – show Documentation introduced 2016-10-30 09:29 UTC by Report Bug Copy Issue Report There is no parameter named `$children...`. Did you maybe mean `$children`? This check looks for PHPDoc comments describing methods or function parameters that do not exist on the corresponding method or function. It has, however, found a similar but not annotated parameter which might be a good fit. Consider the following example. The parameter `$ireland` is not defined by the method `finale(...)`. /** * @param array $germany * @param array $ireland */ function finale($germany, $island) { return "2:1"; } The most likely cause is that the parameter was changed, but the annotation was not. Loading history...
52			*/
53			public function __construct($isValid, $input, Rule $rule, array $properties = [], Result ...$children)
54			{
55			$this->isValid = $isValid;
56			$this->input = $input;
57			$this->rule = $rule;
58			$this->properties = $properties;
59			$this->children = $children;
			0 ignored issues – show Documentation Bug introduced 2016-10-30 09:29 UTC by Report Bug Copy Issue Report It seems like `$children` of type `array<integer,object<Respect\Validation\Result>>` is incompatible with the declared type `array<integer,object<Respect\Validation\Rule>>` of property `$children`. Our type inference engine has found an assignment to a property that is incompatible with the declared type of that property. Either this assignment is in error or the assigned type should be added to the documentation/type hint for that property.. Loading history...
60			}
61
62			/**
63			* {@inheritdoc}
64			*/
65			public function isValid(): bool
66			{
67			return $this->isValid;
68			}
69
70			/**
71			* {@inheritdoc}
72			*/
73			public function getInput()
74			{
75			return $this->input;
76			}
77
78			/**
79			* {@inheritdoc}
80			*/
81			public function getRule(): Rule
82			{
83			return $this->rule;
84			}
85
86			/**
87			* {@inheritdoc}
88			*/
89			public function getChildren(): array
90			{
91			return $this->children;
92			}
93
94			/**
95			* {@inheritdoc}
96			*/
97			public function getProperties(): array
98			{
99			return $this->properties;
100			}
101
102			/**
103			* {@inheritdoc}
104			*/
105			public function with($isValid, array $properties = []): Result
106			{
107			return new self(
108			$isValid,
109			$this->getInput(),
110			$this->getRule(),
111			$properties + $this->getProperties(),
112			...$this->getChildren()
113			);
114			}
115			}
116

Respect / Validation

Scrutinizer GitHub App not installed

Pull Request — master (#678)

StandardResult::getInput() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like