App\Http\Middleware\Authenticate

Complexity

Total Complexity

6

Size/Duplication

Total Lines	64
Duplicated Lines	0 %

Importance

Changes	2
Bugs	1	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	terminate()	0	4	1
A	redirectTo()	0	4	2
A	handle()	0	9	2
A	__construct()	0	3	1

<?php

namespace App\Http\Middleware;

use Auth;
use Closure;
use Illuminate\Auth\Middleware\Authenticate as Middleware;
use Illuminate\Contracts\Auth\Guard;
use Illuminate\Http\Request;
use Illuminate\Support\Facades\Log;
use Illuminate\Support\Facades\Route;

class Authenticate extends Middleware
{
    /**
     * The Guard implementation.
     *
     * @var Guard
     */
    protected $auth;

    /**
     * Create a new filter instance.
     *
     * @param Guard $auth
     *
     * @return void
     */
    public function __construct(Guard $auth)
    {
        $this->auth = $auth;
    }

    /**
     * Get the path the user should be redirected to when they are not authenticated.
     *
     * @param  \Illuminate\Http\Request  $request
     * @return string|null
     */
    protected function redirectTo($request)
    {
        if (! $request->expectsJson()) {
            return route('login');
        }
    }

    /**
     * Handle an incoming request.
     *
     * @param \Illuminate\Http\Request $request
     * @param \Closure                 $next
     *
     * @return mixed
     */
    public function handle($request, Closure $next)
    {
        if (! $this->auth->check()) {
            return redirect()->to('/login')
                ->with('status', 'success')
                ->with('message', 'Please login.');
        }

        return $next($request);
    }

    /**
     * Log a termination.
     * @param \Illuminate\Http\Request $request
     * @param $response
     *
     * @return void
     */
    public function terminate($request, $response)

The parameter $response is not used and could be removed.

The parameter $request is not used and could be removed.

    {
        $user = Auth::user();

The assignment to $user is dead and can be removed.

        $currentRoute = Route::currentRouteName();

The assignment to $currentRoute is dead and can be removed.

        // Log::info('Authenticate middleware was used: '.$currentRoute.'. ', [$user]);
    }
}