app\models\Credit

Complexity

Total Complexity

6

Size/Duplication

Total Lines	50
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	tableName()	0	3	1
A	attributeLabels()	0	5	1
A	rules()	0	10	1
A	assignmentClass()	0	3	1
A	workLogClass()	0	3	1
A	getWorkflowId()	0	3	1

<?php

namespace app\models;

use Yii;
use roaresearch\yii2\workflow\models\Workflow;

/**
 * This is the model class for table `{{%credit}}`.
 *
 * @property integer $id
 * @property string $credit
 */
class Credit extends \roaresearch\yii2\workflow\models\Process
{
    /**
     * @inheritdoc
     */
    public static function tableName()
    {
        return '{{%credit}}';
    }

    protected function assignmentClass(): string
    {
        return CreditAssignment::class;
    }

    protected function workLogClass(): string
    {
        return CreditWorkLog::class;
    }

    public function getWorkflowId(): int
    {
        return $this->workflow_id;

The expression return $this->workflow_id could return the type null which is incompatible with the type-hinted return integer. Consider adding an additional type-check to rule them out.

The property workflow_id does not exist on app\models\Credit. Since you implemented __get, consider adding a @property annotation.

    }

    /**
     * @inheritdoc
     */
    public function rules()
    {
        return array_merge(parent::rules(), [
            [['workflow_id'], 'required'],
            [
                ['workflow_id'],
                'exist',
                'skipOnError' => true,
                'targetClass' => Workflow::class,
                'targetAttribute' => ['workflow_id' => 'id'],
            ],
        ]);
    }

    /**
     * @inheritdoc
     */
    public function attributeLabels()
    {
        return [
            'id' => Yii::t('app', 'ID'),
            'workflow_id' => Yii::t('app', 'Workflow ID'),
        ];
    }
}