From: Björn Schießle Date: Wed, 20 Mar 2013 11:45:24 +0000 (+0100) Subject: skip update if the recipient is the same user as the owner, otherwise we run in a... X-Git-Tag: v6.0.0alpha2~1016 X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=102120f105fd0f88349e3202024fca7c765e3587;p=nextcloud-server.git skip update if the recipient is the same user as the owner, otherwise we run in a infinite loop for group shares --- diff --git a/apps/files_sharing/lib/updater.php b/apps/files_sharing/lib/updater.php index 73e7808f24a..221aaee5421 100644 --- a/apps/files_sharing/lib/updater.php +++ b/apps/files_sharing/lib/updater.php @@ -38,10 +38,12 @@ class Shared_Updater { while (!empty($users)) { $reshareUsers = array(); foreach ($users as $user) { - $etag = \OC\Files\Filesystem::getETag(''); + if ( $user !== $uidOwner ) { + $etag = \OC\Files\Filesystem::getETag(''); \OCP\Config::setUserValue($user, 'files_sharing', 'etag', $etag); // Look for reshares $reshareUsers = array_merge($reshareUsers, \OCP\Share::getUsersItemShared('file', $info['fileid'], $user, true)); + } } $users = $reshareUsers; } @@ -88,10 +90,12 @@ class Shared_Updater { while (!empty($users)) { $reshareUsers = array(); foreach ($users as $user) { - $etag = \OC\Files\Filesystem::getETag(''); - \OCP\Config::setUserValue($user, 'files_sharing', 'etag', $etag); - // Look for reshares - $reshareUsers = array_merge($reshareUsers, \OCP\Share::getUsersItemShared('file', $params['fileSource'], $user, true)); + if ($user !== $uidOwner) { + $etag = \OC\Files\Filesystem::getETag(''); + \OCP\Config::setUserValue($user, 'files_sharing', 'etag', $etag); + // Look for reshares + $reshareUsers = array_merge($reshareUsers, \OCP\Share::getUsersItemShared('file', $params['fileSource'], $user, true)); + } } $users = $reshareUsers; }