MigrationRunner::run() - Code Metrics - baleen/migrations - Measure and Improve Code Quality continuously with Scrutinizer

MigrationRunner::run() B
last analyzed 2016-04-25 12:51 UTC

↳ Parent: MigrationRunner

Complexity

Conditions	3
Paths	3

Size

Total Lines	26
Code Lines	14

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	15
CRAP Score	3

Importance

Changes	3
Bugs	0	Features	1

Metric	Value
c	3
b	0
f	1
dl	0
loc	26
ccs	15
cts	15
cp	1
rs	8.8571
cc	3
eloc	14
nc	3
nop	2
crap	3

<?php
/*
 * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
 * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
 * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
 * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
 * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
 * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
 * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
 * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
 * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
 * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
 * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
 *
 * This software consists of voluntary contributions made by many individuals
 * and is licensed under the MIT license. For more information, see
 * <http://www.doctrine-project.org>.
 */

namespace Baleen\Migrations\Service\Runner;

use Baleen\Migrations\Exception\Service\Runner\RunnerException;
use Baleen\Migrations\Service\MigrationBus\MigrationBus;
use Baleen\Migrations\Service\MigrationBus\MigrationBusInterface;
use Baleen\Migrations\Migration\OptionsInterface;
use Baleen\Migrations\Service\Runner\Event\Migration\MigrateAfterEvent;
use Baleen\Migrations\Service\Runner\Event\Migration\MigrateBeforeEvent;
use Baleen\Migrations\Common\Event\Context\CollectionContext;
use Baleen\Migrations\Common\Event\Context\CollectionContextInterface;
use Baleen\Migrations\Common\Event\Context\ContextInterface;
use Baleen\Migrations\Common\Event\Context\HasContextTrait;
use Baleen\Migrations\Common\Event\Publisher\HasInternalPublisherTrait;
use Baleen\Migrations\Common\Event\PublisherInterface;
use Baleen\Migrations\Delta\DeltaInterface;

/**
 * Class MigrationRunner
 *
 * A Runner that emits domain events using an Emitter
 *
 * @author Gabriel Somoza <[email protected]>
 */
final class MigrationRunner implements MigrationRunnerInterface
{
    use HasInternalPublisherTrait;
    use HasContextTrait;

    /**
     * MigrationRunner constructor.
     *
     * @param PublisherInterface $publisher

     * @param CollectionContextInterface $context

     */
    public function __construct(
        PublisherInterface $publisher = null,
        CollectionContextInterface $context = null
    ) {
        if (null === $context) {
            $context = CollectionContext::createWithProgress(1, 1);
        }
        $this->setContext($context);

        $this->setPublisher($publisher);
    }

    /**
     * Runs a single version using the specified options
     *
     * @param DeltaInterface $version
     * @param OptionsInterface $options
     *
     * @return false|MigrateAfterEvent
     *
     * @throws RunnerException
     */
    public function run(DeltaInterface $version, OptionsInterface $options)
    {
        if (!$this->shouldMigrate($version, $options)) {
            if ($options->isExceptionOnSkip()) {
                throw new RunnerException(sprintf(
                    'Cowardly refusing to run %s() on a delta that is already "%s" (ID: %s).',
                    $options->getDirection(),
                    $options->getDirection(),
                    $version->getId()
                ));
            }

            return false; // skip
        }

        // Dispatch MIGRATE_BEFORE
        $this->getPublisher()->publish(new MigrateBeforeEvent($version, $options, $this->getContext()));
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);

        $version->migrate($options); // state will be changed

        // Dispatch MIGRATE_AFTER
        $event = new MigrateAfterEvent($version, $options, $this->getContext());
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        $this->getPublisher()->publish($event);

        return $event;
    }

    /**
     * Returns true if the operation is forced, or if the direction is the opposite to the state of the migration.
     *
     * @param DeltaInterface $version
     * @param OptionsInterface $options
     *
     * @return bool
     */
    protected function shouldMigrate(DeltaInterface $version, OptionsInterface $options)
    {
        return $options->isForced()
        || ($options->getDirection()->isUp() ^ $version->isMigrated()); // direction is opposite to state
    }

    /**
     * @inheritdoc
     */
    final public function withContext(ContextInterface $context) {

        return new static($this->getPublisher(), $context);
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
    }
}


1		<?php
2		/*
3		* THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
4		* "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
5		* LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
6		* A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
7		* OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
8		* SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
9		* LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
10		* DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
11		* THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
12		* (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
13		* OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
14		*
15		* This software consists of voluntary contributions made by many individuals
16		* and is licensed under the MIT license. For more information, see
17		* <http://www.doctrine-project.org>.
18		*/
19
20		namespace Baleen\Migrations\Service\Runner;
21
22		use Baleen\Migrations\Exception\Service\Runner\RunnerException;
23		use Baleen\Migrations\Service\MigrationBus\MigrationBus;
24		use Baleen\Migrations\Service\MigrationBus\MigrationBusInterface;
25		use Baleen\Migrations\Migration\OptionsInterface;
26		use Baleen\Migrations\Service\Runner\Event\Migration\MigrateAfterEvent;
27		use Baleen\Migrations\Service\Runner\Event\Migration\MigrateBeforeEvent;
28		use Baleen\Migrations\Common\Event\Context\CollectionContext;
29		use Baleen\Migrations\Common\Event\Context\CollectionContextInterface;
30		use Baleen\Migrations\Common\Event\Context\ContextInterface;
31		use Baleen\Migrations\Common\Event\Context\HasContextTrait;
32		use Baleen\Migrations\Common\Event\Publisher\HasInternalPublisherTrait;
33		use Baleen\Migrations\Common\Event\PublisherInterface;
34		use Baleen\Migrations\Delta\DeltaInterface;
35
36		/**
37		* Class MigrationRunner
38		*
39		* A Runner that emits domain events using an Emitter
40		*
41		* @author Gabriel Somoza <[email protected]>
42		*/
43		final class MigrationRunner implements MigrationRunnerInterface
44		{
45		use HasInternalPublisherTrait;
46		use HasContextTrait;
47
48		/**
49		* MigrationRunner constructor.
50		*
51		* @param PublisherInterface $publisher
		0 ignored issues – show Documentation introduced 2015-11-23 16:45 UTC by Report Bug Copy Issue Report Should the type for parameter `$publisher` not be `null\|PublisherInterface`? This check looks for `@param` annotations where the type inferred by our type inference engine differs from the declared type. It makes a suggestion as to what type it considers more descriptive. Most often this is a case of a parameter that can be null in addition to its declared types. Loading history...
52		* @param CollectionContextInterface $context
		0 ignored issues – show Documentation introduced 2015-11-23 16:45 UTC by Report Bug Copy Issue Report Should the type for parameter `$context` not be `null\|CollectionContextInterface`? This check looks for `@param` annotations where the type inferred by our type inference engine differs from the declared type. It makes a suggestion as to what type it considers more descriptive. Most often this is a case of a parameter that can be null in addition to its declared types. Loading history...
53		*/
54	29	public function __construct(
55		PublisherInterface $publisher = null,
56		CollectionContextInterface $context = null
57		) {
58	29	if (null === $context) {
59	23	$context = CollectionContext::createWithProgress(1, 1);
60	23	}
61	29	$this->setContext($context);
62
63	29	$this->setPublisher($publisher);
64	29	}
65
66		/**
67		* Runs a single version using the specified options
68		*
69		* @param DeltaInterface $version
70		* @param OptionsInterface $options
71		*
72		* @return false\|MigrateAfterEvent
73		*
74		* @throws RunnerException
75		*/
76	23	public function run(DeltaInterface $version, OptionsInterface $options)
77		{
78	23	if (!$this->shouldMigrate($version, $options)) {
79	14	if ($options->isExceptionOnSkip()) {
80	2	throw new RunnerException(sprintf(
81	2	'Cowardly refusing to run %s() on a delta that is already "%s" (ID: %s).',
82	2	$options->getDirection(),
83	2	$options->getDirection(),
84	2	$version->getId()
85	2	));
86		}
87
88	12	return false; // skip
89		}
90
91		// Dispatch MIGRATE_BEFORE
92	13	$this->getPublisher()->publish(new MigrateBeforeEvent($version, $options, $this->getContext()));
		0 ignored issues – show Documentation introduced 2016-04-25 10:19 UTC by Report Bug Copy Issue Report `$this->getContext()` is of type `object<Baleen\Migrations...ntext\ContextInterface>`, but the function expects a `null\|object<Baleen\Migra...ectionContextInterface>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
93
94	13	$version->migrate($options); // state will be changed
95
96		// Dispatch MIGRATE_AFTER
97	13	$event = new MigrateAfterEvent($version, $options, $this->getContext());
		0 ignored issues – show Documentation introduced 2016-04-25 10:19 UTC by Report Bug Copy Issue Report `$this->getContext()` is of type `object<Baleen\Migrations...ntext\ContextInterface>`, but the function expects a `null\|object<Baleen\Migra...ectionContextInterface>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
98	13	$this->getPublisher()->publish($event);
99
100	13	return $event;
101		}
102
103		/**
104		* Returns true if the operation is forced, or if the direction is the opposite to the state of the migration.
105		*
106		* @param DeltaInterface $version
107		* @param OptionsInterface $options
108		*
109		* @return bool
110		*/
111	23	protected function shouldMigrate(DeltaInterface $version, OptionsInterface $options)
112		{
113	23	return $options->isForced()
114	23	\|\| ($options->getDirection()->isUp() ^ $version->isMigrated()); // direction is opposite to state
115		}
116
117		/**
118		* @inheritdoc
119		*/
120	18	final public function withContext(ContextInterface $context) {
		0 ignored issues – show Coding Style introduced 2015-11-26 05:55 UTC by Report Bug Copy Issue Report Unnecessary FINAL modifier in FINAL class Loading history...
121	18	return new static($this->getPublisher(), $context);
		0 ignored issues – show Documentation introduced 2016-04-25 10:19 UTC by Report Bug Copy Issue Report `$context` is of type `object<Baleen\Migrations...ntext\ContextInterface>`, but the function expects a `null\|object<Baleen\Migra...ectionContextInterface>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
122		}
123		}
124

baleen / migrations

MigrationRunner::run() B last analyzed 2016-04-25 12:51 UTC

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like

MigrationRunner::run() B
last analyzed 2016-04-25 12:51 UTC