Migration::migrate() - Code Metrics - Inspection of "Refactoring" - czogori/Dami - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 7bba35...80067f )

by Arkadiusz

created 2016-01-24 23:23 UTC

Migration::migrate() A

↳ Parent: Migration

Complexity

Conditions	1
Paths	1

Size

Total Lines	4
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Importance

Changes	6
Bugs	1	Features	2

Metric	Value
dl	0
loc	4
rs	10
c	6
b	1
f	2
cc	1
eloc	2
nc	1
nop	2

<?php

namespace Dami;

use Dami\Migration\MigrationFiles;
use Dami\Migration\SchemaTable;
use Dami\Migration\Api\MigrationApi;
use Rentgen\Schema\Info;
use Rentgen\Schema\Manipulation;

class Migration
{
    /**
     * @param SchemaTable    $schemaTable        description
     * @param MigrationFiles $migrationFiles     description
     * @param Manipulation   $schemaManipulation description
     * @param Info           $schemaInfo         description
     */
    public function __construct(SchemaTable $schemaTable, MigrationFiles $migrationFiles, Manipulation $schemaManipulation, Info $schemaInfo)
    {
        $this->schemaTable = $schemaTable;
class MyClass { }

$x = new MyClass();
$x->foo = true;
        $this->migrationFiles = $migrationFiles;
class MyClass { }

$x = new MyClass();
$x->foo = true;
        $this->schemaManipulation = $schemaManipulation;
class MyClass { }

$x = new MyClass();
$x->foo = true;
        $this->schemaInfo = $schemaInfo;
class MyClass { }

$x = new MyClass();
$x->foo = true;
    }

    /**
     * Migrate the schema to the given version.
     *
     * @return integer Number of migrations.
     */
    public function migrate($version = null, $message = null)
    {
        return $this->execute($version, $message);
    }

    /**
     * Migrate the schema to previous version.
     *
     * @return integer Number of migrations.
     */
    public function migrateToPreviousVersion($message)
    {
        return $this->execute($this->schemaTable->getPreviousVersion(), $message);
    }

    /**
     * Create an instance of migration class.
     *
     * @param $className Name of migration class.
     *
     * @return mixed
     */
    protected function createMigrationApiInstance($className)
    {
        return new $className($this->schemaManipulation, $this->schemaInfo);
    }

    /**
     * Execute migrate.
     *
     * @param string $version The version of migration to rollback or migrate.
     *
     * @return integer Number of executed migrations.
     */
    private function execute($version = null, $message = null)
    {
        $migrateUp = null === $version || $version > $this->schemaTable->getCurrentVersion();
        $files = $this->migrationFiles->get($version);
        if (null === $files) {
            return 0;
        }
        $this->schemaManipulation->execute('BEGIN');
        try {
            foreach ($files as $file) {

                require_once $file->getPath();

                $definition = $this->createMigrationApiInstance($file->getClassName());

                if ($message) {
                    $message($file->getName(), $file->getVersion());
                }
                if ($migrateUp) {
                    $definition->up();
                } else {
                    $definition->down();
                }
                foreach ($definition->getActions() as $action) {
                    if (!is_callable($action)) {
                        throw new \InvalidArgumentException('Migration must be callable');
                    }
                    $action = call_user_func_array($action, array());
                    if ($action instanceof MigrationApi) {
                        $action->execute();

                    }
                }
                $this->schemaTable->migrateToVersion($file->getVersion());
            }
            $this->schemaManipulation->execute('COMMIT');
        } catch (\Exception $e) {
            $this->schemaManipulation->execute('ROLLBACK');
            throw $e;
        }

        return count($files);
    }
}


1			<?php
2
3			namespace Dami;
4
5			use Dami\Migration\MigrationFiles;
6			use Dami\Migration\SchemaTable;
7			use Dami\Migration\Api\MigrationApi;
8			use Rentgen\Schema\Info;
9			use Rentgen\Schema\Manipulation;
10
11			class Migration
12			{
13			/**
14			* @param SchemaTable $schemaTable description
15			* @param MigrationFiles $migrationFiles description
16			* @param Manipulation $schemaManipulation description
17			* @param Info $schemaInfo description
18			*/
19			public function __construct(SchemaTable $schemaTable, MigrationFiles $migrationFiles, Manipulation $schemaManipulation, Info $schemaInfo)
20			{
21			$this->schemaTable = $schemaTable;
			0 ignored issues – show Bug introduced 2016-01-23 10:43 UTC by Report Bug Copy Issue Report The property `schemaTable` does not exist. Did you maybe forget to declare it? In PHP it is possible to write to properties without declaring them. For example, the following is perfectly valid PHP code: class MyClass { } $x = new MyClass(); $x->foo = true; Generally, it is a good practice to explictly declare properties to avoid accidental typos and provide IDE auto-completion: class MyClass { public $foo; } $x = new MyClass(); $x->foo = true; Loading history...
22			$this->migrationFiles = $migrationFiles;
			0 ignored issues – show Bug introduced 2016-01-23 10:43 UTC by Report Bug Copy Issue Report The property `migrationFiles` does not exist. Did you maybe forget to declare it? In PHP it is possible to write to properties without declaring them. For example, the following is perfectly valid PHP code: class MyClass { } $x = new MyClass(); $x->foo = true; Generally, it is a good practice to explictly declare properties to avoid accidental typos and provide IDE auto-completion: class MyClass { public $foo; } $x = new MyClass(); $x->foo = true; Loading history...
23			$this->schemaManipulation = $schemaManipulation;
			0 ignored issues – show Bug introduced 2016-01-23 10:43 UTC by Report Bug Copy Issue Report The property `schemaManipulation` does not exist. Did you maybe forget to declare it? In PHP it is possible to write to properties without declaring them. For example, the following is perfectly valid PHP code: class MyClass { } $x = new MyClass(); $x->foo = true; Generally, it is a good practice to explictly declare properties to avoid accidental typos and provide IDE auto-completion: class MyClass { public $foo; } $x = new MyClass(); $x->foo = true; Loading history...
24			$this->schemaInfo = $schemaInfo;
			0 ignored issues – show Bug introduced 2016-01-23 10:43 UTC by Report Bug Copy Issue Report The property `schemaInfo` does not exist. Did you maybe forget to declare it? In PHP it is possible to write to properties without declaring them. For example, the following is perfectly valid PHP code: class MyClass { } $x = new MyClass(); $x->foo = true; Generally, it is a good practice to explictly declare properties to avoid accidental typos and provide IDE auto-completion: class MyClass { public $foo; } $x = new MyClass(); $x->foo = true; Loading history...
25			}
26
27			/**
28			* Migrate the schema to the given version.
29			*
30			* @return integer Number of migrations.
31			*/
32			public function migrate($version = null, $message = null)
33			{
34			return $this->execute($version, $message);
35			}
36
37			/**
38			* Migrate the schema to previous version.
39			*
40			* @return integer Number of migrations.
41			*/
42			public function migrateToPreviousVersion($message)
43			{
44			return $this->execute($this->schemaTable->getPreviousVersion(), $message);
45			}
46
47			/**
48			* Create an instance of migration class.
49			*
50			* @param $className Name of migration class.
51			*
52			* @return mixed
53			*/
54			protected function createMigrationApiInstance($className)
55			{
56			return new $className($this->schemaManipulation, $this->schemaInfo);
57			}
58
59			/**
60			* Execute migrate.
61			*
62			* @param string $version The version of migration to rollback or migrate.
63			*
64			* @return integer Number of executed migrations.
65			*/
66			private function execute($version = null, $message = null)
67			{
68			$migrateUp = null === $version \|\| $version > $this->schemaTable->getCurrentVersion();
69			$files = $this->migrationFiles->get($version);
70			if (null === $files) {
71			return 0;
72			}
73			$this->schemaManipulation->execute('BEGIN');
74			try {
75			foreach ($files as $file) {
76
77			require_once $file->getPath();
78
79			$definition = $this->createMigrationApiInstance($file->getClassName());
80
81			if ($message) {
82			$message($file->getName(), $file->getVersion());
83			}
84			if ($migrateUp) {
85			$definition->up();
86			} else {
87			$definition->down();
88			}
89			foreach ($definition->getActions() as $action) {
90			if (!is_callable($action)) {
91			throw new \InvalidArgumentException('Migration must be callable');
92			}
93			$action = call_user_func_array($action, array());
94			if ($action instanceof MigrationApi) {
95			$action->execute();
			0 ignored issues – show Bug introduced 2016-01-23 10:43 UTC by Report Bug Copy Issue Report The call to `execute()` misses a required argument `$sql`. This check looks for function calls that miss required arguments. Loading history...
96			}
97			}
98			$this->schemaTable->migrateToVersion($file->getVersion());
99			}
100			$this->schemaManipulation->execute('COMMIT');
101			} catch (\Exception $e) {
102			$this->schemaManipulation->execute('ROLLBACK');
103			throw $e;
104			}
105
106			return count($files);
107			}
108			}
109

czogori / Dami

GitHub Access Token became invalid

Push — master ( 7bba35...80067f )

Migration::migrate() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like