Mascame\Artificer\Http\Controllers\UserController

Complexity

Total Complexity

41

Size/Duplication

Total Lines	228
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	6

Importance

Changes	2
Bugs	0	Features	0

20 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	unban()	0	4	1
A	isBanned()	0	14	3
A	ban()	0	4	1
A	addAttempt()	0	17	3
A	showLogin()	0	6	2
B	login()	0	26	4
A	successLoginRedirect()	0	3	1
A	failedLoginRedirect()	0	4	1
A	attempt()	0	3	2
A	getClosureAttempt()	0	3	1
A	getClosureCheckAuth()	0	3	1
A	checkAuth()	0	3	2
A	check()	0	3	1
A	onFailValidation()	0	8	1
A	getUser()	0	5	1
A	attemptLogin()	0	18	3
A	isValidUser()	0	10	3
A	logout()	0	6	1
C	authFilter()	0	23	8

<?php namespace Mascame\Artificer\Http\Controllers;

use Auth;
use Carbon\Carbon;
use Illuminate\Auth\EloquentUserProvider;
use Input;
use Mascame\Artificer\Options\AdminOption;
use Redirect;
use Session;
use Validator;
use View;

class UserController extends BaseController
{

    public $tries_key = 'artificer.user.login.tries';
    public $ban_key = 'artificer.user.login.banned';
    public $authProvider;

    public function __construct() {
        parent::__construct();

        $this->authProvider = new EloquentUserProvider(app('hash'), 'ArtificerUser');
    }

    /**
     * Unban user
     */
    private function unban()
    {
        Session::forget($this->ban_key);
    }

    /**
     * @return bool
     */
    private function isBanned()
    {
        if (Session::has($this->ban_key)) {
            $ban = Carbon::parse(Session::get($this->ban_key));

            if (! $ban->isPast()) {
                return true;
            }
        }

        $this->unban();

        return false;
    }

    /**
     * Ban user
     */
    private function ban()
    {
        Session::set($this->ban_key, Carbon::now()->addMinutes(AdminOption::get('auth.ban_time')));
    }

    /**
     *
     */
    private function addAttempt()
    {
        $tries = Session::get($this->tries_key);

        if (! $tries) {
            $tries = 1;
        } else {
            $tries++;
        }

        Session::set($this->tries_key, $tries);

        if ($tries >= AdminOption::get('auth.max_login_attempts')) {
            $this->ban();
            Session::forget($this->tries_key);
        }
    }

    /**
     * @return \Illuminate\Http\RedirectResponse|\Illuminate\View\View
     */
    public function showLogin()
    {
        if (Auth::check()) return Redirect::route('admin.home');

The method route() does not seem to exist on object<redirect>.

        return View::make($this->getView('pages.login'));
    }

    /**
     * @return $this|\Illuminate\Http\RedirectResponse
     */
    public function login()
    {
        if ($this->isBanned()) {
            return Redirect::route('admin.showlogin')->withErrors(array("You are banned for too many login attempts"));

The method route() does not seem to exist on object<redirect>.

        }

        $rules = array(
            'username' => 'required|email',
            'password' => 'required|min:3'
        );

        $validator = Validator::make(Input::all(), $rules);

        if ($validator->fails()) {
            return $this->onFailValidation($validator);
        }

        /*
         * Todo: add also to banning in case of fail auth attempt
         */
        if ($this->isValidUser($this->getUser())) {

It seems like $this->getUser() targeting Mascame\Artificer\Http\Controllers\UserController::getUser() can also be of type object<Mascame\Artificer\Http\Controllers\UserController>; however, Mascame\Artificer\Http\Controllers\UserController::isValidUser() does only seem to accept object<Illuminate\Database\Eloquent\Model>|null, maybe add an additional type check?

            return $this->successLoginRedirect();
        }

        return $this->failedLoginRedirect();
    }

    protected function successLoginRedirect() {
        return Redirect::route('admin.home');

The method route() does not seem to exist on object<redirect>.

    }

    protected function failedLoginRedirect() {
        return Redirect::route('admin.login')

The method route() does not seem to exist on object<redirect>.

            ->withInput(Input::except('password'))->withErrors(array('The user credentials are not correct or does not have access'));
    }

    protected static function attempt($attemptClosure, $credentials) {
        return is_callable($attemptClosure) ? $attemptClosure($credentials) : false;
    }

    protected static function getClosureAttempt() {
        return AdminOption::get('auth.attempt');
    }

    protected static function getClosureCheckAuth() {
        return AdminOption::get('auth.check');
    }

    protected static function checkAuth($checkClosure) {
        return is_callable($checkClosure) ? $checkClosure() : false;
    }

    public static function check() {
        return self::checkAuth(self::getClosureCheckAuth());
    }

    protected function onFailValidation($validator)
    {
        $this->addAttempt();

        return Redirect::route('admin.login')

The method route() does not seem to exist on object<redirect>.

            ->withErrors($validator)
            ->withInput();
    }

    /**
     * @return \Illuminate\Database\Eloquent\Model|null|static
     */
    protected function getUser()
    {
        return \Mascame\Artificer\Auth\ArtificerUser::where('email', '=', Input::get('username'))
            ->OrWhere('username', '=', Input::get('username'))->first();
    }

    /**
     * @param $user
     * @return bool
     */
    protected function attemptLogin($user)
    {
        $role_colum = AdminOption::get('auth.role_column');
        if (in_array($user->$role_colum, AdminOption::get('auth.roles'))) {

            $credentials = array(
                'email' => Input::get('username'),
                'password' => Input::get('password')
            );

            if (self::attempt(self::getClosureAttempt(), $credentials)) {
                return true;
            }

        }

        return false;
    }

    /**
     * @param \Illuminate\Database\Eloquent\Model|null $user
     * @return bool
     */
    protected function isValidUser($user)
    {
        if ($user) {
            if ($this->attemptLogin($user)) {
                return true;
            }
        }

        return false;
    }

    /**
     * @return \Illuminate\Http\RedirectResponse
     */
    public function logout()
    {
        Auth::logout();

        return Redirect::route('admin.showlogin');

The method route() does not seem to exist on object<redirect>.

    }


    public function authFilter() {
        $roles = AdminOption::get('auth.roles');
        $role_column = AdminOption::get('auth.role_column');

        if (Auth::guest()
            && \Route::currentRouteName() != 'admin.showlogin'
            && \Route::currentRouteName() != 'admin.login'
        ) {
            if (\Request::ajax()) {
                return \Response::make('Unauthorized', 401);
            } else {
                return Redirect::route('admin.showlogin');

The method route() does not seem to exist on object<redirect>.

            }
        } else {
            if (Auth::check()
                && \Route::currentRouteName() != 'admin.logout'
            ) {
                if (!in_array(Auth::user()->$role_column, $roles)) {
                    return Redirect::route('admin.logout');

The method route() does not seem to exist on object<redirect>.

                }
            }
        }
    }
}