Context::has() - Code Metrics - limoncello-php-dist/auth - Measure and Improve Code Quality continuously with Scrutinizer

Context::has() A
last analyzed 2019-06-17 07:44 UTC

↳ Parent: Context

Complexity

Conditions	2
Paths	2

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	1
CRAP Score	2

Importance

Changes

Metric	Value
dl	0
loc	4
ccs	1
cts	1
cp	1
rs	10
c	0
b	0
f	0
cc	2
nc	2
nop	1
crap	2

<?php declare(strict_types=1);

namespace Limoncello\Auth\Authorization\PolicyInformation;

/**
 * Copyright 2015-2019 [email protected]
 *
 * Licensed under the Apache License, Version 2.0 (the "License");
 * you may not use this file except in compliance with the License.
 * You may obtain a copy of the License at
 *
 * http://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS,
 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 * See the License for the specific language governing permissions and
 * limitations under the License.
 */

use Limoncello\Auth\Contracts\Authorization\PolicyInformation\ContextInterface;
use Limoncello\Auth\Contracts\Authorization\PolicyEnforcement\RequestInterface;
use Limoncello\Container\Container;

/**
 * @package Limoncello\Auth
 */
class Context extends Container implements ContextInterface
{
    /**
     * @var RequestInterface
     */
    private $request;

    /**
     * @param RequestInterface $request
     * @param array            $contextDefinitions
     */
    public function __construct(RequestInterface $request, array $contextDefinitions = [])
    {
        parent::__construct([RequestInterface::class => $request] + $contextDefinitions);

        $this->request = $request;
    }

    /**
     * @inheritdoc
     */
    public function has($key): bool
    {
        return parent::has($key) === true || $this->getRequest()->has($key) === true;
    }

    /**
     * @inheritdoc
     */
    public function get($key)
    {
        if (parent::has($key) === true) {
class Daddy
{
    protected function getFirstName()
    {
        return "Eidur";
    }

    protected function getSurName()
    {
        return "Gudjohnsen";
    }
}

class Son
{
    public function getFirstName()
    {
        return parent::getSurname();
    }
}
            return parent::get($key);
        }

        return $this->getRequest()->get($key);
    }

    /**
     * @return RequestInterface
     */
    public function getRequest(): RequestInterface
    {
        return $this->request;
    }
}


1		<?php declare(strict_types=1);
2
3		namespace Limoncello\Auth\Authorization\PolicyInformation;
4
5		/**
6		* Copyright 2015-2019 [email protected]
7		*
8		* Licensed under the Apache License, Version 2.0 (the "License");
9		* you may not use this file except in compliance with the License.
10		* You may obtain a copy of the License at
11		*
12		* http://www.apache.org/licenses/LICENSE-2.0
13		*
14		* Unless required by applicable law or agreed to in writing, software
15		* distributed under the License is distributed on an "AS IS" BASIS,
16		* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
17		* See the License for the specific language governing permissions and
18		* limitations under the License.
19		*/
20
21		use Limoncello\Auth\Contracts\Authorization\PolicyInformation\ContextInterface;
22		use Limoncello\Auth\Contracts\Authorization\PolicyEnforcement\RequestInterface;
23		use Limoncello\Container\Container;
24
25		/**
26		* @package Limoncello\Auth
27		*/
28		class Context extends Container implements ContextInterface
29		{
30		/**
31		* @var RequestInterface
32		*/
33		private $request;
34
35		/**
36		* @param RequestInterface $request
37	33	* @param array $contextDefinitions
38		*/
39	33	public function __construct(RequestInterface $request, array $contextDefinitions = [])
40		{
41	33	parent::__construct([RequestInterface::class => $request] + $contextDefinitions);
42
43		$this->request = $request;
44		}
45
46		/**
47	26	* @inheritdoc
48		*/
49	26	public function has($key): bool
50		{
51		return parent::has($key) === true \|\| $this->getRequest()->has($key) === true;
52		}
53
54		/**
55	24	* @inheritdoc
56		*/
57	24	public function get($key)
58	11	{
59		if (parent::has($key) === true) {
		0 ignored issues – show Comprehensibility Bug introduced 2017-05-30 16:31 UTC by Report Bug Copy Issue Report It seems like you call parent on a different method (`has()` instead of `get()`). Are you sure this is correct? If so, you might want to change this to `$this->has()`. This check looks for a call to a parent method whose name is different than the method from which it is called. Consider the following code: class Daddy { protected function getFirstName() { return "Eidur"; } protected function getSurName() { return "Gudjohnsen"; } } class Son { public function getFirstName() { return parent::getSurname(); } } The `getFirstName()` method in the `Son` calls the wrong method in the parent class. Loading history...
60		return parent::get($key);
61	21	}
62
63		return $this->getRequest()->get($key);
64		}
65
66		/**
67	24	* @return RequestInterface
68		*/
69	24	public function getRequest(): RequestInterface
70		{
71		return $this->request;
72		}
73		}
74

limoncello-php-dist / auth

Context::has() A last analyzed 2019-06-17 07:44 UTC

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like

Context::has() A
last analyzed 2019-06-17 07:44 UTC