hipanel\modules\server\models\query\ConfigQuery

Complexity

Total Complexity

4

Size/Duplication

Total Lines	26
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	2
Bugs	0	Features	1

3 Methods

Rating	Name	Duplication	Size	Complexity
A	getAvailable()	0	5	1
A	withPrices()	0	6	1
A	withSellerOptions()	0	9	2

<?php
/**
 * Server module for HiPanel
 *
 * @link      https://github.com/hiqdev/hipanel-module-server
 * @package   hipanel-module-server
 * @license   BSD-3-Clause
 * @copyright Copyright (c) 2015-2019, HiQDev (http://hiqdev.com/)
 */

namespace hipanel\modules\server\models\query;

use hiqdev\hiart\ActiveQuery;
use Yii;

class ConfigQuery extends ActiveQuery
{
    public function withPrices(): self
    {
        $this->joinWith('prices');
        $this->andWhere(['with_prices' => true]);

        return $this;
    }

    public function getAvailable(): self
    {
        $this->addAction('get-available');

        return $this;
    }

    public function withSellerOptions(): self
    {
        $options = Yii::$app->user->isGuest ? ['seller' => Yii::$app->params['user.seller']] : [
            'seller' => Yii::$app->user->identity->seller,

Accessing seller on the interface yii\web\IdentityInterface suggest that you code against a concrete implementation. How about adding an instanceof check?

            'seller_id' => Yii::$app->user->identity->seller_id,

Accessing seller_id on the interface yii\web\IdentityInterface suggest that you code against a concrete implementation. How about adding an instanceof check?

        ];
        $this->andWhere($options);

        return $this;
    }
}