Limoncello\Auth\Authorization\PolicyInformation\Context

Complexity

Total Complexity

6

Size/Duplication

Total Lines	46
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

100%

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	has()	0	4	2
A	get()	0	8	2
A	getRequest()	0	4	1

<?php declare(strict_types=1);

namespace Limoncello\Auth\Authorization\PolicyInformation;

/**
 * Copyright 2015-2019 info@neomerx.com
 *
 * 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) {

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().

            return parent::get($key);
        }

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

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