diff options
| author | Lukas Reschke <lukas@statuscode.ch> | 2017-08-01 17:32:03 +0200 |
|---|---|---|
| committer | Lukas Reschke <lukas@statuscode.ch> | 2017-08-01 17:32:03 +0200 |
| commit | f93a82b8b09109e391b0314f92d02d285c356ad6 (patch) | |
| tree | c85246eadd2103200e26af6b208756dc87822e76 /core/Middleware | |
| parent | 84c22fdeef6986f9038d8563937cc234751d5147 (diff) | |
| download | nextcloud-server-f93a82b8b09109e391b0314f92d02d285c356ad6.tar.gz nextcloud-server-f93a82b8b09109e391b0314f92d02d285c356ad6.zip | |
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 <lukas@statuscode.ch>
Diffstat (limited to 'core/Middleware')
| -rw-r--r-- | core/Middleware/TwoFactorMiddleware.php | 4 |
1 files changed, 2 insertions, 2 deletions
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'])) { |