Migrate::reverseActions() - Code Metrics - Inspection of "Removing an extra PHP version and bringing back op..." - ntentan/yentu - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — dev ( 9476d8...09c84d )

by James Ekow Abaka

created 2019-10-23 05:32 UTC

Migrate::reverseActions() A

↳ Parent: Project

Complexity

Conditions	3
Paths	3

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	12

Importance

Changes

Metric	Value
cc	3
nc	3
nop	0
dl	0
loc	14
ccs	0
cts	10
cp	0
crap	12
rs	9.7998
c	0
b	0
f	0

<?php

/*
 * The MIT License
 *
 * Copyright 2014 James Ekow Abaka Ainooson
 *
 * Permission is hereby granted, free of charge, to any person obtaining a copy
 * of this software and associated documentation files (the "Software"), to deal
 * in the Software without restriction, including without limitation the rights
 * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
 * copies of the Software, and to permit persons to whom the Software is
 * furnished to do so, subject to the following conditions:
 *
 * The above copyright notice and this permission notice shall be included in
 * all copies or substantial portions of the Software.
 *
 * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
 * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
 * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
 * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
 * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
 * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
 * THE SOFTWARE.
 */

namespace yentu\commands;

use clearice\io\Io;
use yentu\database\Begin;
use yentu\database\DatabaseItem;
use yentu\ChangeLogger;
use yentu\database\ForeignKey;
use yentu\factories\DatabaseManipulatorFactory;
use yentu\Migrations;

/**
 * The migrate command for the yentu database migration system. This class is
 * responsible for creating and updating items 
 */
class Migrate extends Command implements Reversible
{

    const FILTER_UNRUN = 'unrun';
    const FILTER_LAST_SESSION = 'lastSession';

    private $driver;
    private $dryDriver;
    private $defaultSchema = false;
    private $lastSession;
    private $currentPath;
    private $rollbackCommand;
    private $manipulatorFactory;
    private $migrations;
    private $io;

    public function __construct(Migrations $migrations, DatabaseManipulatorFactory $manipulatorFactory, Io $io)
    {
        $this->manipulatorFactory = $manipulatorFactory;
        $this->migrations = $migrations;
        $this->io = $io;
    }

    public function setupOptions($options, &$filter)
    {
        if (isset($options['no-foreign-keys'])) {
            $this->io->output("Ignoring all foreign key constraints ...\n");
            $this->driver->skip('ForeignKey');
        }

        if (isset($options['only-foreign-keys'])) {
            $this->io->output("Applying only foreign keys ...\n");
            $this->lastSession = $this->driver->getLastSession();
class ParentClass {
    private $data = array();

    public function __call($method, array $args) {
        if (0 === strpos($method, 'get')) {
            return $this->data[strtolower(substr($method, 3))];
        }

        throw new \LogicException(sprintf('Unsupported method: %s', $method));
    }
}

/**
 * If this class knows which fields exist, you can specify the methods here:
 *
 * @method string getName()
 */
class SomeClass extends ParentClass { }
            $this->driver->allowOnly('ForeignKey');
            $filter = self::FILTER_LAST_SESSION;
        }

        if (isset($options['force-foreign-keys'])) {
            $this->io->output("Applying only foreign keys and skipping on errors ...\n");
            $this->lastSession = $this->driver->getLastSession();
class ParentClass {
    private $data = array();

    public function __call($method, array $args) {
        if (0 === strpos($method, 'get')) {
            return $this->data[strtolower(substr($method, 3))];
        }

        throw new \LogicException(sprintf('Unsupported method: %s', $method));
    }
}

/**
 * If this class knows which fields exist, you can specify the methods here:
 *
 * @method string getName()
 */
class SomeClass extends ParentClass { }
            $this->driver->setSkipOnErrors($options['force-foreign-keys']);
            $this->driver->allowOnly('ForeignKey');
            $filter = self::FILTER_LAST_SESSION;
        }

        if (isset($options['default-ondelete-action'])) {
            ForeignKey::$defaultOnDelete = $options['default-ondelete-action'];
        }

        if (isset($options['default-onupdate-action'])) {
            ForeignKey::$defaultOnUpdate = $options['default-onupdate-action'];
        }

        $this->setDefaultSchema($options);
    }

    private function setDefaultSchema($options)
    {
        global $defaultSchema;
        if (isset($options['default-schema'])) {
            $this->driver->setDefaultSchema($options['default-schema']);
            $this->defaultSchema = $options['default-schema'];
            $defaultSchema = $this->defaultSchema;
        }
    }

    private function announceMigration($migrations, $path)
    {
        $size = count($migrations);
        $defaultSchema = null;
        if ($size > 0) {
            if (isset($path['default-schema'])) {
                $defaultSchema = $path['default-schema'];
            }
            $this->io->output("Running $size migration(s) from '{$path['home']}'");
            if ($defaultSchema != '') {
                $this->io->output(" with '$defaultSchema' as the default schema.\n");
            }
        } else {
            $this->io->output("No migrations to run from '{$path['home']}'\n");
        }
    }

    public function getBegin()
    {
        return new Begin($this->defaultSchema);
    }

    private static function fillOptions(&$options)
    {
        if (!isset($options['dump-queries'])) {
            $options['dump-queries'] = false;
        }
        if (!isset($options['dry'])) {
            $options['dry'] = false;
        }
    }

    public function run()
    {
        global $migrateCommand;
        global $migrateVariables;

        self::fillOptions($this->options);

        $migrateCommand = $this;

        $this->driver = ChangeLogger::wrap($this->manipulatorFactory->createManipulator(), $this->migrations, $this->io);
        $this->driver->setDumpQueriesOnly($this->options['dump-queries']);
        $this->driver->setDryRun($this->options['dry']);

        $totalOperations = 0;

        $filter = self::FILTER_UNRUN;
        $this->setupOptions($this->options, $filter);
        DatabaseItem::setDriver($this->driver);

        \yentu\Timer::start();
        $migrationPaths = $this->migrations->getAllPaths();
        //$migrationsToBeRun = [];
        foreach ($migrationPaths as $path) {
            $this->setDefaultSchema($path);
            $migrateVariables = $path['variables'] ?? [];
            $migrations = $this->filter($this->migrations->getMigrationFiles($path['home']), $filter);
            $this->announceMigration($migrations, $path);
            $this->currentPath = $path;

            foreach ($migrations as $migration) {
                $this->countOperations("{$path['home']}/{$migration['file']}");
                $this->driver->setVersion($migration['timestamp']);
                $this->driver->setMigration($migration['migration']);
                $this->io->output("\nApplying '{$migration['migration']}' migration\n");
                require "{$path['home']}/{$migration['file']}";
                DatabaseItem::purge();
                $this->io->output("\n");
                $totalOperations += $this->driver->resetOperations();
            }
        }

        if ($this->driver->getChanges()) {
            $elapsed = \yentu\Timer::stop();
            $this->io->output("\nMigration took " . \yentu\Timer::pretty($elapsed) . "\n");
            $this->io->output($this->driver->getChanges() . " operations performed\n");
            $this->io->output($totalOperations - $this->driver->getChanges() . " operations skipped\n");
        }

        $this->driver->disconnect();
class ParentClass {
    private $data = array();

    public function __call($method, array $args) {
        if (0 === strpos($method, 'get')) {
            return $this->data[strtolower(substr($method, 3))];
        }

        throw new \LogicException(sprintf('Unsupported method: %s', $method));
    }
}

/**
 * If this class knows which fields exist, you can specify the methods here:
 *
 * @method string getName()
 */
class SomeClass extends ParentClass { }
    }

    private function filter($migrations, $type = self::FILTER_UNRUN)

    {
        $filterMethod = "{$type}Filter";
        return $this->$filterMethod($migrations);
    }

    private function countOperations($migrationFile)
    {
        if ($this->dryDriver === null) {
            $this->dryDriver = clone $this->driver;
            $this->dryDriver->setDryRun(true);
        }
        $this->io->pushOutputLevel(Io::OUTPUT_LEVEL_0);
        DatabaseItem::setDriver($this->dryDriver);
        require "$migrationFile";
        DatabaseItem::purge();
        DatabaseItem::setDriver($this->driver);
        $this->io->popOutputLevel();
        $this->driver->setExpectedOperations($this->dryDriver->resetOperations());
    }

    public function getCurrentPath()
    {
        return $this->currentPath;
    }

    private function unrunFilter($input)
    {
        $output = array();
        foreach ($input as $migration) {
            $run = $this->driver->query(
class ParentClass {
    private $data = array();

    public function __call($method, array $args) {
        if (0 === strpos($method, 'get')) {
            return $this->data[strtolower(substr($method, 3))];
        }

        throw new \LogicException(sprintf('Unsupported method: %s', $method));
    }
}

/**
 * If this class knows which fields exist, you can specify the methods here:
 *
 * @method string getName()
 */
class SomeClass extends ParentClass { }
                "SELECT count(*) as number_run FROM yentu_history WHERE migration = ? and version = ? and default_schema = ?", 
                array($migration['migration'], $migration['timestamp'], (string) $this->defaultSchema)
            );

            if ($run[0]['number_run'] == 0) {
                $output[] = $migration;
            }
        }
        return $output;
    }

    private function lastSessionFilter($input)
    {
        $versions = $this->driver->getSessionVersions($this->lastSession);
class ParentClass {
    private $data = array();

    public function __call($method, array $args) {
        if (0 === strpos($method, 'get')) {
            return $this->data[strtolower(substr($method, 3))];
        }

        throw new \LogicException(sprintf('Unsupported method: %s', $method));
    }
}

/**
 * If this class knows which fields exist, you can specify the methods here:
 *
 * @method string getName()
 */
class SomeClass extends ParentClass { }
        $output = array();
        foreach ($input as $migration) {
            if (array_search($migration['timestamp'], $versions) !== false) {
                $output[] = $migration;
            }
        }
        return $output;
    }

    public function getChanges()

    {
        return $this->driver->getChanges();
    }

    public function setRollbackCommand(Rollback $rollbackCommand)
    {
        $this->rollbackCommand = $rollbackCommand;
    }

    public function reverseActions()
    {
        if ($this->driver === null) {
            return;
        }

        $this->io->output("Attempting to reverse all changes ... ");
        if ($this->getChanges() > 0) {
            $this->io->pushOutputLevel(0);
            $this->rollbackCommand->run(array());

            $this->io->popOutputLevel();
        }
        $this->io->output("OK\n");
    }

}


1			<?php
2
3			/*
4			* The MIT License
5			*
6			* Copyright 2014 James Ekow Abaka Ainooson
7			*
8			* Permission is hereby granted, free of charge, to any person obtaining a copy
9			* of this software and associated documentation files (the "Software"), to deal
10			* in the Software without restriction, including without limitation the rights
11			* to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
12			* copies of the Software, and to permit persons to whom the Software is
13			* furnished to do so, subject to the following conditions:
14			*
15			* The above copyright notice and this permission notice shall be included in
16			* all copies or substantial portions of the Software.
17			*
18			* THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
19			* IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
20			* FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
21			* AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
22			* LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
23			* OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
24			* THE SOFTWARE.
25			*/
26
27			namespace yentu\commands;
28
29			use clearice\io\Io;
30			use yentu\database\Begin;
31			use yentu\database\DatabaseItem;
32			use yentu\ChangeLogger;
33			use yentu\database\ForeignKey;
34			use yentu\factories\DatabaseManipulatorFactory;
35			use yentu\Migrations;
36
37			/**
38			* The migrate command for the yentu database migration system. This class is
39			* responsible for creating and updating items
40			*/
41			class Migrate extends Command implements Reversible
42			{
43
44			const FILTER_UNRUN = 'unrun';
45			const FILTER_LAST_SESSION = 'lastSession';
46
47			private $driver;
48			private $dryDriver;
49			private $defaultSchema = false;
50			private $lastSession;
51			private $currentPath;
52			private $rollbackCommand;
53			private $manipulatorFactory;
54			private $migrations;
55			private $io;
56
57	9		public function __construct(Migrations $migrations, DatabaseManipulatorFactory $manipulatorFactory, Io $io)
58			{
59	9		$this->manipulatorFactory = $manipulatorFactory;
60	9		$this->migrations = $migrations;
61	9		$this->io = $io;
62	9		}
63
64	9		public function setupOptions($options, &$filter)
65			{
66	9		if (isset($options['no-foreign-keys'])) {
67	3		$this->io->output("Ignoring all foreign key constraints ...\n");
68	3		$this->driver->skip('ForeignKey');
69			}
70
71	9	View Code Duplication	if (isset($options['only-foreign-keys'])) {
72	3		$this->io->output("Applying only foreign keys ...\n");
73	3		$this->lastSession = $this->driver->getLastSession();
			0 ignored issues – show Documentation Bug introduced 2017-07-29 08:50 UTC by Report Bug Copy Issue Report The method `getLastSession` does not exist on `object<yentu\ChangeLogger>`? Since you implemented `__call`, maybe consider adding a @method annotation. If you implement `__call` and you know which methods are available, you can improve IDE auto-completion and static analysis by adding a @method annotation to the class. This is often the case, when `__call` is implemented by a parent class and only the child class knows which methods exist: class ParentClass { private $data = array(); public function __call($method, array $args) { if (0 === strpos($method, 'get')) { return $this->data[strtolower(substr($method, 3))]; } throw new \LogicException(sprintf('Unsupported method: %s', $method)); } } /** * If this class knows which fields exist, you can specify the methods here: * * @method string getName() */ class SomeClass extends ParentClass { } Loading history...
74	3		$this->driver->allowOnly('ForeignKey');
75	3		$filter = self::FILTER_LAST_SESSION;
76			}
77
78	9	View Code Duplication	if (isset($options['force-foreign-keys'])) {
79			$this->io->output("Applying only foreign keys and skipping on errors ...\n");
80			$this->lastSession = $this->driver->getLastSession();
			0 ignored issues – show Documentation Bug introduced 2017-07-29 08:50 UTC by Report Bug Copy Issue Report The method `getLastSession` does not exist on `object<yentu\ChangeLogger>`? Since you implemented `__call`, maybe consider adding a @method annotation. If you implement `__call` and you know which methods are available, you can improve IDE auto-completion and static analysis by adding a @method annotation to the class. This is often the case, when `__call` is implemented by a parent class and only the child class knows which methods exist: class ParentClass { private $data = array(); public function __call($method, array $args) { if (0 === strpos($method, 'get')) { return $this->data[strtolower(substr($method, 3))]; } throw new \LogicException(sprintf('Unsupported method: %s', $method)); } } /** * If this class knows which fields exist, you can specify the methods here: * * @method string getName() */ class SomeClass extends ParentClass { } Loading history...
81			$this->driver->setSkipOnErrors($options['force-foreign-keys']);
82			$this->driver->allowOnly('ForeignKey');
83			$filter = self::FILTER_LAST_SESSION;
84			}
85
86	9		if (isset($options['default-ondelete-action'])) {
87			ForeignKey::$defaultOnDelete = $options['default-ondelete-action'];
88			}
89
90	9		if (isset($options['default-onupdate-action'])) {
91			ForeignKey::$defaultOnUpdate = $options['default-onupdate-action'];
92			}
93
94	9		$this->setDefaultSchema($options);
95	9		}
96
97	9		private function setDefaultSchema($options)
98			{
99	9		global $defaultSchema;
100	9		if (isset($options['default-schema'])) {
101			$this->driver->setDefaultSchema($options['default-schema']);
102			$this->defaultSchema = $options['default-schema'];
103			$defaultSchema = $this->defaultSchema;
104			}
105	9		}
106
107	9		private function announceMigration($migrations, $path)
108			{
109	9		$size = count($migrations);
110	9		$defaultSchema = null;
111	9		if ($size > 0) {
112	9		if (isset($path['default-schema'])) {
113			$defaultSchema = $path['default-schema'];
114			}
115	9		$this->io->output("Running $size migration(s) from '{$path['home']}'");
116	9		if ($defaultSchema != '') {
117	9		$this->io->output(" with '$defaultSchema' as the default schema.\n");
118			}
119			} else {
120			$this->io->output("No migrations to run from '{$path['home']}'\n");
121			}
122	9		}
123
124	9		public function getBegin()
125			{
126	9		return new Begin($this->defaultSchema);
127			}
128
129	9		private static function fillOptions(&$options)
130			{
131	9		if (!isset($options['dump-queries'])) {
132	9		$options['dump-queries'] = false;
133			}
134	9		if (!isset($options['dry'])) {
135	9		$options['dry'] = false;
136			}
137	9		}
138
139	9		public function run()
140			{
141	9		global $migrateCommand;
142	9		global $migrateVariables;
143
144	9		self::fillOptions($this->options);
145
146	9		$migrateCommand = $this;
147
148	9		$this->driver = ChangeLogger::wrap($this->manipulatorFactory->createManipulator(), $this->migrations, $this->io);
149	9		$this->driver->setDumpQueriesOnly($this->options['dump-queries']);
150	9		$this->driver->setDryRun($this->options['dry']);
151
152	9		$totalOperations = 0;
153
154	9		$filter = self::FILTER_UNRUN;
155	9		$this->setupOptions($this->options, $filter);
156	9		DatabaseItem::setDriver($this->driver);
157
158	9		\yentu\Timer::start();
159	9		$migrationPaths = $this->migrations->getAllPaths();
160			//$migrationsToBeRun = [];
161	9		foreach ($migrationPaths as $path) {
162	9		$this->setDefaultSchema($path);
163	9		$migrateVariables = $path['variables'] ?? [];
164	9		$migrations = $this->filter($this->migrations->getMigrationFiles($path['home']), $filter);
165	9		$this->announceMigration($migrations, $path);
166	9		$this->currentPath = $path;
167
168	9		foreach ($migrations as $migration) {
169	9		$this->countOperations("{$path['home']}/{$migration['file']}");
170	9		$this->driver->setVersion($migration['timestamp']);
171	9		$this->driver->setMigration($migration['migration']);
172	9		$this->io->output("\nApplying '{$migration['migration']}' migration\n");
173	9		require "{$path['home']}/{$migration['file']}";
174	9		DatabaseItem::purge();
175	9		$this->io->output("\n");
176	9		$totalOperations += $this->driver->resetOperations();
177			}
178			}
179
180	9		if ($this->driver->getChanges()) {
181	9		$elapsed = \yentu\Timer::stop();
182	9		$this->io->output("\nMigration took " . \yentu\Timer::pretty($elapsed) . "\n");
183	9		$this->io->output($this->driver->getChanges() . " operations performed\n");
184	9		$this->io->output($totalOperations - $this->driver->getChanges() . " operations skipped\n");
185			}
186
187	9		$this->driver->disconnect();
			0 ignored issues – show Documentation Bug introduced 2017-07-29 08:50 UTC by Report Bug Copy Issue Report The method `disconnect` does not exist on `object<yentu\ChangeLogger>`? Since you implemented `__call`, maybe consider adding a @method annotation. If you implement `__call` and you know which methods are available, you can improve IDE auto-completion and static analysis by adding a @method annotation to the class. This is often the case, when `__call` is implemented by a parent class and only the child class knows which methods exist: class ParentClass { private $data = array(); public function __call($method, array $args) { if (0 === strpos($method, 'get')) { return $this->data[strtolower(substr($method, 3))]; } throw new \LogicException(sprintf('Unsupported method: %s', $method)); } } /** * If this class knows which fields exist, you can specify the methods here: * * @method string getName() */ class SomeClass extends ParentClass { } Loading history...
188	9		}
189
190	9		private function filter($migrations, $type = self::FILTER_UNRUN)
			0 ignored issues – show Documentation introduced 2017-08-26 21:52 UTC by Report Bug Copy Issue Report The return type could not be reliably inferred; please add a `@return` annotation. Our type inference engine in quite powerful, but sometimes the code does not provide enough clues to go by. In these cases we request you to add a `@return` annotation as described here. Loading history...
191			{
192	9		$filterMethod = "{$type}Filter";
193	9		return $this->$filterMethod($migrations);
194			}
195
196	9		private function countOperations($migrationFile)
197			{
198	9		if ($this->dryDriver === null) {
199	9		$this->dryDriver = clone $this->driver;
200	9		$this->dryDriver->setDryRun(true);
201			}
202	9		$this->io->pushOutputLevel(Io::OUTPUT_LEVEL_0);
203	9		DatabaseItem::setDriver($this->dryDriver);
204	9		require "$migrationFile";
205	9		DatabaseItem::purge();
206	9		DatabaseItem::setDriver($this->driver);
207	9		$this->io->popOutputLevel();
208	9		$this->driver->setExpectedOperations($this->dryDriver->resetOperations());
209	9		}
210
211			public function getCurrentPath()
212			{
213			return $this->currentPath;
214			}
215
216	9		private function unrunFilter($input)
217			{
218	9		$output = array();
219	9		foreach ($input as $migration) {
220	9		$run = $this->driver->query(
			0 ignored issues – show Documentation Bug introduced 2017-07-29 08:50 UTC by Report Bug Copy Issue Report The method `query` does not exist on `object<yentu\ChangeLogger>`? Since you implemented `__call`, maybe consider adding a @method annotation. If you implement `__call` and you know which methods are available, you can improve IDE auto-completion and static analysis by adding a @method annotation to the class. This is often the case, when `__call` is implemented by a parent class and only the child class knows which methods exist: class ParentClass { private $data = array(); public function __call($method, array $args) { if (0 === strpos($method, 'get')) { return $this->data[strtolower(substr($method, 3))]; } throw new \LogicException(sprintf('Unsupported method: %s', $method)); } } /** * If this class knows which fields exist, you can specify the methods here: * * @method string getName() */ class SomeClass extends ParentClass { } Loading history...
221	9		"SELECT count(*) as number_run FROM yentu_history WHERE migration = ? and version = ? and default_schema = ?",
222	9		array($migration['migration'], $migration['timestamp'], (string) $this->defaultSchema)
223			);
224
225	9		if ($run[0]['number_run'] == 0) {
226	9		$output[] = $migration;
227			}
228			}
229	9		return $output;
230			}
231
232	3		private function lastSessionFilter($input)
233			{
234	3		$versions = $this->driver->getSessionVersions($this->lastSession);
			0 ignored issues – show Documentation Bug introduced 2017-07-29 08:50 UTC by Report Bug Copy Issue Report The method `getSessionVersions` does not exist on `object<yentu\ChangeLogger>`? Since you implemented `__call`, maybe consider adding a @method annotation. If you implement `__call` and you know which methods are available, you can improve IDE auto-completion and static analysis by adding a @method annotation to the class. This is often the case, when `__call` is implemented by a parent class and only the child class knows which methods exist: class ParentClass { private $data = array(); public function __call($method, array $args) { if (0 === strpos($method, 'get')) { return $this->data[strtolower(substr($method, 3))]; } throw new \LogicException(sprintf('Unsupported method: %s', $method)); } } /** * If this class knows which fields exist, you can specify the methods here: * * @method string getName() */ class SomeClass extends ParentClass { } Loading history...
235	3		$output = array();
236	3		foreach ($input as $migration) {
237	3		if (array_search($migration['timestamp'], $versions) !== false) {
238	3		$output[] = $migration;
239			}
240			}
241	3		return $output;
242			}
243
244			public function getChanges()
			0 ignored issues – show Documentation introduced 2017-08-26 21:52 UTC by Report Bug Copy Issue Report The return type could not be reliably inferred; please add a `@return` annotation. Our type inference engine in quite powerful, but sometimes the code does not provide enough clues to go by. In these cases we request you to add a `@return` annotation as described here. Loading history...
245			{
246			return $this->driver->getChanges();
247			}
248
249			public function setRollbackCommand(Rollback $rollbackCommand)
250			{
251			$this->rollbackCommand = $rollbackCommand;
252			}
253
254			public function reverseActions()
255			{
256			if ($this->driver === null) {
257			return;
258			}
259
260			$this->io->output("Attempting to reverse all changes ... ");
261			if ($this->getChanges() > 0) {
262			$this->io->pushOutputLevel(0);
263			$this->rollbackCommand->run(array());
			0 ignored issues – show Unused Code introduced 2019-10-23 05:34 UTC by Report Bug Copy Issue Report The call to `Rollback::run()` has too many arguments starting with `array()`. This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue. If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress. In this case you can add the `@ignore` PhpDoc annotation to the duplicate definition and it will be ignored. Loading history...
264			$this->io->popOutputLevel();
265			}
266			$this->io->output("OK\n");
267			}
268
269			}
270

ntentan / yentu

Push — dev ( 9476d8...09c84d )

Migrate::reverseActions() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like