Good point; I'd even use !empty($caller['file']) && !empty($caller['line']) to save a few cycles in the (improbable) case where both _may_ be set, but empty (thus making the whole exercise pointless...).

Author: Marius Burkard

server/lib/app.inc.php

@@ -251,7 +251,9 @@ function log($msg, $priority = 0, $dblog = true) {
 				$priority_txt = 'DEBUG';
 				$bt = debug_backtrace(DEBUG_BACKTRACE_IGNORE_ARGS, 1);	// we don't need _all_ data, so we save some processing time here (gwyneth 20220315)
 				$caller = array_shift($bt);
-				$file_line_caller = '[' . strtr(basename($caller['file'], '.php'), '_', ' ') . ':' . $caller['line'] . '] ';
+				if(isset($caller['file']) && isset($caller['line'])) {
+					$file_line_caller = '[' . strtr(basename($caller['file'], '.php'), '_', ' ') . ':' . $caller['line'] . '] ';
+				}
 				break;
 		case 1:
 				$priority_txt = 'WARNING';