hipanel\components\User

Complexity

Total Complexity

5

Size/Duplication

Total Lines	36
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

0%

Importance

Changes	1
Bugs	1	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	not()	0	9	2
A	is()	0	9	2
A	getIsGuest()	0	4	1

<?php

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

namespace hipanel\components;

use yii\base\InvalidConfigException;

class User extends \yii\web\User
{
    /**
     * @var string the seller login
     */
    public $seller;

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

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

        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);
    }
}