carono\yii2crud\actions\DeleteAction

Complexity

Total Complexity

8

Size/Duplication

Total Lines	34
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	run()	0	16	6
A	hasSoftDeleteError()	0	6	2

<?php


namespace carono\yii2crud\actions;

use carono\yii2crud\CrudController;
use Yii;
use yii\db\ActiveRecord;

/**
 * Class DeleteAction
 *
 * @package carono\yii2crud\actions
 * @property CrudController $controller
 * @method getMessageOnDelete()
 */
class DeleteAction extends Action
{
    public $softDeleteAttribute = 'deleted_at';
    public $preventAjaxRedirect = false;
    public $messageOnDelete = 'Model deleted';

    public function run($id)
    {
        $model = $this->controller->findModel($id);
        $model->delete();
        if ($model->hasErrors() || $this->hasSoftDeleteError($model)) {
            $msg = current($model->getFirstErrors());
            Yii::$app->session->setFlash('error', $msg ?: Yii::t('errors', 'Fail Deleting Model'));

The method setFlash() does not exist on null.

        } else {
            Yii::$app->session->setFlash('success', $this->getMessageOnDelete($model));

The call to carono\yii2crud\actions\DeleteAction::getMessageOnDelete() has too many arguments starting with $model.

        }

        if (Yii::$app->request->isAjax && !$this->preventAjaxRedirect) {
            return Yii::$app->response->redirect($this->controller->deleteRedirect($model), 302, false);
        }

        return $this->controller->redirect($this->controller->deleteRedirect($model));
    }

    /**
     * @param ActiveRecord $model
     * @return bool
     */
    public function hasSoftDeleteError($model): bool
    {
        if ($model->hasAttribute($this->softDeleteAttribute)) {
            return empty($model->{$this->softDeleteAttribute});
        }
        return false;
    }
}