Fix permision mask

If we move a file from the temp part file to the original file we don't need update permissions. Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
nextcloud · Oct 22, 2016 · 35f9d3c · 35f9d3c
1 parent dff0eb9
commit 35f9d3c
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 0 deletions.
diff --git a/apps/dav/lib/Files/Sharing/FilesDropPlugin.php b/apps/dav/lib/Files/Sharing/FilesDropPlugin.php
diff --git a/lib/private/Files/Storage/Wrapper/PermissionsMask.php b/lib/private/Files/Storage/Wrapper/PermissionsMask.php
@@ -78,6 +78,14 @@ public function getPermissions($path) {
 	}
 
 	public function rename($path1, $path2) {
+		$p = strpos($path1, $path2);
+		if ($p === 0) {
+			$part = substr($path1, strlen($path2));
+			//This is a rename of the transfer file to the original file
+			if (strpos($part, '.ocTransferId') === 0) {
+				return $this->checkMask(Constants::PERMISSION_CREATE) and parent::rename($path1, $path2);
+			}
+		}
 		return $this->checkMask(Constants::PERMISSION_UPDATE) and parent::rename($path1, $path2);
 	}