Tymon\JWTAuth\Http\Parser

Complexity

Total Complexity

7

Size/Duplication

Total Lines	76
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Test Coverage

Coverage

100%

Importance

Changes	3
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	setChainOrder()	0	6	1
A	parseToken()	0	12	3
A	hasToken()	0	4	1
A	setRequest()	0	6	1

<?php

/*
 * This file is part of jwt-auth
 *
 * (c) Sean Tymon <tymon148@gmail.com>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Tymon\JWTAuth\Http;

use Illuminate\Http\Request;

class Parser
{
    /**
     * @var array
     */
    private $chain;

    /**
     * @var \Illuminate\Http\Request
     */
    protected $request;

    /**
     * @param \Illuminate\Http\Request $request
     */
    public function __construct(Request $request, array $chain = [])

You have injected the Request via parameter $request. This is generally not recommended as there might be multiple instances during a request cycle (f.e. when using sub-requests). Instead, it is recommended to inject the RequestStack and retrieve the current request each time you need it via getCurrentRequest().

    {
        $this->request = $request;
        $this->chain = $chain;
    }

    /**
     * Set the order of the parser chain
     *
     * @param  array  $chain
     */
    public function setChainOrder(array $chain)
    {
        $this->chain = $chain;

        return $this;
    }

    /**
     * Iterate throught the parsers and attempt to retrieve
     * a value, otherwise return null
     *
     * @return string|null
     */
    public function parseToken()
    {
        foreach ($this->chain as $parser) {
            $response = $parser->parse($this->request);

            if ($response !== null) {
                return $response;
            }
        }

        return null;
    }

    /**
     * Check whether a token exists in the chain
     *
     * @return  boolean
     */
    public function hasToken()
    {
        return $this->parseToken() !== null;
    }

    /**
     * Set the request instance.
     *
     * @param \Illuminate\Http\Request $request
     *
     * @return Parser
     */
    public function setRequest(Request $request)

You have injected the Request via parameter $request. This is generally not recommended as there might be multiple instances during a request cycle (f.e. when using sub-requests). Instead, it is recommended to inject the RequestStack and retrieve the current request each time you need it via getCurrentRequest().

    {
        $this->request = $request;

        return $this;
    }
}