CheckResult::hasAttribute() - Code Metrics - Inspection of "timeout class" - phmLabs/Smoke - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( a71aed...831a7e )

by Nils

created 2018-11-21 12:55 UTC

CheckResult::hasAttribute() A

↳ Parent: CheckResult

Complexity

Conditions	3
Paths	3

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	9
rs	9.9666
c	0
b	0
f	0
cc	3
nc	3
nop	1

<?php

namespace whm\Smoke\Rules;

use phm\HttpWebdriverClient\Http\Response\UriAwareResponse;
use Psr\Http\Message\ResponseInterface;

class CheckResult
{
    const STATUS_SUCCESS = 'success';
    const STATUS_FAILURE = 'failure';
    const STATUS_SKIPPED = 'skipped';
    const STATUS_NONE = 'none';

    const HINT_KEY = '__hint';

    const IDENTIFIER_RULE_STANDARD = 'standard';
    const IDENTIFIER_RULE_WITHOUT_SMOKE_PREFIX = 'withoutSmokePrefix';

    private $status;
    private $value;
    private $message;

    /**
     * @var Attribute[]
     */
    private $attributes = array();
    private $ruleName;
    private $url;

    private $tool;
    private $identifierRule = self::IDENTIFIER_RULE_STANDARD;

    /**
     * @var UriAwareResponse
     */
    private $response;

    /**
     * Result constructor.
     *
     * @param $status
     * @param $value
     * @param $message
     */
    public function __construct($status, $message = '', $value = null, $url = null)
    {
        $this->status = $status;
        $this->value = $value;
        $this->message = $message;
        $this->url = $url;
    }

    /**
     * @return Attribute[]
     */
    public function getAttributes()
    {
        return $this->attributes;
    }

    /**
     * @return boolean
     */
    public function hasAttribute($key)
    {
        foreach($this->attributes as $attribute) {
            if($attribute->getKey() == $key) {
                return true;
            }
        }
        return false;
    }

    /**
     * @param array $attributes
/**
 * @param array $germany
 * @param array $ireland
 */
function finale($germany, $island) {
    return "2:1";
}
     */
    public function addAttribute(Attribute $attribute)
    {
        $this->attributes[] = $attribute;
    }

    public function setHint($message)
    {
        $this->addAttribute(new Attribute(self::HINT_KEY, $message, false));
    }

    /**
     * @return mixed
     */
    public function getStatus()
    {
        return $this->status;
    }

    /**
     * @return mixed
     */
    public function getValue()
    {
        return $this->value;
    }

    /**
     * @return string
     */
    public function getTool()
    {
        return $this->tool;
    }

    /**
     * @param string $tool
     */
    public function setTool($tool)
    {
        $this->tool = $tool;
    }

    /**
     * @return string
     */
    public function getIdentifierRule()
    {
        return $this->identifierRule;
    }

    /**
     * @param string $identifier
/**
 * @param array $germany
 * @param array $ireland
 */
function finale($germany, $island) {
    return "2:1";
}
     */
    public function setIdentifierRule($identifierRule)
    {
        $this->identifierRule = $identifierRule;
    }

    /**
     * @return string
     */
    public function getMessage()
    {
        return $this->message;
    }

    /**
     * @param string $message
     */
    public function setMessage($message)
    {
        $this->message = $message;
    }

    /**
     * @return UriAwareResponse
     */
    public function getResponse()
    {
        return $this->response;
    }

    /**
     * @param ResponseInterface $response
     */
    public function setResponse(ResponseInterface $response)
    {
        $this->response = $response;
class Alien {}

class Dalek extends Alien {}

class Plot
{
    /** @var  Dalek */
    public $villain;
}

$alien = new Alien();
$plot = new Plot();
if ($alien instanceof Dalek) {
    $plot->villain = $alien;
}
    }

    /**
     * @return mixed
     */
    public function getRuleName()
    {
        return $this->ruleName;
    }

    /**
     * @param mixed $ruleName
     */
    public function setRuleName($ruleName)
    {
        $this->ruleName = $ruleName;
    }

    /**
     * @return string
     */
    public function getUrl()
    {
        return $this->url;
    }
}


1			<?php
2
3			namespace whm\Smoke\Rules;
4
5			use phm\HttpWebdriverClient\Http\Response\UriAwareResponse;
6			use Psr\Http\Message\ResponseInterface;
7
8			class CheckResult
9			{
10			const STATUS_SUCCESS = 'success';
11			const STATUS_FAILURE = 'failure';
12			const STATUS_SKIPPED = 'skipped';
13			const STATUS_NONE = 'none';
14
15			const HINT_KEY = '__hint';
16
17			const IDENTIFIER_RULE_STANDARD = 'standard';
18			const IDENTIFIER_RULE_WITHOUT_SMOKE_PREFIX = 'withoutSmokePrefix';
19
20			private $status;
21			private $value;
22			private $message;
23
24			/**
25			* @var Attribute[]
26			*/
27			private $attributes = array();
28			private $ruleName;
29			private $url;
30
31			private $tool;
32			private $identifierRule = self::IDENTIFIER_RULE_STANDARD;
33
34			/**
35			* @var UriAwareResponse
36			*/
37			private $response;
38
39			/**
40			* Result constructor.
41			*
42			* @param $status
43			* @param $value
44			* @param $message
45			*/
46			public function __construct($status, $message = '', $value = null, $url = null)
47			{
48			$this->status = $status;
49			$this->value = $value;
50			$this->message = $message;
51			$this->url = $url;
52			}
53
54			/**
55			* @return Attribute[]
56			*/
57			public function getAttributes()
58			{
59			return $this->attributes;
60			}
61
62			/**
63			* @return boolean
64			*/
65			public function hasAttribute($key)
66			{
67			foreach($this->attributes as $attribute) {
68			if($attribute->getKey() == $key) {
69			return true;
70			}
71			}
72			return false;
73			}
74
75			/**
76			* @param array $attributes
			0 ignored issues – show Documentation introduced 2017-03-22 15:24 UTC by Report Bug Copy Issue Report There is no parameter named `$attributes`. Did you maybe mean `$attribute`? 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...
77			*/
78			public function addAttribute(Attribute $attribute)
79			{
80			$this->attributes[] = $attribute;
81			}
82
83			public function setHint($message)
84			{
85			$this->addAttribute(new Attribute(self::HINT_KEY, $message, false));
86			}
87
88			/**
89			* @return mixed
90			*/
91			public function getStatus()
92			{
93			return $this->status;
94			}
95
96			/**
97			* @return mixed
98			*/
99			public function getValue()
100			{
101			return $this->value;
102			}
103
104			/**
105			* @return string
106			*/
107			public function getTool()
108			{
109			return $this->tool;
110			}
111
112			/**
113			* @param string $tool
114			*/
115			public function setTool($tool)
116			{
117			$this->tool = $tool;
118			}
119
120			/**
121			* @return string
122			*/
123			public function getIdentifierRule()
124			{
125			return $this->identifierRule;
126			}
127
128			/**
129			* @param string $identifier
			0 ignored issues – show Documentation introduced 2018-03-25 19:12 UTC by Report Bug Copy Issue Report There is no parameter named `$identifier`. Did you maybe mean `$identifierRule`? 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...
130			*/
131			public function setIdentifierRule($identifierRule)
132			{
133			$this->identifierRule = $identifierRule;
134			}
135
136			/**
137			* @return string
138			*/
139			public function getMessage()
140			{
141			return $this->message;
142			}
143
144			/**
145			* @param string $message
146			*/
147			public function setMessage($message)
148			{
149			$this->message = $message;
150			}
151
152			/**
153			* @return UriAwareResponse
154			*/
155			public function getResponse()
156			{
157			return $this->response;
158			}
159
160			/**
161			* @param ResponseInterface $response
162			*/
163			public function setResponse(ResponseInterface $response)
164			{
165			$this->response = $response;
			0 ignored issues – show Documentation Bug introduced 2018-01-23 19:07 UTC by Report Bug Copy Issue Report `$response` is of type `object<Psr\Http\Message\ResponseInterface>`, but the property `$response` was declared to be of type `object<phm\HttpWebdriver...ponse\UriAwareResponse>`. Are you sure that you always receive this specific sub-class here, or does it make sense to add an `instanceof` check? Our type inference engine has found a suspicous assignment of a value to a property. This check raises an issue when a value that can be of a given class or a super-class is assigned to a property that is type hinted more strictly. Either this assignment is in error or an instanceof check should be added for that assignment. class Alien {} class Dalek extends Alien {} class Plot { /** @var Dalek */ public $villain; } $alien = new Alien(); $plot = new Plot(); if ($alien instanceof Dalek) { $plot->villain = $alien; } Loading history...
166			}
167
168			/**
169			* @return mixed
170			*/
171			public function getRuleName()
172			{
173			return $this->ruleName;
174			}
175
176			/**
177			* @param mixed $ruleName
178			*/
179			public function setRuleName($ruleName)
180			{
181			$this->ruleName = $ruleName;
182			}
183
184			/**
185			* @return string
186			*/
187			public function getUrl()
188			{
189			return $this->url;
190			}
191			}
192

phmLabs / Smoke

Push — master ( a71aed...831a7e )

CheckResult::hasAttribute() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like