fix: Do not register listener when not in console

Registering the listener no matter what, defits the perf optimisation done here: https://github.com/nextcloud/server/blob/cb7c82c13d5d6cf0b21b4499365460c605936c5a/lib/private/Log.php#L331-L337

Signed-off-by: Louis Chemineau <[email protected]>

Author: artonge

lib/AppInfo/Application.php

@@ -24,7 +24,9 @@ public function __construct(array $urlParams = []) {
 	}
 
 	public function register(IRegistrationContext $context): void {
-		$context->registerEventListener(BeforeMessageLoggedEvent::class, LogListener::class);
+		if (defined('OC_CONSOLE') && \OC_CONSOLE) {
+			$context->registerEventListener(BeforeMessageLoggedEvent::class, LogListener::class);
+		}
 		$context->registerService(Formatter::class, function (ContainerInterface $c) {
 			return new Formatter(\OC::$SERVERROOT);
 		});

lib/Listener/LogListener.php

@@ -23,14 +23,10 @@ class LogListener implements IEventListener {
 	private ?Console $console;
 
 	public function __construct(Formatter $formatter, SystemConfig $config) {
-		if (defined('OC_CONSOLE') && \OC_CONSOLE) {
-			$level = getenv('OCC_LOG');
-			if ($level) {
-				$terminal = new Terminal();
-				$this->console = new Console($formatter, $config, $level, $terminal->getWidth());
-			} else {
-				$this->console = null;
-			}
+		$level = getenv('OCC_LOG');
+		if ($level) {
+			$terminal = new Terminal();
+			$this->console = new Console($formatter, $config, $level, $terminal->getWidth());
 		} else {
 			$this->console = null;
 		}