Polidog\SsrBundle\Render\SsrRender

Complexity

Total Complexity

5

Size/Duplication

Total Lines	45
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

88.24%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	render()	0	8	1
A	filter()	0	15	3

<?php
/**
 * Created by PhpStorm.
 * User: polidog
 * Date: 2017/02/22.
 */

namespace Polidog\SsrBundle\Render;

use Koriym\Baracoa\BaracoaInterface;
use Polidog\SsrBundle\Annotations\Ssr;

class SsrRender implements SsrRenderInterface
{
    /**
     * @var BaracoaInterface
     */
    private $baracoa;

    /**
     * SsrRender constructor.
     *
     * @param BaracoaInterface $baracoa
     */
    public function __construct(BaracoaInterface $baracoa)
    {
        $this->baracoa = $baracoa;
    }

    /**
     * {@inheritdoc}
     */
    public function render(Ssr $ssr, array $parameters)
    {
        return $this->baracoa->render(
            $ssr->getApp(),
            $this->filter($ssr->getState(), $parameters),
            $this->filter($ssr->getMetas(), $parameters)
        );
    }

    private function filter(array $keys, array $parameters)
    {
        if ($keys === ['*']) {
            return $parameters;
        }

        $errorKeys = array_keys(array_diff(array_values($keys), array_keys($parameters)));
        if ($errorKeys) {

The expression $errorKeys of type integer[] is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

            throw new \LogicException(implode(',', $errorKeys));
        }

        return array_filter((array) $parameters, function($key) use ($keys) {
            return in_array($key, $keys);
        }, ARRAY_FILTER_USE_KEY);
    }
}

As per coding style, files should not end with a newline character.