From e1b696929fe5f7a1f5f967c6833f7ca5bd4ecc7c Mon Sep 17 00:00:00 2001 From: Julius Härtl Date: Fri, 24 Jul 2020 07:53:48 +0200 Subject: Move NotFoundResponse to a proper TemplateResponse MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Signed-off-by: Julius Härtl --- .../Middleware/AdditionalScriptsMiddleware.php | 17 ++++++++--------- 1 file changed, 8 insertions(+), 9 deletions(-) (limited to 'lib/private/AppFramework/Middleware/AdditionalScriptsMiddleware.php') diff --git a/lib/private/AppFramework/Middleware/AdditionalScriptsMiddleware.php b/lib/private/AppFramework/Middleware/AdditionalScriptsMiddleware.php index 619432be78f..9eafd5b740c 100644 --- a/lib/private/AppFramework/Middleware/AdditionalScriptsMiddleware.php +++ b/lib/private/AppFramework/Middleware/AdditionalScriptsMiddleware.php @@ -53,16 +53,15 @@ class AdditionalScriptsMiddleware extends Middleware { } public function afterController($controller, $methodName, Response $response): Response { - /* - * There is no need to emit these signals on a public share page - * There is a separate event for that already - */ - if ($controller instanceof PublicShareController) { - return $response; - } - if ($response instanceof TemplateResponse) { - $this->legacyDispatcher->dispatch(TemplateResponse::EVENT_LOAD_ADDITIONAL_SCRIPTS, new GenericEvent()); + if (!$controller instanceof PublicShareController) { + /* + * The old event was not dispatched on the public share controller as there was + * OCA\Files_Sharing::loadAdditionalScripts for that. This is kept for compatibility reasons + * only for the old event as this is now also included in BeforeTemplateRenderedEvent + */ + $this->legacyDispatcher->dispatch(TemplateResponse::EVENT_LOAD_ADDITIONAL_SCRIPTS, new GenericEvent()); + } if (!($response instanceof StandaloneTemplateResponse) && $this->userSession->isLoggedIn()) { $this->legacyDispatcher->dispatch(TemplateResponse::EVENT_LOAD_ADDITIONAL_SCRIPTS_LOGGEDIN, new GenericEvent()); -- cgit v1.2.3