Merge pull request #284 from nextcloud/cleanup-after-sync
we don't need to check permissions twice
This commit is contained in:
commit
2f475a9f1e
1 changed files with 2 additions and 4 deletions
|
@ -313,6 +313,7 @@ class Storage {
|
||||||
*
|
*
|
||||||
* @param string $file file name
|
* @param string $file file name
|
||||||
* @param int $revision revision timestamp
|
* @param int $revision revision timestamp
|
||||||
|
* @return bool
|
||||||
*/
|
*/
|
||||||
public static function rollback($file, $revision) {
|
public static function rollback($file, $revision) {
|
||||||
|
|
||||||
|
@ -323,13 +324,10 @@ class Storage {
|
||||||
if ($uid === null || trim($filename, '/') === '') {
|
if ($uid === null || trim($filename, '/') === '') {
|
||||||
return false;
|
return false;
|
||||||
}
|
}
|
||||||
|
|
||||||
$users_view = new View('/'.$uid);
|
$users_view = new View('/'.$uid);
|
||||||
$files_view = new View('/'. User::getUser().'/files');
|
$files_view = new View('/'. User::getUser().'/files');
|
||||||
|
|
||||||
if (!$files_view->isUpdatable($filename)) {
|
|
||||||
return false;
|
|
||||||
}
|
|
||||||
|
|
||||||
$versionCreated = false;
|
$versionCreated = false;
|
||||||
|
|
||||||
$fileInfo = $files_view->getFileInfo($file);
|
$fileInfo = $files_view->getFileInfo($file);
|
||||||
|
|
Loading…
Reference in a new issue