BrainExe\Core\DependencyInjection\CompilerPass\RedisCompilerPass

Complexity

Total Complexity

3

Size/Duplication

Total Lines	34
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	2

Test Coverage

Coverage

75%

Importance

Changes	11
Bugs	0	Features	6

2 Methods

Rating	Name	Duplication	Size	Complexity
A	getArguments()	0	4	1
A	process()	0	17	2

<?php

namespace BrainExe\Core\DependencyInjection\CompilerPass;

use BrainExe\Core\Annotations\CompilerPass;
use Symfony\Component\DependencyInjection\Compiler\CompilerPassInterface;
use Symfony\Component\DependencyInjection\ContainerBuilder;
use Symfony\Component\DependencyInjection\Exception\ParameterNotFoundException;

/**
 * @CompilerPass(priority=9)
 */
class RedisCompilerPass implements CompilerPassInterface
{

    /**
     * {@inheritdoc}
     */
    public function process(ContainerBuilder $container)
    {
        $redis = $container->getDefinition('redis');

        $masterArguments = $this->getArguments($container, '');

        try {
            $slaveArguments = $this->getArguments($container, '.slave');
            $parameters = [$masterArguments, $slaveArguments];
            $options    = ['replication' => true];

            $redis->setArguments([$parameters, $options]);
        } catch (ParameterNotFoundException $e) {
            // master only
            $redis->setArguments([$masterArguments]);
        }
    }

    /**
     * @param ContainerBuilder $container
     * @param string $prefix
     * @return string
     */
    protected function getArguments(ContainerBuilder $container, string $prefix) : string
    {
        return $container->getParameter("redis$prefix.connection");

As per coding-style, please use concatenation or sprintf for the variable $prefix instead of interpolation.

    }
}