TypeHints\Unused\Parser\Action\ClosureParser

Complexity

Total Complexity

8

Size/Duplication

Total Lines	89
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	getContent()	0	3	1
A	parse()	0	5	1
A	__construct()	0	3	1
A	resolveMethod()	0	3	1
A	getRoute()	0	5	1
A	getErrors()	0	3	1
A	fetchContent()	0	8	1
A	getClosure()	0	3	1

<?php

namespace TypeHints\Unused\Parser\Action;

use Illuminate\Routing\Route;
use ReflectionFunction;

class ClosureParser implements ParserActionInterface
{
    /**
     * @var Illuminate\Routing\Route

The type TypeHints\Unused\Parser\Action\Illuminate\Routing\Route was not found. Did you mean Illuminate\Routing\Route? If so, make sure to prefix the type with \.

     */
    protected $route;

    /**
     * @var array
     */
    protected $content = [];

    /**
     * @var array
     */
    protected $errors = [];

    /**
     * @param Route $route
     */
    public function __construct(Route $route)
    {
        $this->route = $route;

It seems like $route of type Illuminate\Routing\Route is incompatible with the declared type TypeHints\Unused\Parser\Action\Illuminate\Routing\Route of property $route.

    }

    /**
     * @return TypeHints\Unused\Parser\Action\ParserActionInterface

The type TypeHints\Unused\Parser\Action\TypeHints\Unused\Parser\Action\ParserActionInterface was not found. Did you mean TypeHints\Unused\Parser\Action\ParserActionInterface? If so, make sure to prefix the type with \.

     */
    public function parse(): ParserActionInterface
    {
        $this->content = $this->fetchContent();

        return $this;

The expression return $this returns the type TypeHints\Unused\Parser\Action\ClosureParser which is incompatible with the documented return type TypeHints\Unused\Parser\Action\TypeHints\Unused\Parser\Action\ParserActionInterface.

    }

    /**
     * @return ReflectionFunction
     */
    protected function resolveMethod(): ReflectionFunction
    {
        return new ReflectionFunction($this->getClosure());
    }

    /**
     * @return array
     */
    protected function fetchContent(): array
    {
        $method = $this->resolveMethod();

        return array_slice(
            file($method->getFileName()),

It seems like file($method->getFileName()) can also be of type false; however, parameter $array of array_slice() does only seem to accept array, maybe add an additional type check?

            $method->getStartLine() - 1,
            $method->getEndLine() - $method->getStartLine() + 1
        );
    }

    /**
     * @return array
     */
    public function getContent(): array
    {
        return $this->content;
    }

    /**
     * @return array
     */
    public function getErrors(): array
    {
        return $this->errors;
    }

    /**
     * @return callable
     */
    public function getClosure(): callable
    {
        return $this->route->getAction('uses');
    }

    /**
     * @return array
     */
    public function getRoute(): array
    {
        return array_merge($this->route->getAction(), [
            'methods' => $this->route->methods(),
            'uri'     => $this->route->uri,
        ]);
    }
}