dev-think-one / nova-laravel-filemanager

src/Http/Middleware/Authorize.php

<?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;
    }
}