EightPoints\Bundle\GuzzleBundle\Twig\Extension\DebugExtension

Complexity

Total Complexity

2

Size/Duplication

Total Lines	35
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	getFunctions()	0	7	1
A	dump()	0	11	1

<?php

namespace EightPoints\Bundle\GuzzleBundle\Twig\Extension;

use Symfony\Component\VarDumper\Cloner\VarCloner;
use Symfony\Component\VarDumper\Dumper\HtmlDumper;

class DebugExtension extends \Twig_Extension
{
    /**
     * @return array
     */
    public function getFunctions() : array
    {
        return [
            new \Twig_SimpleFunction(
                'eight_points_guzzle_dump',
                [$this, 'dump'],
                ['is_safe' => ['html'], 'needs_environment' => true]
            ),
        ];
    }

    /**
     * @param \Twig_Environment $env
     * @param $value
     *
     * @throws \Exception
     *
     * @return bool|string
     */
    public function dump(\Twig_Environment $env, $value)
    {
        $cloner = new VarCloner();

        $dump = fopen('php://memory', 'r+b');
        $dumper = new HtmlDumper($dump, $env->getCharset());

        $dumper->dump($cloner->cloneVar($value));
        rewind($dump);

It seems like $dump can also be of type false; however, parameter $handle of rewind() does only seem to accept resource, maybe add an additional type check?

        return stream_get_contents($dump);

It seems like $dump can also be of type false; however, parameter $handle of stream_get_contents() does only seem to accept resource, maybe add an additional type check?

    }
}