hipanel\modules\server\controllers\PreOrderController::actions()

Complexity

Conditions	3
Paths	1

Size

Total Lines	42
Code Lines	27

Duplication

Lines	11
Ratio	26.19 %

Importance

Changes	2
Bugs	0	Features	2

<?php
    public function actions()
    {
        return [
            'index' => [
                'class' => IndexAction::class,
                'findOptions' => ['state' => 'new', 'class' => 'serverBuy'],
                'data' => function ($action) {
                    return [
                        'states' => $action->controller->getStates(),
                    ];
                },
            ],
            'bulk-approve' => [
                'class' => SmartUpdateAction::class,
                'scenario' => 'approve',
                'success' => Yii::t('hipanel/server', 'Hosting accounts were blocked successfully'),
                'error' => Yii::t('hipanel/server', 'Error during the hosting accounts blocking'),
                'POST html' => [
                    'save'    => true,
                    'success' => [
                        'class' => RedirectAction::class,
                    ],
                ],
                'on beforeSave' => function (Event $event) {

This code seems to be duplicated across your project.

                    /** @var \hipanel\actions\Action $action */
                    $action = $event->sender;
                    $comment = Yii::$app->request->post('comment');
                    if (!empty($type)) {

The variable $type seems to never exist, and therefore empty should always return true. Did you maybe rename this variable?

                        foreach ($action->collection->models as $model) {
                            $model->setAttribute('comment', $comment);

                        }
                    }
                },
            ],
            'bulk-approve-modal' => [
                'class' => PrepareBulkAction::class,
                'scenario' => 'approve',
                'view' => '_bulkApprove',
            ],
        ];
    }