PrimaryKeys::execute() - Code Metrics - Inspection of "Apply fixes from StyleCI" - cycle/migrations - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 10a192...087136 )

by Aleksei

created 2021-09-08 18:03 UTC

PrimaryKeys::execute() A

↳ Parent: PrimaryKeys

Complexity

Conditions	2
Paths	2

Size

Total Lines	12
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Importance

Changes

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

<?php

/**
 * Spiral Framework.
 *
 * @license MIT
 * @author  Anton Titov (Wolfy-J)
 */

declare(strict_types=1);

namespace Cycle\Migrations\Operation\Table;

use Cycle\Migrations\CapsuleInterface;
use Cycle\Migrations\Exception\Operation\TableException;
use Cycle\Migrations\Operation\AbstractOperation;

final class PrimaryKeys extends AbstractOperation
{
    /** @var array */
    private $columns = [];

    /**
     * @param string $table
     * @param array  $columns
     */
    public function __construct(string $table, array $columns)
    {
        parent::__construct($table);
        $this->columns = $columns;
    }

    /**
     * {@inheritdoc}
     */
    public function execute(CapsuleInterface $capsule): void
    {
        $schema = $capsule->getSchema($this->getTable());
        $database = $this->database ?? '[default]';


        if ($schema->exists()) {
            throw new TableException(
                "Unable to set primary keys for table '{$database}'.'{$this->getTable()}', table already exists"
            );
        }

        $schema->setPrimaryKeys($this->columns);
    }
}


1			<?php
2
3			/**
4			* Spiral Framework.
5			*
6			* @license MIT
7			* @author Anton Titov (Wolfy-J)
8			*/
9
10			declare(strict_types=1);
11
12			namespace Cycle\Migrations\Operation\Table;
13
14			use Cycle\Migrations\CapsuleInterface;
15			use Cycle\Migrations\Exception\Operation\TableException;
16			use Cycle\Migrations\Operation\AbstractOperation;
17
18			final class PrimaryKeys extends AbstractOperation
19			{
20			/** @var array */
21			private $columns = [];
22
23			/**
24			* @param string $table
25			* @param array $columns
26			*/
27			public function __construct(string $table, array $columns)
28			{
29			parent::__construct($table);
30			$this->columns = $columns;
31			}
32
33			/**
34			* {@inheritdoc}
35			*/
36			public function execute(CapsuleInterface $capsule): void
37			{
38			$schema = $capsule->getSchema($this->getTable());
39			$database = $this->database ?? '[default]';
			0 ignored issues – show Bug Best Practice introduced 2021-09-08 18:11 UTC by Report Bug Copy Issue Report The property `database` does not exist on `Cycle\Migrations\Operation\Table\PrimaryKeys`. Did you maybe forget to declare it? Loading history...
40
41			if ($schema->exists()) {
42			throw new TableException(
43			"Unable to set primary keys for table '{$database}'.'{$this->getTable()}', table already exists"
44			);
45			}
46
47			$schema->setPrimaryKeys($this->columns);
48			}
49			}
50

cycle / migrations

Push — master ( 10a192...087136 )

PrimaryKeys::execute() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like