diff options
| author | 2023-05-15 19:26:48 +0200 | |
|---|---|---|
| committer | 2023-05-15 19:26:48 +0200 | |
| commit | 2038d50110468d95ff978ba2e8f997175f25ff3b (patch) | |
| tree | a9bc4bc364fe0c9b452702f3324a2091229e5fce /app/Controllers/userController.php | |
| parent | c8d2ead7635e58a5c78d95a9b9a1a74e99a1bcef (diff) | |
PHPStan Level 7 for Minz_Request, FreshRSS_Feed, Minz_Error (#5400)
* PHPStan Level 7 for Minz_Request
* PHPStan Level 7 for FreshRSS_Feed
* PHPStan Level 7 for Minz_Error
Diffstat (limited to 'app/Controllers/userController.php')
| -rw-r--r-- | app/Controllers/userController.php | 11 |
1 files changed, 2 insertions, 9 deletions
diff --git a/app/Controllers/userController.php b/app/Controllers/userController.php index f929fc779..9906ae6bd 100644 --- a/app/Controllers/userController.php +++ b/app/Controllers/userController.php @@ -278,7 +278,6 @@ class FreshRSS_user_Controller extends FreshRSS_ActionController { * - r (i.e. a redirection url, optional) * * @todo clean up this method. Idea: write a method to init a user with basic information. - * @todo handle r redirection in Minz_Request::forward directly? */ public function createAction(): void { if (!FreshRSS_Auth::hasAccess('admin') && max_registrations_reached()) { @@ -372,10 +371,7 @@ class FreshRSS_user_Controller extends FreshRSS_ActionController { } } - $redirect_url = urldecode(Minz_Request::paramString('r', true)); - if ($redirect_url === '') { - $redirect_url = ['c' => 'user', 'a' => 'manage']; - } + $redirect_url = ['c' => 'user', 'a' => 'manage']; Minz_Request::forward($redirect_url, true); } @@ -534,10 +530,7 @@ class FreshRSS_user_Controller extends FreshRSS_ActionController { Minz_Error::error(403); } - $redirect_url = urldecode(Minz_Request::paramString('r', true)); - if ($redirect_url === '') { - $redirect_url = ['c' => 'user', 'a' => 'manage']; - } + $redirect_url = ['c' => 'user', 'a' => 'manage']; if (Minz_Request::isPost()) { $ok = true; |
