Yajra\CMS\Http\Controllers\AuthController

Complexity

Total Complexity

9

Size/Duplication

Total Lines	116
Duplicated Lines	0 %

Coupling/Cohesion

Components	3
Dependencies	7

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	showLoginForm()	0	4	1
A	authenticated()	0	16	4
A	credentials()	0	8	1
A	logout()	0	10	1
A	guard()	0	4	1

<?php

namespace Yajra\CMS\Http\Controllers;

use App\User;
use Auth;
use Illuminate\Foundation\Auth\AuthenticatesUsers;
use Illuminate\Http\Request;

class AuthController extends Controller
{
    use AuthenticatesUsers;

    /**
     * Where to redirect users after login / registration.
     *
     * @var string
     */
    protected $redirectPath = '/administrator';

    /**
     * Authentication guard.
     *
     * @var string
     */
    protected $guard = 'administrator';

    /**
     * Administrator login view.
     *
     * @var string
     */
    protected $loginView = 'admin::auth.login';

    /**
     * Where to redirect users after logout.
     *
     * @var string
     */
    protected $redirectAfterLogout = '/administrator';

    /**
     * Create a new authentication controller instance.
     */
    public function __construct()
    {
        $this->middleware('guest', ['except' => 'logout']);
    }

    /**
     * Show the application's login form.
     *
     * @return \Illuminate\Http\Response
     */
    public function showLoginForm()
    {
        return view($this->loginView);
    }

    /**
     * Handle event when the user was authenticated.
     *
     * @param \Illuminate\Http\Request $request
     * @param \App\User $user
     * @return \Illuminate\Http\RedirectResponse
     */
    public function authenticated(Request $request, User $user)

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

    {
        if ($user->blocked || ! $user->confirmed) {
            if ($user->blocked) {
                $message = 'Your account is currently banned from accessing the site!';
            } else {
                $message = 'Your account is not yet activated!';
            }
            auth($this->getGuard())->logout();

The method getGuard does not exist on object<Yajra\CMS\Http\Controllers\AuthController>? Since you implemented __call, maybe consider adding a @method annotation.

            flash()->error($message);

            return redirect()->route('administrator.login')->withErrors($message);
        }

        return redirect()->intended($this->redirectPath);
    }

    /**
     * Get the needed authorization credentials from the request.
     *
     * @param  \Illuminate\Http\Request $request
     * @return array
     */
    protected function credentials(Request $request)
    {
        $options = config('site.login.backend.options', [
            'administrator' => true,
        ]);

        return array_merge($request->only($this->username(), 'password'), $options);
    }

    /**
     * Log the user out of the application.
     *
     * @param  \Illuminate\Http\Request  $request
     * @return \Illuminate\Http\Response
     */
    public function logout(Request $request)
    {
        $this->guard()->logout();

        $request->session()->flush();

The method flush() does not seem to exist on object<Symfony\Component\HttpFoundation\Session\SessionInterface>.

        $request->session()->regenerate();

The method regenerate() does not seem to exist on object<Symfony\Component\HttpFoundation\Session\SessionInterface>.

        return redirect($this->redirectAfterLogout);
    }

    /**
     * Get the guard to be used during authentication.
     *
     * @return \Illuminate\Contracts\Auth\StatefulGuard
     */
    protected function guard()
    {
        return Auth::guard($this->guard);
    }
}