Merge pull request #53370 from nextcloud/revert-53078-backport/52914/stable30

skjnldsv · web-flow · commit b95e31be8666 · 2025-06-06T12:38:28.000+02:00
diff --git a/apps/dav/lib/DAV/GroupPrincipalBackend.php b/apps/dav/lib/DAV/GroupPrincipalBackend.php
@@ -66,10 +66,8 @@ public function getPrincipalsByPrefix($prefixPath) {
 		$principals = [];
 
 		if ($prefixPath === self::PRINCIPAL_PREFIX) {
-			foreach ($this->groupManager->search('') as $group) {
-				if (!$group->hideFromCollaboration()) {
-					$principals[] = $this->groupToPrincipal($group);
-				}
+			foreach ($this->groupManager->search('') as $user) {
+				$principals[] = $this->groupToPrincipal($user);
 			}
 		}
 
@@ -95,7 +93,7 @@ public function getPrincipalByPath($path) {
 		$name = urldecode($elements[2]);
 		$group = $this->groupManager->get($name);
 
-		if ($group !== null && !$group->hideFromCollaboration()) {
+		if (!is_null($group)) {
 			return $this->groupToPrincipal($group);
 		}
 

Original file line number	Diff line number	Diff line change
`@@ -66,10 +66,8 @@ public function getPrincipalsByPrefix($prefixPath) {`
`66`	`66`	`$principals = [];`
`67`	`67`
`68`	`68`	`if ($prefixPath === self::PRINCIPAL_PREFIX) {`
`69`		`- foreach ($this->groupManager->search('') as $group) {`
`70`		`- if (!$group->hideFromCollaboration()) {`
`71`		`- $principals[] = $this->groupToPrincipal($group);`
`72`		`- }`
	`69`	`+ foreach ($this->groupManager->search('') as $user) {`
	`70`	`+ $principals[] = $this->groupToPrincipal($user);`
`73`	`71`	`}`
`74`	`72`	`}`
`75`	`73`
`@@ -95,7 +93,7 @@ public function getPrincipalByPath($path) {`
`95`	`93`	`$name = urldecode($elements[2]);`
`96`	`94`	`$group = $this->groupManager->get($name);`
`97`	`95`
`98`		`- if ($group !== null && !$group->hideFromCollaboration()) {`
	`96`	`+ if (!is_null($group)) {`
`99`	`97`	`return $this->groupToPrincipal($group);`
`100`	`98`	`}`
`101`	`99`