fix: hide guests group from overall principals

nilsding · backportbot[bot] · commit 78495daaae2f · 2025-07-31T12:38:34.000Z
Follow-up for #52914 and #53369 Signed-off-by: Jyrki Gadinger <nilsding@nilsding.org>
diff --git a/apps/dav/lib/Connector/Sabre/Principal.php b/apps/dav/lib/Connector/Sabre/Principal.php
@@ -212,6 +212,9 @@ public function getGroupMembership($principal, $needGroups = false) {
 		if ($this->hasGroups || $needGroups) {
 			$userGroups = $this->groupManager->getUserGroups($user);
 			foreach ($userGroups as $userGroup) {
+				if ($userGroup->hideFromCollaboration()) {
+					continue;
+				}
 				$groups[] = 'principals/groups/' . urlencode($userGroup->getGID());
 			}
 		}

Original file line number	Diff line number	Diff line change
`@@ -212,6 +212,9 @@ public function getGroupMembership($principal, $needGroups = false) {`
`212`	`212`	`if ($this->hasGroups \|\| $needGroups) {`
`213`	`213`	`$userGroups = $this->groupManager->getUserGroups($user);`
`214`	`214`	`foreach ($userGroups as $userGroup) {`
	`215`	`+ if ($userGroup->hideFromCollaboration()) {`
	`216`	`+ continue;`
	`217`	`+ }`
`215`	`218`	`$groups[] = 'principals/groups/' . urlencode($userGroup->getGID());`
`216`	`219`	`}`
`217`	`220`	`}`