From 4c2d4934f78542b42feb17d31e52327e1d25f9f3 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Roland=20H=C3=A4der?= Date: Wed, 17 May 2017 18:05:18 +0200 Subject: [PATCH] Some CR applied: - removed todo - added explanation - merged next if() block with already merged MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Signed-off-by: Roland Häder --- include/dba_pdo.php | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/include/dba_pdo.php b/include/dba_pdo.php index 40ff09acf..16f00e88f 100644 --- a/include/dba_pdo.php +++ b/include/dba_pdo.php @@ -145,7 +145,10 @@ class dba { $a->save_timestamp($stamp1, "database"); - /// @TODO really check $a->config for 'system'? it is very generic and should be there + /* + * Check if the configuration group 'system' and db_log is there. The + * extra first check needs to be done to avoid endless loop. + */ if (x($a->config, 'system') && x($a->config['system'], 'db_log') && ($duration > $a->config["system"]["db_loglimit"])) { $duration = round($duration, 3); $backtrace = debug_backtrace(DEBUG_BACKTRACE_IGNORE_ARGS);