vistart\Models\traits\RegistrationTrait

Complexity

Total Complexity

36

Size/Duplication

Total Lines	220
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Test Coverage

Coverage

56.1%

Importance

Changes	13
Bugs	0	Features	0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	getAuthManager()	0	5	2
D	register()	0	45	18
B	deregister()	0	26	6
A	getSource()	0	5	2
A	setSource()	0	5	2
A	getSourceRules()	0	10	2
A	setSourceRules()	0	6	3
A	onInitSourceAttribute()	0	6	1

<?php

/**
 *  _   __ __ _____ _____ ___  ____  _____
 * | | / // // ___//_  _//   ||  __||_   _|
 * | |/ // /(__  )  / / / /| || |     | |
 * |___//_//____/  /_/ /_/ |_||_|     |_|
 * @link https://vistart.name/
 * @copyright Copyright (c) 2016 vistart
 * @license https://vistart.name/license/
 */

namespace vistart\Models\traits;

use Yii;
use yii\base\ModelEvent;
use yii\db\Exception;
use yii\db\IntegrityException;
use yii\rbac\BaseManager;
use yii\rbac\Role;

/**
 * User features concerning registration.
 *
 * @property array $sourceRules rules associated with source attribute.
 * @version 2.0
 * @author vistart <i@vistart.name>
 */
trait RegistrationTrait
{

    /**
     * @event Event an event that is triggered after user is registered successfully.
     */
    public static $eventAfterRegister = "afterRegister";

    /**
     * @event Event an event that is triggered before registration.
     */
    public static $eventBeforeRegister = "beforeRegister";

    /**
     * @event Event an event that is triggered when registration failed.
     */
    public static $eventRegisterFailed = "registerFailed";

    /**
     * @event Event an event that is triggered after user is deregistered successfully.
     */
    public static $eventAfterDeregister = "afterDeregister";

    /**
     * @event Event an event that is triggered before deregistration.
     */
    public static $eventBeforeDeregister = "beforeDeregister";

    /**
     * @event Event an event that is triggered when deregistration failed.
     */
    public static $eventDeregisterFailed = "deregisterFailed";

    /**
     * @var string name of attribute which store the source. if you don't want to
     * record source, please assign false.
     */
    public $sourceAttribute = 'source';
    private $_sourceRules = [];
    public static $sourceSelf = '0';

    /**
     * @var string auth manager component id.
     */
    public $authManagerId = 'authManager';

    /**
     * Get auth manager. If auth manager not configured, Yii::$app->authManager
     * will be given.
     * @return BaseManager
     */
    public function getAuthManager()
    {
        $authManagerId = $this->authManagerId;
        return empty($authManagerId) ? Yii::$app->authManager : Yii::$app->$authManagerId;
    }

    /**
     * Register new user.
     * It is equivalent to store the current user and its associated models into
     * database synchronously. The registration will be terminated immediately
     * if any errors occur in the process, and all the earlier steps succeeded
     * are rolled back.
     * If auth manager configured, and auth role(s) provided, it(they) will be
     * assigned to user after registration.
     * If current user is not a new one(isNewRecord = false), the registration
     * will be skipped and return false.
     * The $eventBeforeRegister will be triggered before registration starts.
     * If registration finished, the $eventAfterRegister will be triggered. or
     * $eventRegisterFailed will be triggered when any errors occured.
     * @param array $associatedModels The models associated with user to be stored synchronously.
     * @param string|array $authRoles auth name, auth instance, auth name array or auth instance array.
     * @return boolean Whether the registration succeeds or not.
     * @throws IntegrityException when inserting user and associated models failed.
     */
    public function register($associatedModels = [], $authRoles = [])
    {
        if (!$this->isNewRecord) {

The property isNewRecord does not exist. Did you maybe forget to declare it?

            return false;
        }
        $this->trigger(static::$eventBeforeRegister);

It seems like trigger() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        $transaction = $this->getDb()->beginTransaction();

It seems like getDb() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        try {
            if (!$this->save()) {

It seems like save() must be provided by classes using this trait. How about adding it as abstract method to this trait?

                throw new IntegrityException('Registration Error(s) Occured.', $this->errors);

The property errors does not exist. Did you maybe forget to declare it?

            }
            if ($authManager = $this->getAuthManager() && !empty($authRoles)) {
                if (is_string($authRoles) || $authRoles instanceof Role || !is_array($authRoles)) {
                    $authRoles = [$authRoles];
                }
                foreach ($authRoles as $role) {
                    if (is_string($role)) {
                        $role = $authManager->getRole($role);

The method getRole cannot be called on $authManager (of type boolean).

                    }
                    if ($role instanceof Role) {
                        $authManager->assign($role, $this->guid);

The property guid does not exist. Did you maybe forget to declare it?

The method assign cannot be called on $authManager (of type boolean).

                    }
                }
            }
            if (!empty($associatedModels) && is_array($associatedModels)) {
                foreach ($associatedModels as $model) {
                    if (!$model->save()) {
                        throw new IntegrityException('Registration Error(s) Occured.', $model->errors);
                    }
                }
            }
            $transaction->commit();
        } catch (Exception $ex) {
            $transaction->rollBack();
            $this->trigger(static::$eventRegisterFailed);

It seems like trigger() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            if (YII_DEBUG || YII_ENV !== YII_ENV_PROD) {
                Yii::error($ex->errorInfo, static::className() . '\register');

$ex->errorInfo is of type array, but the function expects a string.

                return $ex;
            }
            Yii::warning($ex->errorInfo, static::className() . '\register');

$ex->errorInfo is of type array, but the function expects a string.

            return false;
        }
        $this->trigger(static::$eventAfterRegister);

It seems like trigger() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        return true;
    }

    /**
     * Deregister current user itself.
     * It is equivalent to delete current user and its associated models. BUT it
     * deletes current user ONLY, the associated models will not be deleted
     * forwardly. So you should set the foreign key of associated models' table
     * referenced from primary key of user table, and their association mode is
     * 'on update cascade' and 'on delete cascade'.
     * the $eventBeforeDeregister will be triggered before deregistration starts.
     * if deregistration finished, the $eventAfterDeregister will be triggered. or
     * $eventDeregisterFailed will be triggered when any errors occured.
     * @return boolean Whether deregistration succeeds or not.
     * @throws IntegrityException when deleting user failed.
     */
    public function deregister()
    {
        if ($this->isNewRecord) {
            return false;
        }
        $this->trigger(static::$eventBeforeDeregister);

It seems like trigger() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        $transaction = $this->getDb()->beginTransaction();

It seems like getDb() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        try {
            $result = $this->delete();

It seems like delete() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            if ($result != 1) {
                throw new IntegrityException('Deregistration Error(s) Occured.', $this->errors);
            }
            $transaction->commit();
        } catch (Exception $ex) {
            $transaction->rollBack();
            $this->trigger(static::$eventDeregisterFailed);

It seems like trigger() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            if (YII_DEBUG || YII_ENV !== YII_ENV_PROD) {
                Yii::error($ex->errorInfo, static::className() . '\deregister');

$ex->errorInfo is of type array, but the function expects a string.

                return $ex;
            }
            Yii::warning($ex->errorInfo, static::className() . '\deregister');

$ex->errorInfo is of type array, but the function expects a string.

            return false;
        }
        $this->trigger(static::$eventAfterDeregister);

It seems like trigger() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        return $result == 1;
    }

    /**
     * Get source.
     * @return string
     */
    public function getSource()
    {
        $sourceAttribute = $this->sourceAttribute;
        return is_string($sourceAttribute) ? $this->$sourceAttribute : null;
    }

    /**
     * Set source.
     * @param string $source
     */
    public function setSource($source)
    {
        $sourceAttribute = $this->sourceAttribute;
        return is_string($sourceAttribute) ? $this->$sourceAttribute = $source : null;
    }

    /**
     * Get the rules associated with source attribute.
     * @return array rules.
     */
    public function getSourceRules()
    {
        if (empty($this->_sourceRules)) {
            $this->_sourceRules = [
                [[$this->sourceAttribute], 'required'],
                [[$this->sourceAttribute], 'string'],
            ];
        }
        return $this->_sourceRules;
    }

    /**
     * Set the rules associated with source attribute.
     * @param array $rules
     */
    public function setSourceRules($rules)
    {
        if (!empty($rules) && is_array($rules)) {
            $this->_sourceRules = $rules;
        }
    }

    /**
     * Initialize the source attribute with $sourceSelf.
     * This method is ONLY used for being triggered by event. DO NOT call,
     * override or modify it directly, unless you know the consequences.
     * @param ModelEvent $event
     */
    public function onInitSourceAttribute($event)
    {
        $sender = $event->sender;
        $sourceAttribute = $sender->sourceAttribute;
        $sender->$sourceAttribute = static::$sourceSelf;
    }
}