JaySDe\HandlebarsBundle\Helper\EsiHelper

Complexity

Total Complexity

2

Size/Duplication

Total Lines	19
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

100%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	handle()	0	6	1

<?php
/**
 * @author @jenschude <jens.schulze@commercetools.de>
 */

namespace JaySDe\HandlebarsBundle\Helper;

use Symfony\Component\HttpKernel\Fragment\FragmentHandler;

class EsiHelper implements HelperInterface
{
    /**
     * @var FragmentHandler
     */
    private $handler;

    public function __construct(FragmentHandler $handler)
    {
        $this->handler = $handler;
    }

    public function handle($context, $options)
    {
        $handler = $this->handler;
        $result = new \LightnCandy\SafeString($handler->render($context, 'esi', $options));

It seems like $handler->render($context, 'esi', $options) targeting Symfony\Component\HttpKernel\Fragment\FragmentHandler::render() can also be of type false or null; however, LightnCandy\SafeString::__construct() does only seem to accept string, maybe add an additional type check?

        return $result;
    }
}