From f93a82b8b09109e391b0314f92d02d285c356ad6 Mon Sep 17 00:00:00 2001 From: Lukas Reschke Date: Tue, 1 Aug 2017 17:32:03 +0200 Subject: Remove explicit type hints for Controller This is public API and breaks the middlewares of existing apps. Since this also requires maintaining two different code paths for 12 and 13 I'm at the moment voting for reverting this change. Signed-off-by: Lukas Reschke --- core/Middleware/TwoFactorMiddleware.php | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'core') diff --git a/core/Middleware/TwoFactorMiddleware.php b/core/Middleware/TwoFactorMiddleware.php index b8edda4db63..a4e0d7219ec 100644 --- a/core/Middleware/TwoFactorMiddleware.php +++ b/core/Middleware/TwoFactorMiddleware.php @@ -79,7 +79,7 @@ class TwoFactorMiddleware extends Middleware { * @param Controller $controller * @param string $methodName */ - public function beforeController(Controller $controller, $methodName) { + public function beforeController($controller, $methodName) { if ($this->reflector->hasAnnotation('PublicPage')) { // Don't block public pages return; @@ -122,7 +122,7 @@ class TwoFactorMiddleware extends Middleware { } } - public function afterException(Controller $controller, $methodName, Exception $exception) { + public function afterException($controller, $methodName, Exception $exception) { if ($exception instanceof TwoFactorAuthRequiredException) { $params = []; if (isset($this->request->server['REQUEST_URI'])) { -- cgit v1.2.3