Distilleries\Expendable\Listeners\UserListener

Complexity

Total Complexity

7

Size/Duplication

Total Lines	72
Duplicated Lines	0 %

Test Coverage

Coverage

95%

Importance

Changes	4
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	handleLockIncrement()	0	8	2
A	handleLogOut()	0	5	1
A	handleLogin()	0	25	4

<?php namespace Distilleries\Expendable\Listeners;

use Distilleries\Expendable\Contracts\LockableContract;
use Distilleries\Expendable\Helpers\UserUtils;
use Distilleries\Expendable\Models\User;

class UserListener extends BaseListener

The type Distilleries\Expendable\Listeners\BaseListener was not found. Maybe you did not declare it correctly or list all dependencies?

{

    /**
     * @var array[
     * 'user.login'=>[
     *      'action'=>'handleLogin',
     *      'priority'=>0
     *  ]
     * ]
     *
     */
    protected $events = [
        'user.login'    => [
            'action'   => 'handleLogin',
            'priority' => 0,
        ],
        'user.logout'   => [
            'action'   => 'handleLogOut',
            'priority' => 0,
        ],
        'user.security' => [
            'action'   => 'handleLockIncrement',
            'priority' => 0,
        ]
    ];

    public function handleLogin(LockableContract $model)
    {


        if ($model->email != '') {

Accessing email on the interface Distilleries\Expendable\Contracts\LockableContract suggest that you code against a concrete implementation. How about adding an instanceof check?

            $model->unlock();
        }

        $areaServices = [];

        $role = $model->role;

Accessing role on the interface Distilleries\Expendable\Contracts\LockableContract suggest that you code against a concrete implementation. How about adding an instanceof check?

        if (!empty($role))
        {
            foreach ($model->role->permissions as $permission)
            {
                $areaServices[] = $permission->service->action;
            }

        }


        UserUtils::setArea($areaServices);
        UserUtils::setIsLoggedIn();
        UserUtils::setDisplayAllStatus();

    }

    public function handleLogOut()
    {
        UserUtils::forgotArea();
        UserUtils::forgotIsLoggedIn();
        UserUtils::forgotDisplayAllStatus();


    }

    public function handleLockIncrement($email)
    {
        $model = app('Distilleries\Expendable\Contracts\LockableContract');
        $user  = $model->where('email', $email)->get()->last();

        if (!empty($user))
        {
            $user->incrementLock();
        }
    }
}