[FrameworkBundle] Parse source link maps using json_decode() instead of parse_str()
This commit is contained in:
parent
059b4363d4
commit
9b174fb0c1
@ -34,9 +34,9 @@ class CodeExtension extends \Twig_Extension
|
||||
$fileLinkFormat = $fileLinkFormat ?: ini_get('xdebug.file_link_format') ?: get_cfg_var('xdebug.file_link_format');
|
||||
if ($fileLinkFormat && !is_array($fileLinkFormat)) {
|
||||
$i = max(strpos($fileLinkFormat, '%f'), strpos($fileLinkFormat, '%l'));
|
||||
$i = strpos($fileLinkFormat, '#', $i) ?: strlen($fileLinkFormat);
|
||||
$i = strpos($fileLinkFormat, '#"', $i) ?: strlen($fileLinkFormat);
|
||||
$fileLinkFormat = array(substr($fileLinkFormat, 0, $i), substr($fileLinkFormat, $i + 1));
|
||||
parse_str($fileLinkFormat[1], $fileLinkFormat[1]);
|
||||
$fileLinkFormat[1] = @json_decode('{'.$fileLinkFormat[1].'}', true) ?: array();
|
||||
}
|
||||
$this->fileLinkFormat = $fileLinkFormat;
|
||||
$this->rootDir = str_replace('/', DIRECTORY_SEPARATOR, dirname($rootDir)).DIRECTORY_SEPARATOR;
|
||||
|
@ -64,6 +64,6 @@ class CodeExtensionTest extends \PHPUnit_Framework_TestCase
|
||||
|
||||
protected function getExtension()
|
||||
{
|
||||
return new CodeExtension('proto://%f#&line=%l#'.substr(__FILE__, 0, 5).'=foobar', '/root', 'UTF-8');
|
||||
return new CodeExtension('proto://%f#&line=%l#'.json_encode(substr(__FILE__, 0, 5)).':"foobar"', '/root', 'UTF-8');
|
||||
}
|
||||
}
|
||||
|
@ -36,9 +36,9 @@ class CodeHelper extends Helper
|
||||
$fileLinkFormat = $fileLinkFormat ?: ini_get('xdebug.file_link_format') ?: get_cfg_var('xdebug.file_link_format');
|
||||
if ($fileLinkFormat && !is_array($fileLinkFormat)) {
|
||||
$i = max(strpos($fileLinkFormat, '%f'), strpos($fileLinkFormat, '%l'));
|
||||
$i = strpos($fileLinkFormat, '#', $i) ?: strlen($fileLinkFormat);
|
||||
$i = strpos($fileLinkFormat, '#"', $i) ?: strlen($fileLinkFormat);
|
||||
$fileLinkFormat = array(substr($fileLinkFormat, 0, $i), substr($fileLinkFormat, $i + 1));
|
||||
parse_str($fileLinkFormat[1], $fileLinkFormat[1]);
|
||||
$fileLinkFormat[1] = @json_decode('{'.$fileLinkFormat[1].'}', true) ?: array();
|
||||
}
|
||||
$this->fileLinkFormat = $fileLinkFormat;
|
||||
$this->rootDir = str_replace('\\', '/', $rootDir).'/';
|
||||
|
@ -40,9 +40,9 @@ class ExceptionHandler
|
||||
$fileLinkFormat = $fileLinkFormat ?: ini_get('xdebug.file_link_format') ?: get_cfg_var('xdebug.file_link_format');
|
||||
if ($fileLinkFormat && !is_array($fileLinkFormat)) {
|
||||
$i = max(strpos($fileLinkFormat, '%f'), strpos($fileLinkFormat, '%l'));
|
||||
$i = strpos($fileLinkFormat, '#', $i) ?: strlen($fileLinkFormat);
|
||||
$i = strpos($fileLinkFormat, '#"', $i) ?: strlen($fileLinkFormat);
|
||||
$fileLinkFormat = array(substr($fileLinkFormat, 0, $i), substr($fileLinkFormat, $i + 1));
|
||||
parse_str($fileLinkFormat[1], $fileLinkFormat[1]);
|
||||
$fileLinkFormat[1] = @json_decode('{'.$fileLinkFormat[1].'}', true) ?: array();
|
||||
}
|
||||
$this->debug = $debug;
|
||||
$this->charset = $charset ?: ini_get('default_charset') ?: 'UTF-8';
|
||||
@ -97,9 +97,9 @@ class ExceptionHandler
|
||||
{
|
||||
$old = $this->fileLinkFormat;
|
||||
if ($fileLinkFormat && !is_array($fileLinkFormat)) {
|
||||
$i = strpos($fileLinkFormat, '#') ?: strlen($fileLinkFormat);
|
||||
$i = strpos($fileLinkFormat, '#"') ?: strlen($fileLinkFormat);
|
||||
$fileLinkFormat = array(substr($fileLinkFormat, 0, $i), substr($fileLinkFormat, $i + 1));
|
||||
parse_str($fileLinkFormat[1], $fileLinkFormat[1]);
|
||||
$fileLinkFormat[1] = @json_decode('{'.$fileLinkFormat[1].'}', true) ?: array();
|
||||
}
|
||||
$this->fileLinkFormat = $fileLinkFormat;
|
||||
if ($old) {
|
||||
|
@ -43,9 +43,9 @@ class DumpDataCollector extends DataCollector implements DataDumperInterface
|
||||
$fileLinkFormat = $fileLinkFormat ?: ini_get('xdebug.file_link_format') ?: get_cfg_var('xdebug.file_link_format');
|
||||
if ($fileLinkFormat && !is_array($fileLinkFormat)) {
|
||||
$i = max(strpos($fileLinkFormat, '%f'), strpos($fileLinkFormat, '%l'));
|
||||
$i = strpos($fileLinkFormat, '#', $i) ?: strlen($fileLinkFormat);
|
||||
$i = strpos($fileLinkFormat, '#"', $i) ?: strlen($fileLinkFormat);
|
||||
$fileLinkFormat = array(substr($fileLinkFormat, 0, $i), substr($fileLinkFormat, $i + 1));
|
||||
parse_str($fileLinkFormat[1], $fileLinkFormat[1]);
|
||||
$fileLinkFormat[1] = @json_decode('{'.$fileLinkFormat[1].'}', true) ?: array();
|
||||
}
|
||||
$this->stopwatch = $stopwatch;
|
||||
$this->fileLinkFormat = $fileLinkFormat;
|
||||
|
Reference in New Issue
Block a user