GBProd\ElasticsearchExtraBundle\Handler\PutIndexMappingsHandler

Complexity

Total Complexity

6

Size/Duplication

Total Lines	57
Duplicated Lines	100 %

Coupling/Cohesion

Components	1
Dependencies	4

Test Coverage

Coverage

94.12%

Importance

Changes	1
Bugs	0	Features	1

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	7	7	1
A	handle()	17	17	3
A	extractMapping()	8	8	2

<?php

namespace GBProd\ElasticsearchExtraBundle\Handler;

use GBProd\ElasticsearchExtraBundle\Repository\ClientRepository;
use GBProd\ElasticsearchExtraBundle\Repository\IndexConfigurationRepository;

/**
 * Handler to put index mappings command
 *
 * @author gbprod <contact@gb-prod.fr>
 */
class PutIndexMappingsHandler

This class seems to be duplicated in your project.

{
    /**
     * @var ClientRepository
     */
    private $clientRepository;

    /**
     * @var IndexConfigurationRepository
     */
    private $configurationRepository;

    /**
     * @param ClientRepository             $clientRepository
     * @param IndexConfigurationRepository $configurationRepository
     */
    public function __construct(
        ClientRepository $clientRepository,
        IndexConfigurationRepository $configurationRepository
    ) {
        $this->clientRepository        = $clientRepository;
        $this->configurationRepository = $configurationRepository;
    }

    /**
     * Handle index creation command
     *
     * @param string $clientId
     * @param string $indexId
     */
    public function handle($clientId, $indexId)
    {
        $client = $this->clientRepository->get($clientId);
        $config = $this->configurationRepository->get($clientId, $indexId);

        if (null === $client || null === $config) {
            throw new \InvalidArgumentException();
        }

        $client
            ->indices()
            ->putMapping([
                'index' => $indexId,
                'body'  => $this->extractMapping($config),
            ])
        ;
    }
    
    private function extractMapping($config)

The return type could not be reliably inferred; please add a @return annotation.

    {
        if (isset($config['mappings'])) {
            return $config['mappings'];
        }
        
        return [];
    }
}