hipanel\controllers\ReminderController

Complexity

Total Complexity

10

Size/Duplication

Total Lines	125
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	8

Test Coverage

Coverage

0%

Importance

Changes	10
Bugs	0	Features	5

6 Methods

Rating	Name	Duplication	Size	Complexity
A	init()	0	4	1
A	behaviors()	0	9	1
B	actions()	0	87	4
A	getPeriodicityOptions()	0	4	1
A	getTypeReminder()	0	4	1
A	actionGetCount()	0	9	2

<?php

namespace hipanel\controllers;

use DateTime;
use hipanel\actions\IndexAction;
use hipanel\actions\OrientationAction;
use hipanel\actions\RedirectAction;
use hipanel\actions\RenderAction;
use hipanel\actions\RenderAjaxAction;
use hipanel\actions\RenderJsonAction;
use hipanel\actions\SmartCreateAction;
use hipanel\actions\SmartDeleteAction;
use hipanel\actions\SmartUpdateAction;
use hipanel\actions\ValidateFormAction;
use hipanel\actions\ViewAction;
use hipanel\models\Reminder;
use hipanel\widgets\ReminderTop;
use Symfony\Component\EventDispatcher\Event;
use Yii;
use yii\base\Widget;
use yii\helpers\ArrayHelper;
use yii\web\NotFoundHttpException;
use yii\web\Response;

class ReminderController extends \hipanel\base\CrudController
{
    public function init()
    {
        $this->viewPath = '@hipanel/views/reminder';
    }

    public function behaviors()
    {
        return [

The return type of return array(array('class' => 'yii\\filters\\HttpCache', 'only' => array('count'))); (array<string,string|string[]>[]) is incompatible with the return type of the parent method hipanel\base\Controller::behaviors of type array<string,array<string,array<string|array<string,string[]|boolean>>>>.

            [
                'class' => 'yii\filters\HttpCache',
                'only' => ['count'],
            ],
        ];
    }

    public function actions()
    {
        return [
            'set-orientation' => [
                'class' => OrientationAction::class,
                'allowedRoutes' => [
                    'reminder/index'
                ]
            ],
            'validate-form' => [
                'class' => ValidateFormAction::class,
            ],
            'view' => [
                'class' => ViewAction::class,
            ],
            'index' => [
                'class' => IndexAction::class,
                'data' => function ($action, $data) {

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

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

                    return [

                    ];
                },
            ],
            'create-modal' => [
                'class' => SmartCreateAction::class,
                'scenario' => 'create',
                'view' => 'create-modal',
                'data' => function ($action, $data) {
                    $object_id = \Yii::$app->request->get('object_id');
                    if (empty($object_id)) {
                        throw new NotFoundHttpException('Object ID is missing');
                    }
                    $data['model']->object_id = $object_id;

                    return $data;
                },
            ],
            'create' => [
                'class' => SmartCreateAction::class,
                'view' => 'create-modal',
            ],
            'update' => [
                'class' => SmartUpdateAction::class,
                'on beforeSave' => function ($event) {
                    /** @var \hipanel\actions\Action $action */
                    $action = $event->sender;
                    if (Yii::$app->request->isAjax) {
                        $reminder = Yii::$app->request->post('Reminder');
                        $action->collection->set(Reminder::find()->where(['id' => $reminder['id']])->one());

It seems like \hipanel\models\Reminder::find()->where(array('id' => $reminder['id']))->one() targeting hiqdev\hiart\ActiveQuery::one() can also be of type null; however, hiqdev\hiart\Collection::set() does only seem to accept array|object<hiqdev\hiart\ActiveRecord>, maybe add an additional type check?

                        foreach ($action->collection->models as $model) {
                            $model->next_time = (new DateTime($model->next_time))->modify($reminder['next_time'])->format('Y-m-d H:i:s');
                        }
                    }
                },
                'POST ajax' => [ 'save' => true,
                    'success' => [
                        'class' => RenderJsonAction::class,
                        'return' => function ($action) {

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

                            return [
                                'success' => true,
                                'widget' => ReminderTop::widget()
                            ];
                        },
                    ],
                ],
            ],
            'delete' => [
                'class' => SmartDeleteAction::class,
                'POST html | POST pjax' => [
                    'save' => true,
                    'success' => [
                        'class' => RedirectAction::class,
                    ],
                ],
            ],
            'ajax-reminders-list' => [
                'class' => RenderAjaxAction::class,
                'view' => '_ajaxReminderList',
                'params' => function ($action) {
                    $reminders = Reminder::find()->where(['to_site' => true])->all();
                    $remindInOptions = Reminder::reminderNextTimeOptions();

                    return compact(['reminders', 'remindInOptions']);
                }
            ]
        ];
    }

    public function getPeriodicityOptions()
    {
        return $this->getRefs('type,periodicity', 'hipanel/reminder');
    }

    public function getTypeReminder()
    {
        return $this->getRefs('type,reminder', 'hipanel/reminder');
    }

    public function actionGetCount()
    {
        if (Yii::$app->request->isAjax) {
            Yii::$app->response->format = Response::FORMAT_JSON;
            $count = Reminder::find()->toSite()->count();

            return compact('count');
        }
    }
}