enigmatix\confirmation\ConfirmationRequestsController

Complexity

Total Complexity

8

Size/Duplication

Total Lines	72
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	5

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	behaviors()	0	11	1
B	actionRelease()	0	25	4
A	actionExpired()	0	4	1
A	findModel()	0	8	2

<?php

namespace enigmatix\confirmation;

use yii\web\Controller;
use Yii;
use enigmatix\confirmation\ConfirmationRequest;
use yii\base\ErrorException;
use yii\web\NotFoundHttpException;
use yii\filters\VerbFilter;

/**
 * ConfirmationRequestsController implements the CRUD actions for ConfirmationRequest model.
 */
class ConfirmationRequestsController extends Controller
{

    var $defaultAction = 'release';

The visibility should be declared for property $defaultAction.

    /**
     * @inheritdoc
     */
    public function behaviors()
    {
        return [
            'verbs' => [
                'class' => VerbFilter::className(),
                'actions' => [
                    'delete' => ['POST'],
                ],
            ],
        ];
    }


    /**
     * Releases a single change as stored in the  ConfirmationRequest model.
     * @param integer $id

There is no parameter named $id. Was it maybe removed?

     * @return mixed
     */
    public function actionRelease($release_token)

The parameter $release_token is not named in camelCase.

    {

        try{
            $model =  $this->findModel($release_token);
        } catch (NotFoundHttpException $e){
            Yii::$app->session->setFlash('danger' , "We were not able to find your change.  Perhaps it has already been processed?");

The string literal We were not able to find your change. Perhaps it has already been processed? does not require double quotes, as per coding-style, please use single quotes.

            return $this->goHome();
        }

        try{
            $model->release();
        } catch (ErrorException $e){
            return $this->redirect(['confirmation-requests/expired']);
        }

        $viewLink = $model->constructObject()->getViewLink();
        $model->delete();

        if(!Yii::$app->user->isGuest){
            return $this->redirect($viewLink);
        }

        return $this->render('@vendor/enigmatix/yii2-confirmation/views/confirm', []);
    }

    public function actionExpired(){
        return $this->render('@vendor/enigmatix/yii2-confirmation/views/expired', []);

    }

    /**
     * Finds the ConfirmationRequest model based on its primary key value.
     * If the model is not found, a 404 HTTP exception will be thrown.
     * @param integer $id

There is no parameter named $id. Was it maybe removed?

     * @return ConfirmationRequest the loaded model
     * @throws NotFoundHttpException if the model cannot be found
     */
    protected function findModel($release_token)

The parameter $release_token is not named in camelCase.

    {
        if (($model = ConfirmationRequest::findOne(['release_token' => $release_token])) !== null) {
            return $model;
        } else {
            throw new NotFoundHttpException('The requested page does not exist.');
        }
    }
}