App\Http\Middleware\CheckBlockedUser::handle()

Complexity

Conditions	6
Paths	3

Size

Total Lines	10
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function handle($request, Closure $next)
{
    if (Auth::check() && Auth::user()->block === "yes") {

Accessing block on the interface Illuminate\Contracts\Auth\Authenticatable suggest that you code against a concrete implementation. How about adding an instanceof check?

        return redirect()->route('blocked');
    }
    if (env('MAINTENANCE_MODE') == 'true' && (Auth::check() && Auth::user()->role != 'admin')) {

Accessing role on the interface Illuminate\Contracts\Auth\Authenticatable suggest that you code against a concrete implementation. How about adding an instanceof check?

        return redirect(url(''));
    }

    return $next($request);
}
}