aboutsummaryrefslogtreecommitdiffstats
path: root/apps
diff options
context:
space:
mode:
authorDaniel Calviño Sánchez <danxuliu@gmail.com>2018-07-24 11:57:52 +0200
committerDaniel Calviño Sánchez <danxuliu@gmail.com>2018-08-08 14:25:44 +0200
commite2e6f23b6722eaf9c0aa2bf94626c6b1674aff0a (patch)
tree747e7bc0355dca939a7cc465d74e507b24e63114 /apps
parente1561f0e8fa0437e485820cadb8c225e8d10d3df (diff)
downloadnextcloud-server-e2e6f23b6722eaf9c0aa2bf94626c6b1674aff0a.tar.gz
nextcloud-server-e2e6f23b6722eaf9c0aa2bf94626c6b1674aff0a.zip
Suppress Phan warnings about calling undeclared class methods
The DeletedShareAPIController and ShareAPIController helpers for room shares are defined in Talk, so the classes do not exist when Talk is not installed. Due to this when the object returned by "getRoomShareHelper" is used Phan complains that the class is not declared. This is not a problem, though, because when the class is not available "getRoomShareHelper" throws an exception, which is then caught where that method was called. Therefore now those warnings from Phan are suppressed (it would be better to use "@phan-suppress-next-line" instead, but it is not yet available in our Phan version). Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
Diffstat (limited to 'apps')
-rw-r--r--apps/files_sharing/lib/Controller/DeletedShareAPIController.php3
-rw-r--r--apps/files_sharing/lib/Controller/ShareAPIController.php5
2 files changed, 8 insertions, 0 deletions
diff --git a/apps/files_sharing/lib/Controller/DeletedShareAPIController.php b/apps/files_sharing/lib/Controller/DeletedShareAPIController.php
index 7648c4e432d..6c7242ef613 100644
--- a/apps/files_sharing/lib/Controller/DeletedShareAPIController.php
+++ b/apps/files_sharing/lib/Controller/DeletedShareAPIController.php
@@ -86,6 +86,9 @@ class DeletedShareAPIController extends OCSController {
$this->serverContainer = $serverContainer;
}
+ /**
+ * @suppress PhanUndeclaredClassMethod
+ */
private function formatShare(IShare $share): array {
$result = [
diff --git a/apps/files_sharing/lib/Controller/ShareAPIController.php b/apps/files_sharing/lib/Controller/ShareAPIController.php
index 15dda8928d4..461c0e47320 100644
--- a/apps/files_sharing/lib/Controller/ShareAPIController.php
+++ b/apps/files_sharing/lib/Controller/ShareAPIController.php
@@ -141,6 +141,8 @@ class ShareAPIController extends OCSController {
* @param Node|null $recipientNode
* @return array
* @throws NotFoundException In case the node can't be resolved.
+ *
+ * @suppress PhanUndeclaredClassMethod
*/
protected function formatShare(\OCP\Share\IShare $share, Node $recipientNode = null): array {
$sharedBy = $this->userManager->get($share->getSharedBy());
@@ -914,6 +916,9 @@ class ShareAPIController extends OCSController {
return new DataResponse($this->formatShare($share));
}
+ /**
+ * @suppress PhanUndeclaredClassMethod
+ */
protected function canAccessShare(\OCP\Share\IShare $share, bool $checkGroups = true): bool {
// A file with permissions 0 can't be accessed by us. So Don't show it
if ($share->getPermissions() === 0) {