From 4cbee618d6f433d3830f80f219a007f76aa0fb1d Mon Sep 17 00:00:00 2001 From: Art4 Date: Tue, 28 Jan 2025 07:23:44 +0000 Subject: [PATCH] Refactor HookEventBridge --- src/EventSubscriber/HookEventBridge.php | 27 +++++++++---------------- 1 file changed, 9 insertions(+), 18 deletions(-) diff --git a/src/EventSubscriber/HookEventBridge.php b/src/EventSubscriber/HookEventBridge.php index 3c4e6adc28..a90c364103 100644 --- a/src/EventSubscriber/HookEventBridge.php +++ b/src/EventSubscriber/HookEventBridge.php @@ -58,33 +58,21 @@ final class HookEventBridge public static function onNamedEvent(NamedEvent $event): void { - $name = $event->getName(); - - $name = static::$eventMapper[$name] ?? $name; + static::callHook($event->getName(), ''); + } - static::callHook($name, ''); + public static function onConfigLoadedEvent(ConfigLoadedEvent $event): void + { + static::callHook($event->getName(), $event->getConfig()); } public static function onHtmlFilterEvent(HtmlFilterEvent $event): void { - $name = $event->getName(); - - $name = static::$eventMapper[$name] ?? $name; - $event->setHtml( - static::callHook($name, $event->getHtml()) + static::callHook($event->getName(), $event->getHtml()) ); } - public static function onConfigLoadedEvent(ConfigLoadedEvent $event): void - { - $name = $event->getName(); - - $name = static::$eventMapper[$name] ?? $name; - - static::callHook($name, $event->getConfig()); - } - /** * @param string|array $data * @@ -92,6 +80,9 @@ final class HookEventBridge */ private static function callHook(string $name, $data) { + // If possible, map the event name to the legacy Hook name + $name = static::$eventMapper[$name] ?? $name; + // Little hack to allow mocking the Hook call in tests. if (static::$mockedCallHook instanceof \Closure) { return (static::$mockedCallHook)->__invoke($name, $data); -- 2.39.5