Do not use $_GET

The appframework is more than capable of handling this properly. Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
nextcloud · Jan 5, 2020 · 019e0f5 · 019e0f5
1 parent d7fdab0
commit 019e0f5
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/apps/files_sharing/lib/Controller/ShareesAPIController.php b/apps/files_sharing/lib/Controller/ShareesAPIController.php
@@ -198,8 +198,8 @@ public function search(string $search = '', string $itemType = null, int $page =
 			$shareTypes[] = Share::SHARE_TYPE_CIRCLE;
 		}
 
-		if (isset($_GET['shareType']) && is_array($_GET['shareType'])) {
-			$shareTypes = array_intersect($shareTypes, $_GET['shareType']);
+		if ($shareType !== null && is_array($shareType)) {
+			$shareTypes = array_intersect($shareTypes, $shareType);
 			sort($shareTypes);
 		} else if (is_numeric($shareType)) {
 			$shareTypes = array_intersect($shareTypes, [(int) $shareType]);