Merge pull request #34001 from nextcloud/versions-create-user

use filesystem user instead of file owner for creating version
This commit is contained in:
blizzz 2022-09-12 18:37:02 +02:00 committed by GitHub
commit a1fcee0c99
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -210,9 +210,18 @@ class Storage {
$ownerNodes = $ownerFolder->getById($file->getId());
if (count($ownerNodes)) {
$file = current($ownerNodes);
$uid = $mount->getShare()->getShareOwner();
}
}
/** @var IUserManager $userManager */
$userManager = \OC::$server->get(IUserManager::class);
$user = $userManager->get($uid);
if (!$user) {
return false;
}
// no use making versions for empty files
if ($file->getSize() === 0) {
return false;
@ -227,7 +236,7 @@ class Storage {
/** @var IVersionManager $versionManager */
$versionManager = \OC::$server->get(IVersionManager::class);
$versionManager->createVersion($file->getOwner(), $file);
$versionManager->createVersion($user, $file);
}