Comments from Vincent

2 files changed, 8 insertions, 8 deletions

diff --git a/lib/private/share20/defaultshareprovider.php b/lib/private/share20/defaultshareprovider.php
index 596addb1e67..e0b437bbcc7 100644
--- a/lib/private/share20/defaultshareprovider.php
+++ b/lib/private/share20/defaultshareprovider.php
@@ -367,8 +367,8 @@ class DefaultShareProvider implements IShareProvider {
 	/**
 	 * Get shared with the given user
 	 *
-	 * @param IUser $user
-	 * @param int $shareType
+	 * @param IUser $user get shares where this user is the recipient
+	 * @param int $shareType \OCP\Share::SHARE_TYPE_USER or \OCP\Share::SHARE_TYPE_GROUP are supported
 	 * @param int $limit The maximum number of shares, -1 for all
 	 * @param int $offset
 	 * @return IShare[]
@@ -450,7 +450,7 @@ class DefaultShareProvider implements IShareProvider {
  			 */
 			$shares = array_map([$this, 'resolveGroupShare'], $shares);
 		} else {
-			throw new BackendError();
+			throw new BackendError('Invalid backend');
 		}
 
 
@@ -585,7 +585,7 @@ class DefaultShareProvider implements IShareProvider {
 	 * Thus if the user moved their group share make sure this is properly reflected here.
 	 *
 	 * @param Share $share
-	 * @return Share
+	 * @return Share Returns the updated share if one was found else return the original share.
 	 */
 	private function resolveGroupShare(Share $share) {
 		$qb = $this->dbConn->getQueryBuilder();
diff --git a/lib/private/share20/ishareprovider.php b/lib/private/share20/ishareprovider.php
index a5a9d3b6708..36d0f10c7f1 100644
--- a/lib/private/share20/ishareprovider.php
+++ b/lib/private/share20/ishareprovider.php
@@ -62,13 +62,13 @@ interface IShareProvider {
 	 *
 	 * @param IUser $user
 	 * @param int $shareType
-	 * @param \OCP\Files\File|\OCP\Files\Folder $path
-	 * @param bool $reshares
+	 * @param \OCP\Files\File|\OCP\Files\Folder $node
+	 * @param bool $reshares Also get the shares where $user is the owner instead of just the shares where $user is the initiator
 	 * @param int $limit The maximum number of shares to be returned, -1 for all shares
 	 * @param int $offset
 	 * @return Share[]
 	 */
-	public function getSharesBy(IUser $user, $shareType, $path, $reshares, $limit, $offset);
+	public function getSharesBy(IUser $user, $shareType, $node, $reshares, $limit, $offset);
 
 	/**
 	 * Get share by id
@@ -98,7 +98,7 @@ interface IShareProvider {
 	/**
 	 * Get shared with the given user
 	 *
-	 * @param IUser $user
+	 * @param IUser $user get shares where this user is the recipient
 	 * @param int $shareType
 	 * @param int $limit The max number of entries returned, -1 for all
 	 * @param int $offset