mbk-lab
/
nextcloud
2
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

use the permissions mask cache wrapper instead of the read only cache

This commit is contained in:
Robin Appelman 2015-04-22 15:28:06 +02:00
parent 42d9ba0f83
commit 03b7f1d015
2 changed files with 4 additions and 94 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
apps/files_sharing/lib/readonlywrapper.php
View File

@ -23,7 +23,9 @@
namespace OCA\Files_Sharing; namespace OCA\Files_Sharing;
use OC\Files\Cache\Wrapper\CachePermissionsMask;
use OC\Files\Storage\Wrapper\Wrapper; use OC\Files\Storage\Wrapper\Wrapper;
use OCP\Constants;
class ReadOnlyWrapper extends Wrapper { class ReadOnlyWrapper extends Wrapper {
public function isUpdatable($path) { public function isUpdatable($path) {
@ -66,6 +68,7 @@ class ReadOnlyWrapper extends Wrapper {
if (!$storage) { if (!$storage) {
$storage = $this; $storage = $this;
} }
return new ReadOnlyCache($storage); $sourceCache = $this->storage->getCache($path, $storage);
return new CachePermissionsMask($sourceCache, Constants::PERMISSION_READ | Constants::PERMISSION_SHARE);
} }
} }

93
apps/files_sharing/tests/readonlycache.php
View File

@ -1,93 +0,0 @@
<?php
/**
* @author Olivier Paroz <owncloud@interfasys.ch>
*
* @copyright Copyright (c) 2015, ownCloud, Inc.
* @license AGPL-3.0
*
* This code is free software: you can redistribute it and/or modify
* it under the terms of the GNU Affero General Public License, version 3,
* as published by the Free Software Foundation.
*
* This program is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
* GNU Affero General Public License for more details.
*
* You should have received a copy of the GNU Affero General Public License, version 3,
* along with this program. If not, see <http://www.gnu.org/licenses/>
*
*/
namespace OCA\Files_Sharing\Tests;
class ReadOnlyCache extends TestCase {
/** @var \OC\Files\Storage\Storage */
protected $storage;
/** @var \OC\Files\Storage\StorageFactory */
protected $loader;
/** @var \OC\Files\Mount\MountPoint */
protected $readOnlyMount;
/** @var \OCA\Files_Sharing\ReadOnlyWrapper */
protected $readOnlyStorage;
/** @var \OC\Files\Cache\Cache */
protected $readOnlyCache;
protected function setUp() {
parent::setUp();
$this->view->mkdir('readonly');
$this->view->file_put_contents('readonly/foo.txt', 'foo');
$this->view->file_put_contents('readonly/bar.txt', 'bar');
list($this->storage) = $this->view->resolvePath('');
$this->loader = new \OC\Files\Storage\StorageFactory();
$this->readOnlyMount = new \OC\Files\Mount\MountPoint($this->storage,
'/readonly', [[]], $this->loader);
$this->readOnlyStorage = $this->loader->getInstance($this->readOnlyMount,
'\OCA\Files_Sharing\ReadOnlyWrapper', ['storage' => $this->storage]);
$this->readOnlyCache = $this->readOnlyStorage->getCache();
}
public function testSetup() {
$this->assertTrue($this->view->file_exists('/readonly/foo.txt'));
$perms = $this->readOnlyStorage->getPermissions('files/readonly/foo.txt');
$this->assertEquals(17, $perms);
$this->assertFalse($this->readOnlyStorage->unlink('files/readonly/foo.txt'));
$this->assertTrue($this->readOnlyStorage->file_exists('files/readonly/foo.txt'));
$this->assertInstanceOf('\OCA\Files_Sharing\ReadOnlyCache', $this->readOnlyCache);
}
public function testGetWhenFileExists() {
$result = $this->readOnlyCache->get('files/readonly/foo.txt');
$this->assertNotEmpty($result);
}
public function testGetWhenFileDoesNotExist() {
$result = $this->readOnlyCache->get('files/readonly/proof does not exist.md');
$this->assertFalse($result);
}
public function testGetFolderContentsWhenFolderExists() {
$results = $this->readOnlyCache->getFolderContents('files/readonly');
$this->assertNotEmpty($results);
foreach ($results as $result) {
$this->assertNotEmpty($result);
}
}
public function testGetFolderContentsWhenFolderDoesNotExist() {
$results = $this->readOnlyCache->getFolderContents('files/iamaghost');
$this->assertEmpty($results);
}
}