AmaTeam\ElasticSearch\Entity\Mapping\EntityProviderWrapperMappingProvider

Complexity

Total Complexity

5

Size/Duplication

Total Lines	29
Duplicated Lines	0 %

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
A	exists()	0	3	1
A	get()	0	3	1
A	find()	0	4	2

<?php

declare(strict_types=1);

namespace AmaTeam\ElasticSearch\Entity\Mapping;

use AmaTeam\ElasticSearch\API\Entity\Mapping\ClassMappingInterface;
use AmaTeam\ElasticSearch\API\Entity\Mapping\ProviderInterface;
use AmaTeam\ElasticSearch\API\Entity\ProviderInterface as EntityProviderInterface;

class EntityProviderWrapperMappingProvider implements ProviderInterface
{
    /**
     * @var EntityProviderInterface
     */
    private $provider;

    /**
     * @param EntityProviderInterface $provider
     */
    public function __construct(EntityProviderInterface $provider)
    {
        $this->provider = $provider;
    }

    public function exists(string $name): bool
    {
        return $this->provider->exists($name);
    }

    public function get(string $name): ClassMappingInterface
    {
        return $this->provider->getResolved($name)->getMapping();

The expression return $this->provider->getResolved($name)->getMapping() could return the type null which is incompatible with the type-hinted return AmaTeam\ElasticSearch\API\Entity\Mapping\ClassMappingInterface. Consider adding an additional type-check to rule them out.

    }

    public function find(string $name): ?ClassMappingInterface
    {
        $entity = $this->provider->findResolved($name);
        return $entity ? $entity->getMapping() : null;
    }
}