ApplyController::__construct() - Code Metrics - Inspection of "refactor migration apply command" - t4web/Migrations - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( bdc00f...dd9c2b )

by max

created 2016-03-04 07:52 UTC

ApplyController::__construct() A

↳ Parent: ApplyController

Complexity

Conditions	1
Paths	1

Size

Total Lines	4
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

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

<?php

namespace T4web\Migrations\Controller;

use Zend\Mvc\Controller\AbstractActionController;
use Zend\Console\Request as ConsoleRequest;
use Zend\Mvc\MvcEvent;
use T4web\Migrations\Service\Migration;
use T4web\Migrations\Exception\RuntimeException;

class ApplyController extends AbstractActionController
{
    /**
     * @var Migration
     */
    protected $migration;

    /**
     * @param Migration $migration
     */
    public function __construct(Migration $migration)
    {
        $this->migration = $migration;
    }

    public function onDispatch(MvcEvent $e)
    {
        if (!$e->getRequest() instanceof ConsoleRequest) {
            throw new RuntimeException('You can only use this action from a console!');
        }

        $version = $e->getRequest()->getParam('version');
        $force = $e->getRequest()->getParam('force');
        $down = $e->getRequest()->getParam('down');
        $fake = $e->getRequest()->getParam('fake');

        if (is_null($version) && $force) {
            $response = "Can't force migration apply without migration version explicitly set.\n";
            $e->setResult($response);
            return $response;
        }

        if (is_null($version) && $fake) {
            $response = "Can't fake migration apply without migration version explicitly set.\n";
            $e->setResult($response);
            return $response;
        }

        $this->migration->migrate($version, $force, $down, $fake);
        $response = "Migrations applied!\n";

        $e->setResult($response);
        return $response;
    }
}


1			<?php
2
3			namespace T4web\Migrations\Controller;
4
5			use Zend\Mvc\Controller\AbstractActionController;
6			use Zend\Console\Request as ConsoleRequest;
7			use Zend\Mvc\MvcEvent;
8			use T4web\Migrations\Service\Migration;
9			use T4web\Migrations\Exception\RuntimeException;
10
11			class ApplyController extends AbstractActionController
12			{
13			/**
14			* @var Migration
15			*/
16			protected $migration;
17
18			/**
19			* @param Migration $migration
20			*/
21			public function __construct(Migration $migration)
22			{
23			$this->migration = $migration;
24			}
25
26			public function onDispatch(MvcEvent $e)
27			{
28			if (!$e->getRequest() instanceof ConsoleRequest) {
29			throw new RuntimeException('You can only use this action from a console!');
30			}
31
32			$version = $e->getRequest()->getParam('version');
33			$force = $e->getRequest()->getParam('force');
34			$down = $e->getRequest()->getParam('down');
35			$fake = $e->getRequest()->getParam('fake');
36
37			if (is_null($version) && $force) {
38			$response = "Can't force migration apply without migration version explicitly set.\n";
39			$e->setResult($response);
40			return $response;
41			}
42
43			if (is_null($version) && $fake) {
44			$response = "Can't fake migration apply without migration version explicitly set.\n";
45			$e->setResult($response);
46			return $response;
47			}
48
49			$this->migration->migrate($version, $force, $down, $fake);
50			$response = "Migrations applied!\n";
51
52			$e->setResult($response);
53			return $response;
54			}
55			}
56

t4web / Migrations

Push — master ( bdc00f...dd9c2b )

ApplyController::__construct() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like