diff options
| author | Vincent Petry <pvince81@owncloud.com> | 2015-10-05 11:27:47 +0200 |
|---|---|---|
| committer | Vincent Petry <pvince81@owncloud.com> | 2015-10-05 17:21:34 +0200 |
| commit | 1c7244c1201ac607cc1b244551fa8f5282572ba7 (patch) | |
| tree | 6007201840d62970b6af508ab7337b75ee0bc076 /lib | |
| parent | cab0be6eedef14c419a2c94dce0461418e5f9cf8 (diff) | |
| download | nextcloud-server-1c7244c1201ac607cc1b244551fa8f5282572ba7.tar.gz nextcloud-server-1c7244c1201ac607cc1b244551fa8f5282572ba7.zip | |
Remove invalid share items from result when missing group membership
Group shares usually have subshare entries for every user. In some
situations it can happen that the user was removed from the group but
the subshare entries still exist.
This fix makes sure that such subshare entries are not returned any more
as the user isn't in the group any more.
Diffstat (limited to 'lib')
| -rw-r--r-- | lib/private/share/share.php | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/lib/private/share/share.php b/lib/private/share/share.php index 32389f34868..264dac30bb3 100644 --- a/lib/private/share/share.php +++ b/lib/private/share/share.php @@ -1908,6 +1908,12 @@ class Share extends Constants { $items = array_merge($items, $collectionItems); } + // filter out invalid items, these can appear when subshare entries exist + // for a group in which the requested user isn't a member any more + $items = array_filter($items, function($item) { + return $item['share_type'] !== self::$shareTypeGroupUserUnique; + }); + return self::formatResult($items, $column, $backend, $format, $parameters); } elseif ($includeCollections && $collectionTypes && in_array('folder', $collectionTypes)) { // FIXME: Thats a dirty hack to improve file sharing performance, |