AmaTeam\ElasticSearch\Mapping\TypeRegistry

Complexity

Total Complexity

9

Size/Duplication

Total Lines	60
Duplicated Lines	0 %

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	getInstance()	0	7	2
A	register()	0	5	1
A	findType()	0	9	3
A	withDefaultTypes()	0	16	3

<?php

declare(strict_types=1);

namespace AmaTeam\ElasticSearch\Mapping;

use Symfony\Component\Finder\Finder;
use Symfony\Component\Finder\SplFileInfo;

class TypeRegistry
{
    /**
     * @var TypeInterface
     */
    private $idIndex = [];
    /**
     * @var TypeInterface
     */
    private $friendlyIdIndex = [];

    /**
     * @param string $id
     * @return TypeInterface|null
     */
    public function findType(string $id): ?TypeInterface
    {
        if (isset($this->friendlyIdIndex[$id])) {
            return $this->friendlyIdIndex[$id];
        }
        if (isset($this->idIndex[$id])) {
            return $this->idIndex[$id];
        }
        return null;
    }

    public function register(TypeInterface $type): TypeRegistry
    {
        $this->idIndex[$type->getId()] = $type;
        $this->friendlyIdIndex[$type->getFriendlyId()] = $type;
        return $this;
    }

    public function withDefaultTypes(): TypeRegistry
    {
        // TODO: this method should not touch the filesystem
        $finder = (new Finder())
            ->in(sprintf('%s/%s', __DIR__, 'Type'))
            ->name('*.php')
            ->notName('Abstract*.php');
        /** @var SplFileInfo $file */
        foreach ($finder as $file) {
            $components = [__NAMESPACE__, 'Type', $file->getBasename('.php')];
            $name = implode('\\', $components);
            $accessible = method_exists($name, 'getInstance');
            $type = $accessible ? $name::getInstance() : new $name();
            $this->register($type);
        }
        return $this;
    }

    private static $instance;

    public static function getInstance(): TypeRegistry
    {
        if (!static::$instance) {

Since $instance is declared private, accessing it with static will lead to errors in possible sub-classes; you can either use self, or increase the visibility of $instance to at least protected.

            static::$instance = (new static())
                ->withDefaultTypes();
        }
        return static::$instance;
    }
}