Cycle\Migrations\Operation\Index\Add::execute()

Complexity

Conditions	3
Paths	3

Size

Total Lines	17
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    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?

        }