From 78e6bf0daf01e9934f799a9739b7b87244cd512d Mon Sep 17 00:00:00 2001 From: tobiasKaminsky Date: Wed, 5 Dec 2018 15:20:44 +0100 Subject: [PATCH 1/2] Add note to dav endpoint Signed-off-by: tobiasKaminsky --- apps/dav/lib/Connector/Sabre/FilesPlugin.php | 8 +++++ apps/dav/lib/Connector/Sabre/Node.php | 31 ++++++++++++++++++++ 2 files changed, 39 insertions(+) diff --git a/apps/dav/lib/Connector/Sabre/FilesPlugin.php b/apps/dav/lib/Connector/Sabre/FilesPlugin.php index f53f13c568..99317f2bc1 100644 --- a/apps/dav/lib/Connector/Sabre/FilesPlugin.php +++ b/apps/dav/lib/Connector/Sabre/FilesPlugin.php @@ -70,6 +70,7 @@ class FilesPlugin extends ServerPlugin { const HAS_PREVIEW_PROPERTYNAME = '{http://nextcloud.org/ns}has-preview'; const MOUNT_TYPE_PROPERTYNAME = '{http://nextcloud.org/ns}mount-type'; const IS_ENCRYPTED_PROPERTYNAME = '{http://nextcloud.org/ns}is-encrypted'; + const SHARE_NOTE = '{http://nextcloud.org/ns}note'; /** * Reference to main server object @@ -161,6 +162,7 @@ class FilesPlugin extends ServerPlugin { $server->protectedProperties[] = self::HAS_PREVIEW_PROPERTYNAME; $server->protectedProperties[] = self::MOUNT_TYPE_PROPERTYNAME; $server->protectedProperties[] = self::IS_ENCRYPTED_PROPERTYNAME; + $server->protectedProperties[] = self::SHARE_NOTE; // normally these cannot be changed (RFC4918), but we want them modifiable through PROPPATCH $allowedProperties = ['{DAV:}getetag']; @@ -359,6 +361,12 @@ class FilesPlugin extends ServerPlugin { $propFind->handle(self::MOUNT_TYPE_PROPERTYNAME, function () use ($node) { return $node->getFileInfo()->getMountPoint()->getMountType(); }); + + $propFind->handle(self::SHARE_NOTE, function() use ($node, $httpRequest) { + return $node->getNoteFromShare( + $httpRequest->getRawServerValue('PHP_AUTH_USER') + ); + }); } if ($node instanceof \OCA\DAV\Connector\Sabre\Node) { diff --git a/apps/dav/lib/Connector/Sabre/Node.php b/apps/dav/lib/Connector/Sabre/Node.php index 38d0ff57fb..b6433e6e0b 100644 --- a/apps/dav/lib/Connector/Sabre/Node.php +++ b/apps/dav/lib/Connector/Sabre/Node.php @@ -41,6 +41,8 @@ use OCP\Files\FileInfo; use OCP\Files\StorageNotAvailableException; use OCP\Share\Exceptions\ShareNotFound; use OCP\Share\IManager; +use OCP\Share; +use OCP\Share\IShare; abstract class Node implements \Sabre\DAV\INode { @@ -290,6 +292,35 @@ abstract class Node implements \Sabre\DAV\INode { return $permissions; } + /** + * @param string $user + * @return string + */ + public function getNoteFromShare($user) { + if ($user == null) { + return ''; + } + + $types = [ + Share::SHARE_TYPE_USER, + Share::SHARE_TYPE_GROUP, + Share::SHARE_TYPE_CIRCLE, + Share::SHARE_TYPE_ROOM + ]; + + foreach ($types as $shareType) { + $shares = $this->shareManager->getSharedWith($user, $shareType, $this, -1); + foreach ($shares as $share) { + $note = $share->getNote(); + if($share->getShareOwner() !== $user && !empty($note)) { + return $note; + } + } + } + + return ''; + } + /** * @return string */ From aee348c5f561d1166257d060f8153811f41c5d54 Mon Sep 17 00:00:00 2001 From: Christoph Wurst Date: Mon, 4 Mar 2019 08:24:14 +0100 Subject: [PATCH 2/2] Update apps/dav/lib/Connector/Sabre/Node.php Co-Authored-By: tobiasKaminsky --- apps/dav/lib/Connector/Sabre/Node.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/apps/dav/lib/Connector/Sabre/Node.php b/apps/dav/lib/Connector/Sabre/Node.php index b6433e6e0b..32ba6c53a0 100644 --- a/apps/dav/lib/Connector/Sabre/Node.php +++ b/apps/dav/lib/Connector/Sabre/Node.php @@ -297,7 +297,7 @@ abstract class Node implements \Sabre\DAV\INode { * @return string */ public function getNoteFromShare($user) { - if ($user == null) { + if ($user === null) { return ''; }