Distilleries\Expendable\Listeners\UserListener

Complexity

Total Complexity

7

Size/Duplication

Total Lines	75
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	3

Test Coverage

Coverage

93.1%

Importance

Changes	5
Bugs	1	Features	3

3 Methods

Rating	Name	Duplication	Size	Complexity
B	handleLogin()	0	27	4
A	handleLogOut()	0	8	1
A	handleLockIncrement()	0	10	2

<?php namespace Distilleries\Expendable\Listeners;

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

class UserListener extends BaseListener
{

    /**
     * @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)

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

            {
                $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();
        }
    }
}