only log diagnostic events if a treshhold is set

this prevents log spam and it's rare that you actually want to very short events logged anyway

Signed-off-by: Robin Appelman <robin@icewind.nl>

Author: icewind1991

config/config.sample.php

@@ -1791,7 +1791,7 @@
 
 /**
  * Enforce the user theme. This will disable the user theming settings
- * This must be a valid ITheme ID. 
+ * This must be a valid ITheme ID.
  * E.g. light, dark, highcontrast, dark-highcontrast...
  */
 'enforce_theme' => '',
@@ -2146,6 +2146,8 @@
 
 /**
  * Limit diagnostics event logging to events longer than the configured threshold in ms
+ *
+ * when set to 0 no diagnostics events will be logged
  */
 'diagnostics.logging.threshold' => 0,
 

lib/private/Diagnostics/EventLogger.php

@@ -126,7 +126,7 @@ private function writeLog(IEvent $event) {
 			$timeInMs = round($duration * 1000, 4);
 
 			$loggingMinimum = (int)$this->config->getValue('diagnostics.logging.threshold', 0);
-			if ($loggingMinimum > 0 && $timeInMs < $loggingMinimum) {
+			if ($loggingMinimum == 0 || $timeInMs < $loggingMinimum) {
 				return;
 			}