LaravelFreelancerNL\Aranguent\Schema\Concerns\HandlesIndexes

Complexity

Total Complexity

5

Size/Duplication

Total Lines	43
Duplicated Lines	0 %

Test Coverage

Coverage

83.33%

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	hasIndex()	0	13	3
A	getIndex()	0	3	1
A	getIndexes()	0	4	1

<?php

declare(strict_types=1);

namespace LaravelFreelancerNL\Aranguent\Schema\Concerns;

use ArangoClient\Exceptions\ArangoException;

trait HandlesIndexes
{
    /**
     * Determine if the given table has a given index.
     *
     * @param  string  $table
     * @param  string|array<string>  $index
     * @param  string|null  $type
     * @return bool
     */
    public function hasIndex($table, $index, $type = null, array $options = [])
    {
        $name = $index;

        if ($type === null) {
            $type = 'persistent';
        }

        if (is_array($index)) {
            $name = $this->createIndexName($type, $index, $options, $table);

It seems like createIndexName() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        }

        return !!$this->schemaManager->getIndexByName($table, $name);
    }

    /**
     * @param string $id
     * @return array<mixed>
     */
    public function getIndex(string $id)
    {
        return (array) $this->schemaManager->getIndex($id);
    }

    /**
     * @param string $table
     * @return mixed[]
     * @throws ArangoException
     */
    public function getIndexes($table)
    {
        return $this->mapResultsToArray(

It seems like mapResultsToArray() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            $this->schemaManager->getIndexes($table),
        );
    }
}