roboapp\crud\Create

Complexity

Total Complexity

13

Size/Duplication

Total Lines	77
Duplicated Lines	27.27 %

Coupling/Cohesion

Components	1
Dependencies	5

Importance

Changes	4
Bugs	0	Features	1

2 Methods

Rating	Name	Duplication	Size	Complexity
C	newModel()	0	22	9
A	run()	21	21	4

<?php
/**
 * @link https://github.com/roboapp
 * @copyright Copyright (c) 2016 Roboapp
 * @license [MIT License](https://opensource.org/licenses/MIT)
 */

namespace roboapp\crud;

use yii\base\InvalidConfigException;
use yii\base\Model;
use yii\db\ActiveRecord;
use yii\web\Response;
use yii\widgets\ActiveForm;

/**
 * Action supports creation of the new model using web form.
 *
 * @author iRipVanWinkle <iripvanwinkle@gmail.com>
 * @since 1.0
 */
class Create extends BaseAction
{
    /**
     * @var string the scenario to be assigned to the new model
     */
    public $scenario = Model::SCENARIO_DEFAULT;

    /**
     * @var string name of the view
     */
    public $view = 'create';

    /**
     * @var string|array|callable a PHP callable that will be called to create the new model or class name
     * The callable should return the new model instance.
     */
    public $newModel;

    /**
     * @var string the name variable model in the view template
     */
    public $nameVariableModel = 'model';

    /**
     * Creates new model instance.
     * @return ActiveRecord new model instance.
     * @throws InvalidConfigException on invalid configuration.
     */
    public function newModel()
    {
        $newModel = $this->newModel;

        if (!is_string($newModel) && !is_array($newModel) && !is_callable($newModel)) {
            throw new InvalidConfigException('"' . get_class($this) .
                '::newModel" must be set callable or the name of existing class.');
        }

        if (is_string($newModel) && !class_exists($newModel)) {
            throw new InvalidConfigException('"' . get_class($this) .
                '::newModel" must be set the class name of existing class.');
        }

        if (is_array($newModel) && isset($newModel['class']) && !class_exists($newModel['class'])) {
            throw new InvalidConfigException('"' . get_class($this) .
                '::newModel" must be an array containing a "class" element and ' .
                'set the class name of existing class.');
        }

        return \Yii::createObject($newModel);
    }

    /**
     * Creates new record.
     * @return mixed response
     */
    public function run()

This method seems to be duplicated in your project.

    {
        $model = $this->newModel();
        $model->setScenario($this->scenario);
        if ($model->load(\Yii::$app->getRequest()->post())) {
            if (\Yii::$app->getRequest()->isAjax) {
                \Yii::$app->getResponse()->format = Response::FORMAT_JSON;
                return ActiveForm::validate($model);
            }
            if ($model->save()) {
                $this->addSuccessFlash();
                return $this->redirect($model);
            } else {
                $this->addErrorFlash();
            }
        }

        return $this->controller->render($this->view, [
            $this->nameVariableModel => $model,
        ]);
    }
}