Use TempManager instead of tmpFolder
This commit is contained in:
parent
6a7dbf3cf2
commit
90b0831cec
17 changed files with 17 additions and 29 deletions
|
@ -86,7 +86,7 @@ class OC_Archive_TAR extends OC_Archive {
|
|||
* @return bool
|
||||
*/
|
||||
function addFolder($path) {
|
||||
$tmpBase = OC_Helper::tmpFolder();
|
||||
$tmpBase = \OC::$server->getTempManager()->getTemporaryFolder();
|
||||
if (substr($path, -1, 1) != '/') {
|
||||
$path .= '/';
|
||||
}
|
||||
|
|
|
@ -248,7 +248,7 @@ abstract class Common implements Storage {
|
|||
}
|
||||
|
||||
public function getLocalFolder($path) {
|
||||
$baseDir = \OC_Helper::tmpFolder();
|
||||
$baseDir = \OC::$server->getTempManager()->getTemporaryFolder();
|
||||
$this->addLocalFolder($path, $baseDir);
|
||||
return $baseDir;
|
||||
}
|
||||
|
|
|
@ -29,7 +29,7 @@ namespace OC\Files\Storage;
|
|||
*/
|
||||
class Temporary extends Local{
|
||||
public function __construct($arguments = null) {
|
||||
parent::__construct(array('datadir' => \OC_Helper::tmpFolder()));
|
||||
parent::__construct(array('datadir' => \OC::$server->getTempManager()->getTemporaryFolder()));
|
||||
}
|
||||
|
||||
public function cleanUp() {
|
||||
|
|
|
@ -469,18 +469,6 @@ class OC_Helper {
|
|||
return \OC::$server->getTempManager()->getTemporaryFile($postfix);
|
||||
}
|
||||
|
||||
/**
|
||||
* create a temporary folder with an unique filename
|
||||
*
|
||||
* @return string
|
||||
* @deprecated Use the TempManager instead
|
||||
*
|
||||
* temporary files are automatically cleaned up after the script is finished
|
||||
*/
|
||||
public static function tmpFolder() {
|
||||
return \OC::$server->getTempManager()->getTemporaryFolder();
|
||||
}
|
||||
|
||||
/**
|
||||
* Adds a suffix to the name in case the file exists
|
||||
*
|
||||
|
|
|
@ -284,7 +284,7 @@ class OC_Installer{
|
|||
}
|
||||
|
||||
//extract the archive in a temporary folder
|
||||
$extractDir=OC_Helper::tmpFolder();
|
||||
$extractDir = \OC::$server->getTempManager()->getTemporaryFolder();
|
||||
OC_Helper::rmdirr($extractDir);
|
||||
mkdir($extractDir);
|
||||
if($archive=OC_Archive::open($path)) {
|
||||
|
|
|
@ -23,7 +23,7 @@ class ConfigTests extends TestCase {
|
|||
protected function setUp() {
|
||||
parent::setUp();
|
||||
|
||||
$this->randomTmpDir = \OC_Helper::tmpFolder();
|
||||
$this->randomTmpDir = \OC::$server->getTempManager()->getTemporaryFolder();
|
||||
$this->configFile = $this->randomTmpDir.'testconfig.php';
|
||||
file_put_contents($this->configFile, self::TESTCONTENT);
|
||||
$this->config = new \OC\Config($this->randomTmpDir, 'testconfig.php');
|
||||
|
|
2
tests/lib/files/cache/homecache.php
vendored
2
tests/lib/files/cache/homecache.php
vendored
|
@ -69,7 +69,7 @@ class HomeCache extends \Test\TestCase {
|
|||
protected function setUp() {
|
||||
parent::setUp();
|
||||
|
||||
$this->user = new DummyUser('foo', \OC_Helper::tmpFolder());
|
||||
$this->user = new DummyUser('foo', \OC::$server->getTempManager()->getTemporaryFolder());
|
||||
$this->storage = new \OC\Files\Storage\Home(array('user' => $this->user));
|
||||
$this->cache = $this->storage->getCache();
|
||||
}
|
||||
|
|
|
@ -39,7 +39,7 @@ class EtagTest extends \Test\TestCase {
|
|||
|
||||
$config = \OC::$server->getConfig();
|
||||
$this->datadir = $config->getSystemValue('datadirectory');
|
||||
$this->tmpDir = \OC_Helper::tmpFolder();
|
||||
$this->tmpDir = \OC::$server->getTempManager()->getTemporaryFolder();
|
||||
$config->setSystemValue('datadirectory', $this->tmpDir);
|
||||
|
||||
$this->userBackend = new \Test\Util\User\Dummy();
|
||||
|
|
|
@ -72,7 +72,7 @@ class Filesystem extends \Test\TestCase {
|
|||
* @return array
|
||||
*/
|
||||
private function getStorageData() {
|
||||
$dir = \OC_Helper::tmpFolder();
|
||||
$dir = \OC::$server->getTempManager()->getTemporaryFolder();
|
||||
$this->tmpDirs[] = $dir;
|
||||
return array('datadir' => $dir);
|
||||
}
|
||||
|
@ -416,7 +416,7 @@ class Filesystem extends \Test\TestCase {
|
|||
$config = \OC::$server->getConfig();
|
||||
$oldCachePath = $config->getSystemValue('cache_path', '');
|
||||
// set cache path to temp dir
|
||||
$cachePath = \OC_Helper::tmpFolder() . '/extcache';
|
||||
$cachePath = \OC::$server->getTempManager()->getTemporaryFolder() . '/extcache';
|
||||
$config->setSystemValue('cache_path', $cachePath);
|
||||
|
||||
\OC::$server->getUserManager()->createUser($userId, $userId);
|
||||
|
|
|
@ -37,7 +37,7 @@ class CommonTest extends Storage {
|
|||
protected function setUp() {
|
||||
parent::setUp();
|
||||
|
||||
$this->tmpDir=\OC_Helper::tmpFolder();
|
||||
$this->tmpDir = \OC::$server->getTempManager()->getTemporaryFolder();
|
||||
$this->instance=new \OC\Files\Storage\CommonTest(array('datadir'=>$this->tmpDir));
|
||||
}
|
||||
|
||||
|
|
|
@ -70,7 +70,7 @@ class Home extends Storage {
|
|||
protected function setUp() {
|
||||
parent::setUp();
|
||||
|
||||
$this->tmpDir = \OC_Helper::tmpFolder();
|
||||
$this->tmpDir = \OC::$server->getTempManager()->getTemporaryFolder();
|
||||
$this->userId = $this->getUniqueID('user_');
|
||||
$this->user = new DummyUser($this->userId, $this->tmpDir);
|
||||
$this->instance = new \OC\Files\Storage\Home(array('user' => $this->user));
|
||||
|
|
|
@ -38,7 +38,7 @@ class Local extends Storage {
|
|||
protected function setUp() {
|
||||
parent::setUp();
|
||||
|
||||
$this->tmpDir = \OC_Helper::tmpFolder();
|
||||
$this->tmpDir = \OC::$server->getTempManager()->getTemporaryFolder();
|
||||
$this->instance = new \OC\Files\Storage\Local(array('datadir' => $this->tmpDir));
|
||||
}
|
||||
|
||||
|
|
|
@ -27,7 +27,7 @@ class Quota extends \Test\Files\Storage\Storage {
|
|||
protected function setUp() {
|
||||
parent::setUp();
|
||||
|
||||
$this->tmpDir = \OC_Helper::tmpFolder();
|
||||
$this->tmpDir = \OC::$server->getTempManager()->getTemporaryFolder();
|
||||
$storage = new \OC\Files\Storage\Local(array('datadir' => $this->tmpDir));
|
||||
$this->instance = new \OC\Files\Storage\Wrapper\Quota(array('storage' => $storage, 'quota' => 10000000));
|
||||
}
|
||||
|
|
|
@ -17,7 +17,7 @@ class Wrapper extends \Test\Files\Storage\Storage {
|
|||
protected function setUp() {
|
||||
parent::setUp();
|
||||
|
||||
$this->tmpDir = \OC_Helper::tmpFolder();
|
||||
$this->tmpDir = \OC::$server->getTempManager()->getTemporaryFolder();
|
||||
$storage = new \OC\Files\Storage\Local(array('datadir' => $this->tmpDir));
|
||||
$this->instance = new \OC\Files\Storage\Wrapper\Wrapper(array('storage' => $storage));
|
||||
}
|
||||
|
|
|
@ -757,7 +757,7 @@ class View extends \Test\TestCase {
|
|||
* 228 is the max path length in windows
|
||||
*/
|
||||
$folderName = 'abcdefghijklmnopqrstuvwxyz012345678901234567890123456789';
|
||||
$tmpdirLength = strlen(\OC_Helper::tmpFolder());
|
||||
$tmpdirLength = strlen(\OC::$server->getTempManager()->getTemporaryFolder());
|
||||
if (\OC_Util::runningOnWindows()) {
|
||||
$this->markTestSkipped('[Windows] ');
|
||||
$depth = ((260 - $tmpdirLength) / 57);
|
||||
|
|
|
@ -403,7 +403,7 @@ class Test_Helper extends \Test\TestCase {
|
|||
* Tests recursive folder deletion with rmdirr()
|
||||
*/
|
||||
public function testRecursiveFolderDeletion() {
|
||||
$baseDir = \OC_Helper::tmpFolder() . '/';
|
||||
$baseDir = \OC::$server->getTempManager()->getTemporaryFolder() . '/';
|
||||
mkdir($baseDir . 'a/b/c/d/e', 0777, true);
|
||||
mkdir($baseDir . 'a/b/c1/d/e', 0777, true);
|
||||
mkdir($baseDir . 'a/b/c2/d/e', 0777, true);
|
||||
|
|
|
@ -37,7 +37,7 @@ class Test_Util_CheckServer extends \Test\TestCase {
|
|||
protected function setUp() {
|
||||
parent::setUp();
|
||||
|
||||
$this->datadir = \OC_Helper::tmpFolder();
|
||||
$this->datadir = \OC::$server->getTempManager()->getTemporaryFolder();
|
||||
|
||||
file_put_contents($this->datadir . '/.ocdata', '');
|
||||
\OC::$server->getSession()->set('checkServer_succeeded', false);
|
||||
|
|
Loading…
Reference in a new issue