summaryrefslogtreecommitdiff
path: root/include
diff options
context:
space:
mode:
authorAndrew Dolgov <[email protected]>2021-02-25 17:10:03 +0300
committerAndrew Dolgov <[email protected]>2021-02-25 17:10:03 +0300
commit34c74400a471b31fea22ea85636f47c76d783710 (patch)
tree460528b4f177a1500b61035ed39644747cb47d66 /include
parentdcf0135285f1a515454807cdfe1e819f37a23a86 (diff)
enforce some stricter type checking for loggers
Diffstat (limited to 'include')
-rw-r--r--include/errorhandler.php12
1 files changed, 4 insertions, 8 deletions
diff --git a/include/errorhandler.php b/include/errorhandler.php
index a59d7a616..1908bd39c 100644
--- a/include/errorhandler.php
+++ b/include/errorhandler.php
@@ -40,13 +40,13 @@ function format_backtrace($trace) {
}
function ttrss_error_handler($errno, $errstr, $file, $line) {
- if (version_compare(PHP_VERSION, '8.0.0', '<')) {
+ /*if (version_compare(PHP_VERSION, '8.0.0', '<')) {
if (error_reporting() == 0 || !$errno) return false;
} else {
if (!(error_reporting() & $errno)) return false;
}
- if (error_reporting() == 0 || !$errno) return false;
+ if (error_reporting() == 0 || !$errno) return false;*/
$file = substr(str_replace(dirname(__DIR__), "", $file), 1);
@@ -54,12 +54,10 @@ function ttrss_error_handler($errno, $errstr, $file, $line) {
$errstr = truncate_middle($errstr, 16384, " (...) ");
if (class_exists("Logger"))
- return Logger::log_error($errno, $errstr, $file, $line, $context);
+ return Logger::log_error((int)$errno, $errstr, $file, (int)$line, $context);
}
function ttrss_fatal_handler() {
- global $last_query;
-
$error = error_get_last();
if ($error !== NULL) {
@@ -74,10 +72,8 @@ function ttrss_fatal_handler() {
$file = substr(str_replace(dirname(__DIR__), "", $file), 1);
- if ($last_query) $errstr .= " [Last query: $last_query]";
-
if (class_exists("Logger"))
- return Logger::log_error($errno, $errstr, $file, $line, $context);
+ return Logger::log_error((int)$errno, $errstr, $file, (int)$line, $context);
}
return false;