Spiral\Migrations\Operation\Index\Add

Complexity

Total Complexity

4

Size/Duplication

Total Lines	35
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	execute()	0	17	3
A	__construct()	0	4	1

<?php

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

declare(strict_types=1);

namespace Cycle\Migrations\Operation\Index;

use Cycle\Migrations\CapsuleInterface;
use Cycle\Migrations\Exception\Operation\IndexException;
use Cycle\Migrations\Operation\Traits\OptionsTrait;

final class Add extends Index
{
    use OptionsTrait;

The trait Cycle\Migrations\Operation\Traits\OptionsTrait requires the property $aliases which is not provided by Cycle\Migrations\Operation\Index\Add.

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

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

        if ($schema->hasIndex($this->columns)) {
            $columns = implode(',', $this->columns);
            throw new IndexException(
                "Unable to create index '{$schema->getName()}'.({$columns}), index already exists"
            );
        }

        $schema->index($this->columns)->unique(
            $this->getOption('unique', false)

It seems like $this->getOption('unique', false) can also be of type null; however, parameter $unique of Cycle\Database\Schema\AbstractIndex::unique() does only seem to accept boolean, maybe add an additional type check?

        );

        if ($this->hasOption('name')) {
            $schema->index($this->columns)->setName($this->getOption('name'));

It seems like $this->getOption('name') can also be of type false and null; however, parameter $name of Cycle\Database\Schema\AbstractIndex::setName() does only seem to accept string, maybe add an additional type check?

        }
    }
}