Completed
Push — master ( e221c0...5a5b2e )
by Matze
05:22
created

RedisCompilerPass::getArguments()   A

Complexity

Conditions 1
Paths 1

Size

Total Lines 4
Code Lines 2

Duplication

Lines 0
Ratio 0 %

Code Coverage

Tests 2
CRAP Score 1

Importance

Changes 2
Bugs 0 Features 1
Metric Value
c 2
b 0
f 1
dl 0
loc 4
ccs 2
cts 2
cp 1
rs 10
cc 1
eloc 2
nc 1
nop 2
crap 1
1
<?php
2
3
namespace BrainExe\Core\DependencyInjection\CompilerPass;
4
5
use BrainExe\Core\Annotations\CompilerPass;
6
use Symfony\Component\DependencyInjection\Compiler\CompilerPassInterface;
7
use Symfony\Component\DependencyInjection\ContainerBuilder;
8
use Symfony\Component\DependencyInjection\Exception\ParameterNotFoundException;
9
10
/**
11
 * @CompilerPass(priority=9)
12
 */
13
class RedisCompilerPass implements CompilerPassInterface
14
{
15
16
    /**
17
     * {@inheritdoc}
18
     */
19 3
    public function process(ContainerBuilder $container)
20
    {
21 3
        $redis = $container->getDefinition('redis');
22
23 3
        $masterArguments = $this->getArguments($container, '');
24
25
        try {
26 3
            $slaveArguments = $this->getArguments($container, '.slave');
27
            $parameters = [$masterArguments, $slaveArguments];
28
            $options    = ['replication' => true];
29
30
            $redis->setArguments([$parameters, $options]);
31 3
        } catch (ParameterNotFoundException $e) {
32
            // master only
33 3
            $redis->setArguments([$masterArguments]);
34
        }
35 3
    }
36
37
    /**
38
     * @param ContainerBuilder $container
39
     * @param string $prefix
40
     * @return string
41
     */
42 3
    protected function getArguments(ContainerBuilder $container, string $prefix) : string
43
    {
44 3
        return $container->getParameter("redis$prefix.connection");
0 ignored issues
show
Coding Style Best Practice introduced by
As per coding-style, please use concatenation or sprintf for the variable $prefix instead of interpolation.

It is generally a best practice as it is often more readable to use concatenation instead of interpolation for variables inside strings.

// Instead of
$x = "foo $bar $baz";

// Better use either
$x = "foo " . $bar . " " . $baz;
$x = sprintf("foo %s %s", $bar, $baz);
Loading history...
45
    }
46
}
47