Dump function requires html_errors to be on but doesn't allow for unset

[zorfling]

The Dump function requires html_errors to be on, otherwise it double encodes xdebug_var_dump.

There seems to be an error (or an omission really) in Twig_Extension_Debug.

Line 21 of twig/lib/Twig/Extension/Debug.php currently reads:

$isDumpOutputHtmlSafe = extension_loaded('xdebug') && (false === get_cfg_var('xdebug.overload_var_dump') || get_cfg_var('xdebug.overload_var_dump')) && get_cfg_var('html_errors');

Perhaps it should read:

$isDumpOutputHtmlSafe = extension_loaded('xdebug') && (false === get_cfg_var('xdebug.overload_var_dump') || get_cfg_var('xdebug.overload_var_dump')) && (false === get_cfg_var('html_errors') || get_cfg_var('html_errors'));

I've simply added the same unset clause as used in the overload_var_dump check.