Merge pull request #21299 from nextcloud/fix/generic-event-wrapper-constructor-get-class

Fix missing parent constructor call and get_class usage in GenericEventWrapper

1 files changed, 5 insertions, 3 deletions

diff --git a/lib/private/EventDispatcher/GenericEventWrapper.php b/lib/private/EventDispatcher/GenericEventWrapper.php
index d1d136affc0..dc3e7553f0c 100644
--- a/lib/private/EventDispatcher/GenericEventWrapper.php
+++ b/lib/private/EventDispatcher/GenericEventWrapper.php
@@ -40,24 +40,26 @@ class GenericEventWrapper extends GenericEvent {
 	private $eventName;
 
 	public function __construct(ILogger $logger, string $eventName, ?GenericEvent $event) {
+		parent::__construct($eventName);
 		$this->logger = $logger;
 		$this->event = $event;
 		$this->eventName = $eventName;
 	}
 
 	private function log() {
+		$class = ($this->event !== null && is_object($this->event)) ? get_class($this->event) : 'null';
 		$this->logger->info(
 			'Deprecated event type for {name}: {class} is used',
-			[ 'name' => $this->eventName, 'class' => is_object($this->event) ? get_class($this->event) : 'null' ]
+			[ 'name' => $this->eventName, 'class' => $class]
 		);
 	}
 
-	public function isPropagationStopped() {
+	public function isPropagationStopped(): bool {
 		$this->log();
 		return $this->event->isPropagationStopped();
 	}
 
-	public function stopPropagation() {
+	public function stopPropagation(): void {
 		$this->log();
 		$this->event->stopPropagation();
 	}