do not remove valid group shares

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>

Author: blizzz

apps/dav/lib/Connector/Sabre/Principal.php

@@ -154,6 +154,7 @@ public function getPrincipalsByPrefix($prefixPath) {
 	 */
 	public function getPrincipalByPath($path) {
 		list($prefix, $name) = \Sabre\Uri\split($path);
+		$name = urldecode($name);
 
 		if ($name === 'calendar-proxy-write' || $name === 'calendar-proxy-read') {
 			list($prefix2, $name2) = \Sabre\Uri\split($prefix);
@@ -175,7 +176,7 @@ public function getPrincipalByPath($path) {
 			// is called either with a urlencoded version of the name or with a non-urlencoded one.
 			// The urldecode function replaces %## and +, both of which are forbidden in usernames.
 			// Hence there can be no ambiguity here and it is safe to call urldecode on all usernames
-			$user = $this->userManager->get(urldecode($name));
+			$user = $this->userManager->get($name);
 
 			if ($user !== null) {
 				return $this->userToPrincipal($user);
@@ -184,6 +185,14 @@ public function getPrincipalByPath($path) {
 			if ($this->userSession->getUser() !== null) {
 				return $this->circleToPrincipal($name);
 			}
+		} elseif ($prefix === 'principals/groups') {
+			if ($this->hasGroups && $this->groupManager->groupExists($name)) {
+				$group = $this->groupManager->get($name);
+				return [
+					'uri' => 'principals/groups/' . $name,
+					'{DAV:}displayname' => $group->getDisplayName(),
+				];
+			}
 		}
 		return null;
 	}