diff options
author | Bjoern Schiessle <schiessle@owncloud.com> | 2015-04-15 09:49:50 +0200 |
---|---|---|
committer | Joas Schilling <nickvergessen@owncloud.com> | 2015-05-06 14:20:02 +0200 |
commit | dc39bda8704d54d7481b9b42d2cea44146d18a3d (patch) | |
tree | 8f449397060624c1cdd38e715518adf54832a377 | |
parent | f2d46693c6a7633c27db0b0cbe5dea57525d809e (diff) | |
download | nextcloud-server-dc39bda8704d54d7481b9b42d2cea44146d18a3d.tar.gz nextcloud-server-dc39bda8704d54d7481b9b42d2cea44146d18a3d.zip |
move/copy from storage
-rw-r--r-- | lib/private/files/storage/wrapper/encryption.php | 74 | ||||
-rw-r--r-- | tests/lib/files/storage/wrapper/encryption.php | 19 |
2 files changed, 77 insertions, 16 deletions
diff --git a/lib/private/files/storage/wrapper/encryption.php b/lib/private/files/storage/wrapper/encryption.php index 24bbef9b93c..c37d3b43471 100644 --- a/lib/private/files/storage/wrapper/encryption.php +++ b/lib/private/files/storage/wrapper/encryption.php @@ -61,19 +61,20 @@ class Encryption extends Wrapper { /** @var IMountPoint */ private $mount; - /** @var \OCP\Encryption\Keys\IStorage */ + + /** @var IStorage */ private $keyStorage; - /** @var \OC\Encryption\Update */ + /** @var Update */ private $update; /** * @param array $parameters - * @param \OCP\Encryption\IManager $encryptionManager - * @param \OC\Encryption\Util $util - * @param \OCP\ILogger $logger - * @param \OCP\Encryption\IFile $fileHelper - * @param string $uid user who perform the read/write operation (null for public access) + * @param IManager $encryptionManager + * @param Util $util + * @param ILogger $logger + * @param IFile $fileHelper + * @param string $uid * @param IStorage $keyStorage * @param Update $update */ @@ -366,6 +367,65 @@ class Encryption extends Wrapper { } /** + * @param \OCP\Files\Storage $sourceStorage + * @param string $sourceInternalPath + * @param string $targetInternalPath + * @param bool $preserveMtime + * @return bool + */ + public function moveFromStorage(\OCP\Files\Storage $sourceStorage, $sourceInternalPath, $targetInternalPath, $preserveMtime = false) { + $result = $this->copyFromStorage($sourceStorage, $sourceInternalPath, $targetInternalPath, true); + if ($result) { + if ($sourceStorage->is_dir($sourceInternalPath)) { + $result &= $sourceStorage->rmdir($sourceInternalPath); + } else { + $result &= $sourceStorage->unlink($sourceInternalPath); + } + } + return $result; + } + + + /** + * @param \OCP\Files\Storage $sourceStorage + * @param string $sourceInternalPath + * @param string $targetInternalPath + * @param bool $preserveMtime + * @return bool + */ + public function copyFromStorage(\OCP\Files\Storage $sourceStorage, $sourceInternalPath, $targetInternalPath, $preserveMtime = false) { + if ($sourceStorage->is_dir($sourceInternalPath)) { + $dh = $sourceStorage->opendir($sourceInternalPath); + $result = $this->mkdir($targetInternalPath); + if (is_resource($dh)) { + while ($result and ($file = readdir($dh)) !== false) { + if (!Filesystem::isIgnoredDir($file)) { + $result &= $this->copyFromStorage($sourceStorage, $sourceInternalPath . '/' . $file, $targetInternalPath . '/' . $file); + } + } + } + } else { + $source = $sourceStorage->fopen($sourceInternalPath, 'r'); + $target = $this->fopen($targetInternalPath, 'w'); + list(, $result) = \OC_Helper::streamCopy($source, $target); + if ($result and $preserveMtime) { + $this->touch($targetInternalPath, $sourceStorage->filemtime($sourceInternalPath)); + } + fclose($source); + fclose($target); + + if (!$result) { + // delete partially written target file + $this->unlink($targetInternalPath); + // delete cache entry that was created by fopen + $this->getCache()->remove($targetInternalPath); + } + } + return (bool)$result; + + } + + /** * get the path to a local version of the file. * The local version of the file can be temporary and doesn't have to be persistent across requests * diff --git a/tests/lib/files/storage/wrapper/encryption.php b/tests/lib/files/storage/wrapper/encryption.php index 6c593bb7749..bc803f5f875 100644 --- a/tests/lib/files/storage/wrapper/encryption.php +++ b/tests/lib/files/storage/wrapper/encryption.php @@ -106,15 +106,17 @@ class Encryption extends \Test\Files\Storage\Storage { ->willReturn(['encrypted' => false]); $this->instance = $this->getMockBuilder('\OC\Files\Storage\Wrapper\Encryption') - ->setConstructorArgs([ + ->setConstructorArgs( [ - 'storage' => $this->sourceStorage, - 'root' => 'foo', - 'mountPoint' => '/', - 'mount' => $mount - ], - $this->encryptionManager, $this->util, $logger, $file, null, $this->keyStore, $this->update - ]) + [ + 'storage' => $this->sourceStorage, + 'root' => 'foo', + 'mountPoint' => '/', + 'mount' => $mount + ], + $this->encryptionManager, $this->util, $logger, $file, null, $this->keyStore, $this->update + ] + ) ->setMethods(['getMetaData', 'getCache']) ->getMock(); @@ -125,7 +127,6 @@ class Encryption extends \Test\Files\Storage\Storage { $this->instance->expects($this->any()) ->method('getCache') ->willReturn($this->cache); - } /** |