Skip to content

Commit

Permalink
merged branch Seldaek/fix-deprecated-logs (PR #7587)
Browse files Browse the repository at this point in the history
This PR was merged into the 2.2 branch.

Discussion
----------

[2.2][HttpKernel] Remove args from 5.3 stack traces to avoid filling log files

Fixes #7259 - it just makes the PHP 5.3 behavior match the one on 5.4.

Commits
-------

99256e4 [HttpKernel] Remove args from 5.3 stack traces to avoid filling log files, fixes #7259
  • Loading branch information
fabpot committed Apr 7, 2013
2 parents 52389d6 + 474f1be commit 674d72b
Showing 1 changed file with 11 additions and 1 deletion.
12 changes: 11 additions & 1 deletion src/Symfony/Component/HttpKernel/Debug/ErrorHandler.php
Expand Up @@ -87,7 +87,17 @@ public function handle($level, $message, $file, $line, $context)

if ($level & (E_USER_DEPRECATED | E_DEPRECATED)) {
if (null !== self::$logger) {
$stack = version_compare(PHP_VERSION, '5.4', '<') ? array_slice(debug_backtrace(false), 0, 10) : debug_backtrace(DEBUG_BACKTRACE_IGNORE_ARGS, 10);
if (version_compare(PHP_VERSION, '5.4', '<')) {
$stack = array_map(
function ($row) {
unset($row['args']);
return $row;
},
array_slice(debug_backtrace(false), 0, 10)
);
} else {
$stack = debug_backtrace(DEBUG_BACKTRACE_IGNORE_ARGS, 10);
}

self::$logger->warning($message, array('type' => self::TYPE_DEPRECATION, 'stack' => $stack));
}
Expand Down

0 comments on commit 674d72b

Please sign in to comment.