app\controllers\ArticleController::actionView()

Complexity

Conditions	2
Paths	2

Size

Total Lines	19
Code Lines	12

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function actionView($alias)
    {
        $model = Article::find()->where([
            'alias' => $alias
        ])->andWhere([
            'active' => 1
        ])->one();

        if (null === $model) {
            throw new NotFoundHttpException('Article not fount with alias = '.$alias.'.');
        }

        $this->setMetaParams($model);

It seems like $model can also be of type array; however, parameter $model of app\controllers\BaseController::setMetaParams() does only seem to accept null|yii\db\ActiveRecord, maybe add an additional type check?

        $images = OwnerMediafile::getImageFiles(Article::tableName(), $model->id);

It seems like $model->id can also be of type null; however, parameter $ownerId of Itstructure\MFUploader\models\OwnerMediafile::getImageFiles() does only seem to accept integer, maybe add an additional type check?

        return $this->render('view', [
            'model' => $model,
            'images' => $images
        ]);