8p / EightPointsGuzzleBundle

src/Twig/Extension/DebugExtension.php

<?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);
    }
}