Merge pull request #46924 from nextcloud/fix/empty-email

skjnldsv · web-flow · commit 119543e694b8 · 2024-08-06T09:40:16.000+02:00
diff --git a/apps/files_sharing/lib/Controller/ShareesAPIController.php b/apps/files_sharing/lib/Controller/ShareesAPIController.php
@@ -257,7 +257,7 @@ private function getAllSharees(string $user, array $shareTypes): ISearchResult {
 			$sharees = $this->getAllShareesByType($user, $shareType);
 			$shareTypeResults = [];
 			foreach ($sharees as [$sharee, $displayname]) {
-				if (!isset($this->searchResultTypeMap[$shareType])) {
+				if (!isset($this->searchResultTypeMap[$shareType]) || trim($sharee) === '') {
 					continue;
 				}
 

Original file line number	Diff line number	Diff line change
`@@ -257,7 +257,7 @@ private function getAllSharees(string $user, array $shareTypes): ISearchResult {`
`257`	`257`	`$sharees = $this->getAllShareesByType($user, $shareType);`
`258`	`258`	`$shareTypeResults = [];`
`259`	`259`	`foreach ($sharees as [$sharee, $displayname]) {`
`260`		`- if (!isset($this->searchResultTypeMap[$shareType])) {`
	`260`	`+ if (!isset($this->searchResultTypeMap[$shareType]) \|\| trim($sharee) === '') {`
`261`	`261`	`continue;`
`262`	`262`	`}`
`263`	`263`