App\Http\Controller\ArticleController

Complexity

Total Complexity

6

Size/Duplication

Total Lines	61
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	afterAction()	0	9	2
A	actionView()	0	6	1
A	actionIndex()	0	22	1
A	findModel()	0	9	2

<?php
namespace App\Http\Controller;

use App\Model\Article;
use App\Model\StatusInterface;
use yii\data\Pagination;
use yii\web\NotFoundHttpException;

class ArticleController extends Controller
{
    public function afterAction($action, $result)
    {
        if ($action->id === 'view') {
            $this->model->updateCounters([
                'views' => 1
            ]);
        }

        return parent::afterAction($action, $result);
    }

    public function actionIndex()
    {
        $query = Article::find()
            ->alias('a')
            ->joinWith('category')
            ->where([
                'a.status' => StatusInterface::STATUS_ACTIVE,
                'a.is_deleted' => 0,
            ])
            ->andWhere(['<=', 'a.release_time', time()])
            ->orderBy(['a.release_time' => SORT_DESC]);

        $count = $query->count();
        $pagination = new Pagination(['totalCount' => $count]);

        $articles = $query->offset($pagination->offset)
            ->limit($pagination->limit)
            ->all();
        
        return $this->render('index', [
            'articles' => $articles,
            'pagination' => $pagination,
        ]);
    }

    public function actionView($id)
    {
        $model = $this->findModel($id);
        
        return $this->render('view', [
            'model' => $model,
        ]);
    }

    /**
     * @var Article $model
     */
    private $model;

    private function findModel($id)
    {
        $model = Article::findOne($id);
        if (!$model) {

$model is of type yii\db\ActiveRecord, thus it always evaluated to true.

            throw new NotFoundHttpException('Article does not exists');
        }

        $this->model = $model;
        return $this->model;
    }
}