ymaker\newsletter\frontend\actions\SubscribeAction

Complexity

Total Complexity

4

Size/Duplication

Total Lines	24
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	run()	0	16	4

<?php
/**
 * @link https://github.com/yiimaker/yii2-newsletter
 * @copyright Copyright (c) 2017 Yii Maker
 * @license BSD 3-Clause License
 */

namespace ymaker\newsletter\frontend\actions;

use Yii;
use yii\base\Action;
use yii\web\NotFoundHttpException;
use yii\web\Response;
use ymaker\newsletter\common\services\ServiceInterface;

/**
 * Action for subscribe on newsletter.
 *
 * @author Vladimir Kuprienko <vldmr.kuprienko@gmail.com>
 * @since 1.0
 */
class SubscribeAction extends Action
{
    /**
     * Running action.
     *
     * @return \yii\web\Response|array
     * @throws NotFoundHttpException
     */
    public function run()
    {
        $request = Yii::$app->getRequest();
        if ($request->getIsPost()) {
            /* @var ServiceInterface $service */
            $service = Yii::createObject(ServiceInterface::class);
            $response = Yii::$app->getResponse();

            if (!$service->create($request->post()) && $request->getIsAjax()) {
                $response->format = Response::FORMAT_JSON;
                return $service->getErrors();
            }

            return $response->redirect($request->getReferrer());

The expression return $response->redirect($request->getReferrer()) also could return the type yii\console\Response which is incompatible with the documented return type yii\web\Response|array.

        }
        throw new NotFoundHttpException();
    }
}