Mongi\Mongicommerce\Http\Middleware\AdminMiddleware

Complexity

Total Complexity

3

Size/Duplication

Total Lines	15
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	1

1 Method

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

<?php

namespace Mongi\Mongicommerce\Http\Middleware;

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

class AdminMiddleware
{
    public function handle($request, Closure $next)
    {
        if (!Auth::check()) {
            // I included this check because you have it, but it really should be part of your 'auth' middleware, most likely added as part of a route group.
            return redirect(route('admin.login'));
        }
        // Perform action
        $user = Auth::user();

        if ($user->admin == true) {

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

            return $next($request);
        } else {
            abort(403);
        }
    }
}