reset encryptionVersion to '1' if a file was stream copied, because this means that we basically write the file from scratch

Signed-off-by: Bjoern Schiessle <bjoern@schiessle.org>

Author: schiessle

lib/private/Files/Storage/Wrapper/Encryption.php

@@ -652,21 +652,22 @@ public function copyFromStorage(Storage\IStorage $sourceStorage, $sourceInternal
 	 * @param string $sourceInternalPath
 	 * @param string $targetInternalPath
 	 * @param bool $isRename
+	 * @param bool $keepEncryptionVersion
 	 */
-	private function updateEncryptedVersion(Storage\IStorage $sourceStorage, $sourceInternalPath, $targetInternalPath, $isRename) {
-		$isEncrypted = $this->encryptionManager->isEnabled() && $this->shouldEncrypt($targetInternalPath) ? 1 : 0;
+	private function updateEncryptedVersion(Storage\IStorage $sourceStorage, $sourceInternalPath, $targetInternalPath, $isRename, $keepEncryptionVersion) {
+		$isEncrypted = $this->encryptionManager->isEnabled() && $this->shouldEncrypt($targetInternalPath);
 		$cacheInformation = [
-			'encrypted' => (bool)$isEncrypted,
+			'encrypted' => $isEncrypted,
 		];
-		if($isEncrypted === 1) {
+		if($isEncrypted) {
 			$encryptedVersion = $sourceStorage->getCache()->get($sourceInternalPath)['encryptedVersion'];
 
 			// In case of a move operation from an unencrypted to an encrypted
 			// storage the old encrypted version would stay with "0" while the
 			// correct value would be "1". Thus we manually set the value to "1"
 			// for those cases.
 			// See also https://github.com/owncloud/core/issues/23078
-			if($encryptedVersion === 0) {
+			if($encryptedVersion === 0 || !$keepEncryptionVersion) {
 				$encryptedVersion = 1;
 			}
 
@@ -714,7 +715,7 @@ private function copyBetweenStorage(Storage\IStorage $sourceStorage, $sourceInte
 						$info['size']
 					);
 				}
-				$this->updateEncryptedVersion($sourceStorage, $sourceInternalPath, $targetInternalPath, $isRename);
+				$this->updateEncryptedVersion($sourceStorage, $sourceInternalPath, $targetInternalPath, $isRename, true);
 			}
 			return $result;
 		}
@@ -757,7 +758,7 @@ private function copyBetweenStorage(Storage\IStorage $sourceStorage, $sourceInte
 				if ($preserveMtime) {
 					$this->touch($targetInternalPath, $sourceStorage->filemtime($sourceInternalPath));
 				}
-				$this->updateEncryptedVersion($sourceStorage, $sourceInternalPath, $targetInternalPath, $isRename);
+				$this->updateEncryptedVersion($sourceStorage, $sourceInternalPath, $targetInternalPath, $isRename, false);
 			} else {
 				// delete partially written target file
 				$this->unlink($targetInternalPath);

tests/lib/Files/Storage/Wrapper/EncryptionTest.php

@@ -802,7 +802,7 @@ public function testCopyBetweenStorage($encryptionEnabled, $mountPointEncryption
 			'encrypted' => $expectedEncrypted,
 		];
 		if($expectedEncrypted === true) {
-			$expectedCachePut['encryptedVersion'] = 12345;
+			$expectedCachePut['encryptedVersion'] = 1;
 		}
 
 		$this->arrayCache->expects($this->never())->method('set');