diff options
| author | Robin Windey <ro.windey@gmail.com> | 2023-08-13 16:34:06 +0000 |
|---|---|---|
| committer | backportbot-nextcloud[bot] <backportbot-nextcloud[bot]@users.noreply.github.com> | 2023-09-22 11:14:53 +0000 |
| commit | 3354d61c8ef6af8bacfd37476b3dfc04285b99c6 (patch) | |
| tree | f0cac7a99341e10346a5e094637cacc0df6f8ea4 | |
| parent | f5d0264296616d2dc9975e3deb1acc08dec8ca0a (diff) | |
| download | nextcloud-server-3354d61c8ef6af8bacfd37476b3dfc04285b99c6.tar.gz nextcloud-server-3354d61c8ef6af8bacfd37476b3dfc04285b99c6.zip | |
Make scope parsing more readable
Signed-off-by: GitHub <noreply@github.com>
| -rw-r--r-- | apps/files_trashbin/lib/Command/RestoreAllFiles.php | 19 |
1 files changed, 10 insertions, 9 deletions
diff --git a/apps/files_trashbin/lib/Command/RestoreAllFiles.php b/apps/files_trashbin/lib/Command/RestoreAllFiles.php index 1e852c3658e..172e1af385b 100644 --- a/apps/files_trashbin/lib/Command/RestoreAllFiles.php +++ b/apps/files_trashbin/lib/Command/RestoreAllFiles.php @@ -39,6 +39,12 @@ class RestoreAllFiles extends Base { private const SCOPE_USER = 1; private const SCOPE_GROUPFOLDERS = 2; + private static $SCOPE_MAP = [ + 'user' => self::SCOPE_USER, + 'groupfolders' => self::SCOPE_GROUPFOLDERS, + 'all' => self::SCOPE_ALL + ]; + /** @var IUserManager */ protected $userManager; @@ -243,16 +249,11 @@ class RestoreAllFiles extends Base { * @return int */ protected function parseScope(string $scope): int { - switch ($scope) { - case 'user': - return self::SCOPE_USER; - case 'groupfolders': - return self::SCOPE_GROUPFOLDERS; - case 'all': - return self::SCOPE_ALL; - default: - throw new InvalidOptionException("Invalid scope '$scope'"); + if (isset(self::$SCOPE_MAP[$scope])) { + return self::$SCOPE_MAP[$scope]; } + + throw new InvalidOptionException("Invalid scope '$scope'"); } /** |