From 52d77eb9fe367a55dd0e8dc05abf7be6b00e298c Mon Sep 17 00:00:00 2001 From: Robin Windey Date: Sun, 13 Aug 2023 16:34:06 +0000 Subject: Make scope parsing more readable Signed-off-by: GitHub --- apps/files_trashbin/lib/Command/RestoreAllFiles.php | 19 ++++++++++--------- 1 file changed, 10 insertions(+), 9 deletions(-) (limited to 'apps/files_trashbin/lib') 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'"); } /** -- cgit v1.2.3