b2pweb / bdf-prime

src/Schema/Bag/IndexSet.php

<?php

namespace Bdf\Prime\Schema\Bag;

use Bdf\Prime\Schema\IndexInterface;
use Bdf\Prime\Schema\IndexSetInterface;

/**
 * Index set class
 * An index set represents a set of index, with a primary key
 * The indexes names are case insensitive
 */
final class IndexSet implements IndexSetInterface
{
    /**
     * @var IndexInterface[]
     */
    private $indexes;


    /**
     * IndexSet constructor.
     *
     * @param IndexInterface[] $indexes
     */
    public function __construct(array $indexes)
    {
        $this->indexes = [];

        foreach ($indexes as $index) {
            $this->indexes[strtolower($index->name())] = $index;

It seems like $index->name() can also be of type null; however, parameter $string of strtolower() does only seem to accept string, maybe add an additional type check?

        }
    }

    /**
     * {@inheritdoc}
     */
    public function primary(): ?IndexInterface
    {
        foreach ($this->indexes as $index) {
            if ($index->primary()) {
                return $index;
            }
        }

        return null;
    }

    /**
     * {@inheritdoc}
     */
    public function all(): array
    {
        return $this->indexes;
    }

    /**
     * {@inheritdoc}
     */
    public function get(string $name): IndexInterface
    {
        return $this->indexes[strtolower($name)];
    }

    /**
     * {@inheritdoc}
     */
    public function has(string $name): bool
    {
        return isset($this->indexes[strtolower($name)]);
    }
}