ThinkOne\NovaLaravelFilemanager\Http\Middleware\Authorize

Complexity

Total Complexity

3

Size/Duplication

Total Lines	25
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	matchesTool()	0	3	1
A	handle()	0	5	2

<?php

namespace ThinkOne\NovaLaravelFilemanager\Http\Middleware;

use Laravel\Nova\Nova;
use ThinkOne\NovaLaravelFilemanager\NLFileManager;

class Authorize
{
    /**
     * Handle the incoming request.
     *
     * @param  \Illuminate\Http\Request  $request
     * @param  \Closure(\Illuminate\Http\Request):mixed  $next
     * @return \Illuminate\Http\Response
     */
    public function handle($request, $next)
    {
        $tool = collect(Nova::registeredTools())->first([$this, 'matchesTool']);

Laravel\Nova\Nova::registeredTools() of type array<integer,Laravel\Nova\Tool> is incompatible with the type Illuminate\Contracts\Support\Arrayable expected by parameter $value of collect().

        return optional($tool)->authorize($request) ? $next($request) : abort(403);

Are you sure the usage of abort(403) is correct as it seems to always return null.

    }

    /**
     * Determine whether this tool belongs to the package.
     *
     * @param  \Laravel\Nova\Tool  $tool
     * @return bool
     */
    public function matchesTool($tool)
    {
        return $tool instanceof NLFileManager;
    }
}