hipanel\modules\ticket\models\Answer

Complexity

Total Complexity

20

Size/Duplication

Total Lines	187
Duplicated Lines	5.88 %

Coupling/Cohesion

Components	2
Dependencies	4

Test Coverage

Coverage

0%

Importance

Changes

0

19 Methods

Rating	Name	Duplication	Size	Complexity
A	tableName()	0	4	1
A	init()	0	5	1
A	load()	0	7	1
A	beforeChange()	0	6	1
A	switchId()	0	6	2
A	behaviors()	11	11	1
A	attributes()	0	24	1
A	rules()	0	28	1
A	attributeLabels()	0	17	1
A	getClient()	0	4	1
A	getClient_id()	0	4	1
A	getSeller()	0	4	1
A	getSeller_id()	0	4	1
A	prepareSpentTime()	0	4	1
A	getThread()	0	4	1
A	getFiles()	0	4	1
A	scenarioActions()	0	6	1
A	canSetSpent()	0	4	1
A	isOpen()	0	4	1

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

namespace hipanel\modules\ticket\models;

use hipanel\models\File;
use Yii;

/**
 * Class Answer.
 *
 * @property Thread $thread
 */
class Answer extends \hipanel\base\Model
{
    use \hipanel\base\ModelTrait;

    public static $i18nDictionary = 'hipanel:ticket';

    public static function tableName()
    {
        return 'thread';
    }

    public function init()
    {
        $this->on(static::EVENT_BEFORE_INSERT, [$this, 'beforeChange']);
        $this->on(static::EVENT_BEFORE_UPDATE, [$this, 'beforeChange']);
    }

    public function load($data, $formName = null)
    {
        $result = parent::load($data, $formName);
        $this->prepareSpentTime();

        return $result;
    }

    public function beforeChange($event)

The parameter $event is not used and could be removed.

    {
        $this->switchId();

        return true;
    }

    private function switchId()
    {
        if ($this->scenario === 'update') {
            $this->id = $this->answer_id;

The property id does not exist on object<hipanel\modules\ticket\models\Answer>. Since you implemented __set, maybe consider adding a @property annotation.

The property answer_id does not exist on object<hipanel\modules\ticket\models\Answer>. Since you implemented __set, maybe consider adding a @property annotation.

        }
    }

    /**
     * @return array
     */
    public function behaviors()

This method seems to be duplicated in your project.

    {
        return [
            [
                'class' => 'hipanel\behaviors\File',
                'attribute' => 'file',
                'targetAttribute' => 'file_ids',
                'scenarios' => ['create', 'answer'],
            ],
        ];
    }

    /**
     * @return array
     */
    public function attributes()
    {
        return [
            'id',
            'answer_id',
            'message',
            'private',
            'is_private',
            'is_answer',
            'changed_num',
            'author',
            'author_id',
            'author_seller',
            'seller_id',
            'account',
            'email_hash',
            'create_time',
            'spent',
            'spent_hours',
            'is_moved',
            'ip',
            'file',
        ];
    }

    /**
     * {@inheritdoc}
     */
    public function rules()
    {
        return [
            [
                ['id', 'answer_id', 'spent', 'spent_hours'],
                'integer',
                'enableClientValidation' => false,
            ],
            [
                ['message', 'id', 'answer_id'],
                'required',
                'on' => 'update',
            ],
            [
                ['spent', '!spent_hours'],
                'safe',
                'on' => 'update',
            ],
            [
                ['is_private'],
                'boolean',
                'on' => 'update',
                'when' => function () {
                    return Yii::$app->user->can('support');
                },
            ],
        ];
    }

    /**
     * {@inheritdoc}
     */
    public function attributeLabels()
    {
        return $this->mergeAttributeLabels([
            'author' => Yii::t('hipanel:ticket', 'Author'),
            'author_id' => Yii::t('hipanel:ticket', 'Author'),
            'recipient' => Yii::t('hipanel:ticket', 'Recipient'),
            'is_private' => Yii::t('hipanel:ticket', 'Make private'),
            'responsible' => Yii::t('hipanel:ticket', 'Assignee'),
            'responsible_id' => Yii::t('hipanel:ticket', 'Assignee'),
            'spent' => Yii::t('hipanel:ticket', 'Spent time'),
            'create_time' => Yii::t('hipanel:ticket', 'Created'),
            'a_reply_time' => Yii::t('hipanel:ticket', 'a_reply_time'),
            'file' => Yii::t('hipanel:ticket', 'Files'),
            'lastanswer' => Yii::t('hipanel:ticket', 'Last answer'),
            'author_seller' => Yii::t('hipanel:ticket', 'Seller'),
        ]);
    }

    public function getClient()
    {
        return $this->author;

The property author does not exist on object<hipanel\modules\ticket\models\Answer>. Since you implemented __get, maybe consider adding a @property annotation.

    }

    public function getClient_id()
    {
        return $this->author_id;

The property author_id does not exist on object<hipanel\modules\ticket\models\Answer>. Since you implemented __get, maybe consider adding a @property annotation.

    }

    public function getSeller()
    {
        return $this->author_seller;

The property author_seller does not exist on object<hipanel\modules\ticket\models\Answer>. Since you implemented __get, maybe consider adding a @property annotation.

    }

    public function getSeller_id()
    {
        return $this->author_seller_id;

The property author_seller_id does not exist on object<hipanel\modules\ticket\models\Answer>. Since you implemented __get, maybe consider adding a @property annotation.

    }

    public function prepareSpentTime()
    {
        list($this->spent_hours, $this->spent) = explode(':', $this->spent, 2);

The property spent_hours does not exist on object<hipanel\modules\ticket\models\Answer>. Since you implemented __get, maybe consider adding a @property annotation.

The property spent does not exist on object<hipanel\modules\ticket\models\Answer>. Since you implemented __get, maybe consider adding a @property annotation.

    }

    public function getThread()
    {
        return $this->hasOne(Thread::class, ['id' => 'id']);
    }

    public function getFiles()
    {
        return $this->hasMany(File::class, ['object_id' => 'id']);
    }

    public function scenarioActions()
    {
        return [
            'update' => 'update-answer',
        ];
    }

    public function canSetSpent()
    {
        return $this->thread->canSetSpent();
    }

    public function isOpen()
    {
        return $this->thread->isOpen();
    }
}