Some CR applied:

- removed todo
- added explanation
- merged next if() block with already merged

Signed-off-by: Roland Häder <roland@mxchange.org>

diff --git a/include/dba_pdo.php b/include/dba_pdo.php
index 40ff09acf0761c8078766bcde7514cfdeedda340..16f00e88f5475bb940921dd00884c16cbbf4463f 100644 (file)
--- 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);