Bmatovu\RouteList\View\Components\Table

Complexity

Total Complexity

10

Size/Duplication

Total Lines	105
Duplicated Lines	28.57 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	render()	0	9	1
A	getRoutes()	16	22	3
A	getRouteMiddleware()	0	6	2
A	matches()	14	14	3

<?php

namespace Bmatovu\RouteList\View\Components;

use Illuminate\Routing\Router;
use Illuminate\View\Component;

class Table extends Component
{
    /**
     * Router.
     *
     * @var \Illuminate\Routing\Router
     */
    protected $router;

    /**
     * Table ID.
     *
     * @var string
     */
    protected $id;

    /**
     * Create a new component instance.
     *
     * @param string                     $id
     * @param \Illuminate\Routing\Router $router
     *
     * @return void

Adding a @return annotation to constructors is generally not recommended as a constructor does not have a meaningful return value.

     */
    public function __construct(Router $router, $id = 'routes')
    {
        $this->router = $router;
        $this->id = $id;
    }

    /**
     * Get the view / contents that represent the component.
     *
     * @return \Illuminate\View\View
     */
    public function render()
    {
        $routes = $this->getRoutes();

        return view('route-list::components.table', [

The expression view('route-list::components.table', array('id' => $this->id, 'routes' => $routes)); of type Illuminate\Contracts\View\View|Illuminate\Contracts\View\Factory adds the type Illuminate\Contracts\View\Factory to the return on line 47 which is incompatible with the return type declared by the abstract method Illuminate\View\Component::render of type Illuminate\Contracts\View\View|Illuminate\Contracts\Support\Htmlable|Closure|string.

            'id' => $this->id,
            'routes' => $routes,
        ]);
    }

    protected function getRoutes()
    {
        return collect($this->router->getRoutes())
            ->filter(function ($route) {
                return ! $this->matches(config('route-list.excluded'), $route->uri);
            })->map(function ($route) {

This code seems to be duplicated across your project.

                $name = $route->action['as'] ?? '';

                if ($name && $this->matches(['/^generated::/'], $name)) {
                    $name = '';
                }

                return [
                    // 'host' => $route->action['where'],
                    'uri' => $route->uri,
                    'name' => $name,
                    'methods' => $route->methods,
                    'action' => $route->action['controller'] ?? 'Closure',
                    'middleware' => $this->getRouteMiddleware($route),
                ];
            });
    }

    /**
     * Get route middleware.
     *
     * @param \Illuminate\Routing\Route $route
     *
     * @return string
     */
    protected function getRouteMiddleware($route)
    {
        return collect($route->gatherMiddleware())->map(function ($middleware) {
            return $middleware instanceof \Closure ? 'Closure' : $middleware;
        })->implode(', ');
    }

    /**
     * Perform a regular expression match.
     *
     * @param array  $patterns
     * @param string $subject
     *
     * @return bool
     */
    protected function matches($patterns, $subject)

This method seems to be duplicated in your project.

    {
        $isMatched = false;

        foreach ($patterns as $pattern) {
            if (preg_match($pattern, $subject)) {
                $isMatched = true;

                break;
            }
        }

        return $isMatched;
    }
}