Skip to content

Commit ca279f3

Browse files
committed
Update phpdoc for circleToPrincipal
as the QueryException is already catched inside circleToPrincipal. Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
1 parent 2a154f5 commit ca279f3

File tree

1 file changed

+1
-8
lines changed

1 file changed

+1
-8
lines changed

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

Lines changed: 1 addition & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -178,11 +178,7 @@ public function getPrincipalByPath($path) {
178178
return $this->userToPrincipal($user);
179179
}
180180
} elseif ($prefix === 'principals/circles') {
181-
try {
182-
return $this->circleToPrincipal($name);
183-
} catch (QueryException $e) {
184-
return null;
185-
}
181+
return $this->circleToPrincipal($name);
186182
}
187183
return null;
188184
}
@@ -471,9 +467,6 @@ public function getPrincipalPrefix() {
471467
/**
472468
* @param string $circleUniqueId
473469
* @return array|null
474-
* @throws \OCP\AppFramework\QueryException
475-
* @suppress PhanUndeclaredClassMethod
476-
* @suppress PhanUndeclaredClassCatch
477470
*/
478471
protected function circleToPrincipal($circleUniqueId) {
479472
if (!$this->appManager->isEnabledForUser('circles') || !class_exists('\OCA\Circles\Api\v1\Circles')) {

0 commit comments

Comments
 (0)