Arcanedev\LaravelAuth\Observers\UserObserver

Complexity

Total Complexity

13

Size/Duplication

Total Lines	138
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Test Coverage

Coverage

94.59%

Importance

Changes	3
Bugs	0	Features	0

10 Methods

Rating	Name	Duplication	Size	Complexity
A	creating()	0	8	2
A	created()	0	4	1
A	updating()	0	4	1
A	updated()	0	4	1
A	saving()	0	4	1
A	saved()	0	4	1
A	deleting()	0	12	3
A	deleted()	0	4	1
A	restoring()	0	4	1
A	restored()	0	4	1

<?php namespace Arcanedev\LaravelAuth\Observers;

use Arcanedev\LaravelAuth\Bases\ModelObserver;
use Arcanedev\LaravelAuth\Services\UserConfirmator;
use Arcanesoft\Contracts\Auth\Models\User;

/**
 * Class     UserObserver
 *
 * @package  Arcanedev\LaravelAuth\Observers
 * @author   ARCANEDEV <arcanedev.maroc@gmail.com>
 */
class UserObserver extends ModelObserver
{
    /* ------------------------------------------------------------------------------------------------
     |  Model Events
     | ------------------------------------------------------------------------------------------------
     */
    /**
     * Eloquent 'creating' event method.
     *
     * @param  \Arcanedev\LaravelAuth\Models\User|User  $user
     *
     * @return mixed
     */
    public function creating(User $user)
    {
        if (UserConfirmator::isEnabled()) {
            $user->confirmation_code = UserConfirmator::generateCode();

Accessing confirmation_code on the interface Arcanesoft\Contracts\Auth\Models\User suggest that you code against a concrete implementation. How about adding an instanceof check?

        }

        $this->event->fire('auth.users.creating', compact('user'));
    }

    /**
     * Eloquent 'created' event method.
     *
     * @param  \Arcanedev\LaravelAuth\Models\User|User  $user
     *
     * @return mixed
     */
    public function created(User $user)
    {
        $this->event->fire('auth.users.created', compact('user'));
    }

    /**
     * Eloquent 'updating' event method.
     *
     * @param  \Arcanedev\LaravelAuth\Models\User|User  $user
     *
     * @return mixed
     */
    public function updating(User $user)
    {
        $this->event->fire('auth.users.updating', compact('user'));
    }

    /**
     * Eloquent 'updated' event method.
     *
     * @param  \Arcanedev\LaravelAuth\Models\User|User  $user
     *
     * @return mixed
     */
    public function updated(User $user)
    {
        $this->event->fire('auth.users.updated', compact('user'));
    }

    /**
     * Eloquent 'saving' event method.
     *
     * @param  \Arcanedev\LaravelAuth\Models\User|User  $user
     *
     * @return mixed
     */
    public function saving(User $user)
    {
        $this->event->fire('auth.users.saving', compact('user'));
    }

    /**
     * Eloquent 'saved' event method.
     *
     * @param  \Arcanedev\LaravelAuth\Models\User|User  $user
     *
     * @return mixed
     */
    public function saved(User $user)
    {
        $this->event->fire('auth.users.saved', compact('user'));
    }

    /**
     * Eloquent 'deleting' event method.
     *
     * @param  \Arcanedev\LaravelAuth\Models\User|User  $user
     *
     * @return mixed
     */
    public function deleting(User $user)
    {
        if ($user->isAdmin()) return false;

        if ($user->forceDeleting) {

Accessing forceDeleting on the interface Arcanesoft\Contracts\Auth\Models\User suggest that you code against a concrete implementation. How about adding an instanceof check?

            $user->roles()->detach();
        }

        $this->event->fire('auth.users.deleting', compact('user'));

        return true;
    }

    /**
     * Eloquent 'deleted' event method.
     *
     * @param  \Arcanedev\LaravelAuth\Models\User|User  $user
     *
     * @return mixed
     */
    public function deleted(User $user)
    {
        $this->event->fire('auth.users.deleted', compact('user'));
    }

    /**
     * Eloquent 'restoring' event method.
     *
     * @param  \Arcanedev\LaravelAuth\Models\User|User  $user
     *
     * @return mixed
     */
    public function restoring(User $user)
    {
        $this->event->fire('auth.users.restoring', compact('user'));
    }

    /**
     * Eloquent 'restored' event method.
     *
     * @param  \Arcanedev\LaravelAuth\Models\User|User  $user
     *
     * @return mixed
     */
    public function restored(User $user)
    {
        $this->event->fire('auth.users.restored', compact('user'));
    }
}