pngcrush.sh

[tt-rss.git] / include / errorhandler.php

diff --git a/include/errorhandler.php b/include/errorhandler.php
index f7fadc17236a01b187897c23f6d0fe8cb0f25928..95c9edc69df9ac7b8ad3f414741a844a88b3fe6c 100644 (file)
--- a/include/errorhandler.php
+++ b/include/errorhandler.php
@@ -1,26 +1,50 @@
 <?php
-// TODO: make configurable
-require_once "classes/logger.php";
-require_once "classes/logger/sql.php";
+function format_backtrace($trace) {
+       $rv = "";
+       $idx = 1;
 
-function ttrss_error_handler($errno, $errstr, $file, $line, $context) {
-       global $logger;
+       if (is_array($trace)) {
+               foreach ($trace as $e) {
+                       if (isset($e["file"]) && isset($e["line"])) {
+                               $fmt_args = [];
 
-       if (error_reporting() == 0) return false;
+                               if (is_array($e["args"])) {
+                                       foreach ($e["args"] as $a) {
+                                               if (!is_object($a)) {
+                                                       array_push($fmt_args, $a);
+                                               } else {
+                                                       array_push($fmt_args, "[" . get_class($a) . "]");
+                                               }
+                                       }
+                               }
 
-       if (!$logger) $logger = new Logger_SQL();
+                               $filename = str_replace(dirname(__DIR__) . "/", "", $e["file"]);
 
-       $file = substr(str_replace(dirname(dirname(__FILE__)), "", $file), 1);
+                               $rv .= sprintf("%d. %s(%s): %s(%s)\n",
+                                       $idx, $filename, $e["line"], $e["function"], implode(", ", $fmt_args));
 
-       if ($logger) {
-               return $logger->log_error($errno, $errstr, $file, $line, $context);
+                               $idx++;
+                       }
+               }
        }
 
-       return false;
+       return $rv;
+}
+
+function ttrss_error_handler($errno, $errstr, $file, $line, $context) {
+       if (error_reporting() == 0 || !$errno) return false;
+
+       $file = substr(str_replace(dirname(dirname(__FILE__)), "", $file), 1);
+
+       $context = format_backtrace(debug_backtrace());
+       $errstr = truncate_middle($errstr, 16384, " (...) ");
+
+       if (class_exists("Logger"))
+               return Logger::get()->log_error($errno, $errstr, $file, $line, $context);
 }
 
 function ttrss_fatal_handler() {
-       global $logger;
+       global $last_query;
 
        $error = error_get_last();
 
@@ -30,18 +54,21 @@ function ttrss_fatal_handler() {
                $line = $error["line"];
                $errstr  = $error["message"];
 
-               $context = debug_backtrace();
+               if (!$errno) return false;
+
+               $context = format_backtrace(debug_backtrace());
 
                $file = substr(str_replace(dirname(dirname(__FILE__)), "", $file), 1);
 
-               if (!$logger) $logger = new Logger_SQL();
+               if ($last_query) $errstr .= " [Last query: $last_query]";
 
-               if ($logger) {
-                       $logger->log_error($errno, $errstr, $file, $line, $context);
-               }
+               if (class_exists("Logger"))
+                       return Logger::get()->log_error($errno, $errstr, $file, $line, $context);
        }
+
+       return false;
 }
 
 register_shutdown_function('ttrss_fatal_handler');
 set_error_handler('ttrss_error_handler');
-?>
+