App\Http\Middleware\AdminOnly

Complexity

Total Complexity

4

Size/Duplication

Total Lines	23
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	3

Test Coverage

Coverage

83.33%

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	handle()	0	13	4

<?php

namespace App\Http\Middleware;

use Closure;
use Illuminate\Support\Facades\Auth;

class AdminOnly
{
    /**
     * Handle an incoming request.
     *
     * @param  \Illuminate\Http\Request $request
     * @param  \Closure $next
     * @return mixed
     */
    public function handle($request, Closure $next, $guard = null)
    {
        if (!Auth::guard($guard)->user()->isAdmin()) {
            if ($request->ajax() || $request->wantsJson()) {
                return response(trans('general.text.unauthorized'), 401);

It seems like trans('general.text.unauthorized') targeting trans() can also be of type object<Illuminate\Contracts\Translation\Translator>; however, response() does only seem to accept string, maybe add an additional type check?

            } else {
                //FIXME: Errors not working
                return redirect('/')->withErrors(['msg', trans('general.text.unauthorized')]);
            }
        }

        return $next($request);
    }
}