dominus77\maintenance\actions\frontend\SubscribeAction

Complexity

Total Complexity

5

Size/Duplication

Total Lines	20
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	run()	0	15	5

<?php

namespace dominus77\maintenance\actions\frontend;

use Yii;
use yii\base\Action;
use yii\web\Response;
use yii\web\Session;
use dominus77\maintenance\models\SubscribeForm;
use dominus77\maintenance\BackendMaintenance;

/**
 * Class SubscribeAction
 * @package dominus77\maintenance\actions\frontend
 */
class SubscribeAction extends Action
{
    /**
     * @return Response
     */
    public function run()
    {
        $model = new SubscribeForm();
        $msgSuccess = BackendMaintenance::t('app', 'We will inform you when everything is ready!');
        $msgInfo = BackendMaintenance::t('app', 'You have already subscribed to the alert!');
        if (($post = Yii::$app->request->post()) && $model->load($post) && $model->validate()) {

It seems like $post can also be of type object; however, parameter $data of yii\base\Model::load() does only seem to accept array, maybe add an additional type check?

            /** @var Session $session */
            $session = Yii::$app->session;
            if ($model->subscribe()) {
                $session->setFlash($model::SUBSCRIBE_SUCCESS, $msgSuccess);
            } else {
                $session->setFlash($model::SUBSCRIBE_INFO, $msgInfo);
            }
        }
        return $this->controller->redirect(Yii::$app->request->referrer);
    }
}