mbk-lab
/
nextcloud
2
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Update phpdoc for circleToPrincipal 

as the QueryException is already catched inside circleToPrincipal.

Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
This commit is contained in:
Daniel Kesselberg 2020-10-23 21:12:15 +02:00 committed by backportbot[bot]
parent ae3bfb97c3
commit 04364169ed
1 changed files with 1 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
apps/dav/lib/Connector/Sabre/Principal.php
View File

@ -178,11 +178,7 @@ class Principal implements BackendInterface {
return $this->userToPrincipal($user); return $this->userToPrincipal($user);
} }
} elseif ($prefix === 'principals/circles') { } elseif ($prefix === 'principals/circles') {
try { return $this->circleToPrincipal($name);
return $this->circleToPrincipal($name);
} catch (QueryException $e) {
return null;
}
} }
return null; return null;
} }
@ -471,9 +467,6 @@ class Principal implements BackendInterface {
/** /**
* @param string $circleUniqueId * @param string $circleUniqueId
* @return array|null * @return array|null
* @throws \OCP\AppFramework\QueryException
* @suppress PhanUndeclaredClassMethod
* @suppress PhanUndeclaredClassCatch
*/ */
protected function circleToPrincipal($circleUniqueId) { protected function circleToPrincipal($circleUniqueId) {
if (!$this->appManager->isEnabledForUser('circles') || !class_exists('\OCA\Circles\Api\v1\Circles')) { if (!$this->appManager->isEnabledForUser('circles') || !class_exists('\OCA\Circles\Api\v1\Circles')) {