hipanel\components\User

Complexity

Total Complexity

15

Size/Duplication

Total Lines	95
Duplicated Lines	0 %

Coupling/Cohesion

Components	3
Dependencies	4

Test Coverage

Coverage

0%

Importance

Changes

0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	init()	0	7	2
A	not()	0	9	2
A	is()	0	9	2
A	getIsGuest()	0	4	1
A	getAuthData()	0	4	2
A	getGuestAuthData()	0	11	2
A	getAuthorizedAuthData()	0	19	3
A	redirectLogin()	0	5	1

<?php
/**
 * HiPanel core package.
 *
 * @link      https://hipanel.com/
 * @package   hipanel-core
 * @license   BSD-3-Clause
 * @copyright Copyright (c) 2014-2017, HiQDev (http://hiqdev.com/)
 */

namespace hipanel\components;

use Yii;
use yii\base\InvalidCallException;
use yii\base\InvalidConfigException;

class User extends \yii\web\User
{
    public $isGuestAllowed = false;

    /**
     * @var string the seller login
     */
    public $seller;

    public function init()
    {
        parent::init();
        if (empty($this->seller)) {
            throw new InvalidConfigException('User "seller" must be set');
        }
    }

    public function not($key)
    {
        $identity = $this->getIdentity();
        if (!$identity) {

The expression $identity of type null|boolean is loosely compared to false; this is ambiguous if the boolean can be false. You might want to explicitly use !== null instead.

            throw new InvalidCallException();
        }

        return $identity->not($key);

The method not cannot be called on $identity (of type boolean).

    }

    public function is($key)
    {
        $identity = $this->getIdentity();
        if (!$identity) {

The expression $identity of type null|boolean is loosely compared to false; this is ambiguous if the boolean can be false. You might want to explicitly use !== null instead.

            throw new InvalidCallException();
        }

        return $identity->is($key);

The method is cannot be called on $identity (of type boolean).

    }

    /**
     * {@inheritdoc}
     * XXX fixes redirect loop when identity is set but the object is empty.
     * @return bool
     */
    public function getIsGuest()
    {
        return empty($this->getIdentity()->id);
    }

    /**
     * Prepares authorization data.
     * Redirects to authorization if necessary.
     * @return array
     */
    public function getAuthData()
    {
        return $this->isGuest ? $this->getGuestAuthData() : $this->getAuthorizedAuthData();
    }

    protected function getGuestAuthData()
    {
        if (!$this->isGuestAllowed) {
            $this->redirectLogin();
        }

        return [
            'auth_ip' => Yii::$app->request->getUserIP(),
            'seller' => Yii::$app->params['user.seller'],
        ];
    }

    protected function getAuthorizedAuthData()
    {
        try {
            $token = $this->identity->getAccessToken();

It seems like you code against a concrete implementation and not the interface yii\web\IdentityInterface as the method getAccessToken() does only exist in the following implementations of said interface: hipanel\models\User.

        } catch (\Exception $e) {
            $token = null;
        }

        if (empty($token)) {
            /// logout() is very important here, else - redirect loop
            $this->logout();
            $this->redirectLogin();
        }

        return [
            'access_token' => $token,
            'auth_ip' => Yii::$app->request->getUserIP(),
        ];
    }

    public function redirectLogin()
    {
        $this->loginRequired();
        Yii::$app->end();
    }
}