app\modules\admin\controllers\UsersController

Complexity

Total Complexity

14

Size/Duplication

Total Lines	125
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	9

Test Coverage

Coverage

64.71%

Importance

Changes	1
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	actionIndex()	0	13	1
A	behaviors()	0	15	1
B	actions()	0	34	1
B	actionEdit()	0	27	6
B	actionProfile()	0	27	5

<?php

namespace app\modules\admin\controllers;

use Yii;
use yii\filters\VerbFilter;
use yii\helpers\Url;
use app\traits\ModelTrait;
use app\models\User;
use app\models\UserProfile;
use app\modules\admin\models\forms\UserForm;
use app\modules\admin\models\forms\UserProfileForm;
use app\modules\admin\models\search\UserSearch;

class UsersController extends \yii\web\Controller
{
    use ModelTrait;

    public function behaviors()
    {
        return [
            'verbs' => [
                'class' => VerbFilter::class,
                'actions' => [
                    'set-active' => ['post'],
                    'set-block' => ['post'],
                    'delete' => ['post'],
                    'operations' => ['post'],
                    'photo-upload' => ['post'],
                ],
            ],
        ];
    }

    public function actions()
    {
        return [
            'operations' => [
                'class' => 'app\modules\admin\controllers\common\OperationsAction',
                'modelClass' => User::class,
                'operations' => [
                    'delete' => [],
                    'set-active' => ['status' => User::STATUS_ACTIVE],
                    'set-block' => ['status' => User::STATUS_BLOCKED]
                ]
            ],
            'set-active' => [
                'class' => 'app\modules\admin\controllers\common\UpdateAttributesAction',
                'modelClass' => User::class,
                'attributes' => ['status' => User::STATUS_ACTIVE],
            ],
            'set-block' => [
                'class' => 'app\modules\admin\controllers\common\UpdateAttributesAction',
                'modelClass' => User::class,
                'attributes' => ['status' => User::STATUS_BLOCKED],
            ],
            'delete' => [
                'class' => 'app\modules\admin\controllers\common\DeleteAction',
                'modelClass' => User::class,
            ],
            'photo-upload' => [
                'class'     => 'rkit\filemanager\actions\UploadAction',
                'modelClass' => UserProfile::class,
                'attribute' => 'photo',
                'inputName' => 'file',
            ],
        ];
    }

    public function actionIndex()
    {
        $userSearch = new UserSearch();
        $dataProvider = $userSearch->search(Yii::$app->request->get());
        $statuses = User::getStatuses();

        return $this->render('index', [
            'userSearch' => $userSearch,
            'dataProvider' => $dataProvider,
            'statuses' => $statuses,
            'roles' => Yii::$app->authManager->getRoles()
        ]);
    }

    public function actionEdit($id = null)
    {
        $model = new UserForm();

        if ($id) {
            $model->setModel($this->findModel(new User, $id));

$this->findModel(new \app\models\User(), $id) of type object<yii\db\ActiveRecord> is not a sub-type of object<app\models\User>. It seems like you assume a child class of the class yii\db\ActiveRecord to be always present.

        }

        if (Yii::$app->request->isPost) {
            if ($model->load(Yii::$app->request->post()) && $model->validate()) {
                try {
                    $model->save();

                    Yii::$app->session->setFlash('success', Yii::t('app.msg', 'Saved successfully'));
                    return $this->asJson(['redirect' => Url::toRoute(['edit', 'id' => $model->id])]);
                } catch (\Exception $e) {
                    Yii::error($e);
                    return $this->asJson(false);
                }
            }
            return $this->asJson($this->collectErrors($model));
        }

        return $this->render('edit', [
            'model' => $model,
        ]);
    }

    public function actionProfile($id)
    {
        $model = new UserProfileForm();
        $model->setModel($this->findModel(new UserProfile, $id));

$this->findModel(new \app\models\UserProfile(), $id) of type object<yii\db\ActiveRecord> is not a sub-type of object<app\models\UserProfile>. It seems like you assume a child class of the class yii\db\ActiveRecord to be always present.

        if (Yii::$app->request->isPost) {
            $data = Yii::$app->request->post();
            $data['UserProfileForm'] = $data['UserProfileForm'] + $data['UserProfile'];

            if ($model->load($data) && $model->validate()) {
                try {
                    $model->save();

                    Yii::$app->session->setFlash('success', Yii::t('app.msg', 'Saved successfully'));
                    return $this->asJson(['redirect' => Url::toRoute(['profile', 'id' => $model->user_id])]);
                } catch (\Exception $e) {
                    Yii::error($e);
                    return $this->asJson(false);
                }
            }
            return $this->asJson($this->collectErrors($model));
        }

        return $this->render('profile', [
            'model' => $model
        ]);
    }
}