EightPoints\Bundle\GuzzleBundle\Twig\Extension\DebugExtension

Complexity

Total Complexity

3

Size/Duplication

Total Lines	47
Duplicated Lines	0 %

Importance

Changes	3
Bugs	0	Features	0

3 Methods

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

<?php

namespace EightPoints\Bundle\GuzzleBundle\Twig\Extension;

use Symfony\Component\VarDumper\Cloner\VarCloner;
use Symfony\Component\VarDumper\Dumper\HtmlDumper;
use Twig\Environment;
use Twig\Extension\AbstractExtension;
use Twig\TwigFunction;

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

    /**
     * @param Environment $env
     * @param $value
     *
     * @throws \Exception
     *
     * @return bool|string
     */
    public function dump(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?

    }

    /**
     * This method is removed from interface in Twig v2.0
     *
     * @TODO Remove this method when drop support of Symfony < 5.0
     *
     * @return string
     */
    public function getName() : string
    {
        return get_class($this);
    }
}