Migrator - Code Metrics - Inspection of "refactor migration of taxonomy due to conflict wit..." - nkostadinov/yii2-taxonomy - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 414d92...35a2e4 )

by Nikola

created 2017-05-23 05:55 UTC

Migrator A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	27
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

Metric	Value
wmc	5
lcom	1
cbo	1
dl	0
loc	27
rs	10
c	0
b	0
f	0

3 Methods

Rating	Name	Size	Complexity
A	__construct()	4	1
A	runTaxonomy()	9	2
A	removeTaxonomy()	9	2

<?php


namespace nkostadinov\taxonomy\controllers;

use nkostadinov\taxonomy\models\TaxonomyTerms;
use Yii;
use nkostadinov\taxonomy\models\TaxonomyDef;
use nkostadinov\taxonomy\models\TaxonomyDefSearch;
use yii\base\InvalidConfigException;
use yii\data\ActiveDataProvider;
use yii\helpers\Url;
use yii\web\Controller;
use yii\web\NotFoundHttpException;
use yii\filters\VerbFilter;

// fcgi doesn't have STDIN and STDOUT defined by default
defined('STDIN') or define('STDIN', fopen('php://stdin', 'r'));
// Logical operators have lower precedence:
$f = false or true;

// is executed like this:
($f = false) or true;


// Boolean operators have higher precedence:
$f = false || true;

// is executed like this:
$f = (false || true);
defined('STDOUT') or define('STDOUT', fopen('php://stdout', 'w'));
// Logical operators have lower precedence:
$f = false or true;

// is executed like this:
($f = false) or true;


// Boolean operators have higher precedence:
$f = false || true;

// is executed like this:
$f = (false || true);

class Migrator extends \yii\console\controllers\MigrateController
{
    public function __construct($id, $module, $config = [])
    {
        parent::__construct($id, $module, $config); // TODO: Change the autogenerated stub
    }

    public function runTaxonomy($class, $migrationPath)

    {
        ob_start();
        //$this->migrationPath = Yii::getAlias($migrationPath);

        if($this->beforeAction('up'))
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            $this->migrateUp($class);
        $result = ob_get_contents();
        return $result;
    }

    public function removeTaxonomy($class, $migrationPath)

    {
        ob_start();
        //$this->migrationPath = Yii::getAlias($migrationPath);

        if($this->beforeAction('down'))
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            $this->migrateDown($class);
        $result = ob_get_contents();
        return $result;
    }
}

/**
 * DefController implements the CRUD actions for TaxonomyDef model.
 */
class DefController extends Controller

{
    public function behaviors()
    {
        return [
            'verbs' => [
                'class' => VerbFilter::className(),
                'actions' => [
                    'delete' => ['post'],
                ],
            ],
        ];
    }

    /**
     * Lists all TaxonomyDef models.
     * @return mixed
     * @throws InvalidConfigException
     */
    public function actionIndex()
    {
        if(!$this->getComponent()->isInstalled())
            throw new InvalidConfigException("Please run the migration first!");

        $searchModel = new TaxonomyDefSearch();
        $dataProvider = $searchModel->search(Yii::$app->request->queryParams);

        return $this->render('index', [
            'searchModel' => $searchModel,
            'dataProvider' => $dataProvider,
        ]);
    }

    /**
     * Displays a single TaxonomyDef model.
     * @param integer $id
     * @return mixed
     */
    public function actionView($id)
    {
        $termProvider = new ActiveDataProvider([
            'query' => TaxonomyTerms::find()->andFilterWhere(['taxonomy_id' => $id]),
            'sort' => [
                // Set the default sort by name ASC and created_at DESC.
                'defaultOrder' => [
                    'total_count' => SORT_DESC,
                ]
            ],
        ]);

        return $this->render('view', [
            'model' => $this->findModel($id),
            'termProvider' => $termProvider
        ]);
    }

    /**
     * Creates a new TaxonomyDef model.
     * If creation is successful, the browser will be redirected to the 'view' page.
     * @return mixed
     */
    public function actionCreate()
    {
        $model = new TaxonomyDef();
        $definitions = $this->getComponent()->getDefinitions();

        if ($model->load(Yii::$app->request->post())) {
            //install the term
            $term = Yii::createObject($model->attributes);
                //$this->getComponent()->getTerm($model->name);

            $term->install();
            $migration = new Migrator('migrate', Yii::$app);
            $messsage = $migration->runTaxonomy($term->migration, $term->migrationPath);
            Yii::$app->session->setFlash('info', $messsage, true);

            return $this->redirect(['index', 'id' => $model->id]);
        } else {
            return $this->render('create', [
                'model' => $model,
                'definitions' => $definitions,
            ]);
        }
    }

    /**
     * Updates an existing TaxonomyDef model.
     * If update is successful, the browser will be redirected to the 'view' page.
     * @param integer $id
     * @return mixed
     */
    public function actionUpdate($id)
    {
        $model = $this->findModel($id);
        $definitions = $this->getComponent()->getDefinitions();

        if ($model->load(Yii::$app->request->post()) && $model->save()) {
            return $this->redirect(['view', 'id' => $model->id]);
        } else {
            return $this->render('update', [
                'model' => $model,
                'definitions' => $definitions,
            ]);
        }
    }

    /**
     * Deletes an existing TaxonomyDef model.
     * If deletion is successful, the browser will be redirected to the 'index' page.
     * @param integer $id
     * @return mixed
     */
    public function actionDelete($id)
    {
        $model = $this->findModel($id);
        //UNinstall the term
        $term = $this->getComponent()->getTerm($model->name);
        $migration = new Migrator('migrate', Yii::$app);
        $messsage = $migration->removeTaxonomy($term->migration, $term->migrationPath);
        Yii::$app->session->setFlash('info', $messsage, true);

        $term->uninstall();


        return $this->redirect(['index']);
    }

    /**
     * Finds the TaxonomyDef model based on its primary key value.
     * If the model is not found, a 404 HTTP exception will be thrown.
     * @param integer $id
     * @return TaxonomyDef the loaded model
     * @throws NotFoundHttpException if the model cannot be found
     */
    protected function findModel($id)
    {
        if (($model = TaxonomyDef::findOne($id)) !== null) {
            return $model;
        } else {
            throw new NotFoundHttpException('The requested page does not exist.');
        }
    }

    public function getComponent()
    {
        if(\Yii::$app->has($this->module->component))

            return \Yii::$app->{$this->module->component};

        else
            throw new InvalidConfigException("Cannot find taxonomy component({$this->module->component})");

    }

    public function actionInstall()
    {
        if(!$this->getComponent()->isInstalled() and \Yii::$app->request->isPost) {
// Logical operators have lower precedence:
$f = false or true;

// is executed like this:
($f = false) or true;


// Boolean operators have higher precedence:
$f = false || true;

// is executed like this:
$f = (false || true);
            //start installation
            if($this->getComponent()) {
                $this->getComponent()->install();

                $this->redirect(['/'.$this->module->id . '/' . $this->id . '/index']);
            } else
                throw new InvalidConfigException("Cannot find taxonomy component({$this->module->component})");

        }
        return $this->render('install');
    }

    public function actionInstallterm($term)
    {
        $term = $this->getComponent()->getTerm($term);
        $term->install();
    }
}


Push — master ( 414d92...35a2e4 )

Migrator A

Complexity

Size/Duplication

Coupling/Cohesion

Importance

3 Methods

Logical Operators are used for Control-Flow

Logical Operators are used for Control-Flow

Logical Operators are used for Control-Flow

nkostadinov / yii2-taxonomy

Push — master ( 414d92...35a2e4 )

Migrator A

Complexity

Size/Duplication

Coupling/Cohesion

Importance

3 Methods

Logical Operators are used for Control-Flow

Logical Operators are used for Control-Flow

Logical Operators are used for Control-Flow

Duplication Side-by-Side

Filter issues like