Merge pull request #14995 from owncloud/remove-unrequired-backtick-removal

Remove unrequired backtick removal
This commit is contained in:
Vincent Petry 2015-03-18 19:34:49 +01:00
commit fe8002a7db
2 changed files with 22 additions and 3 deletions

View file

@ -245,9 +245,6 @@ class Cache {
$queryParts[] = '`storage`';
$params[] = $this->getNumericStorageId();
$params = array_map(function($item) {
return trim($item, "`");
}, $params);
$queryParts = array_map(function($item) {
return trim($item, "`");
}, $queryParts);

View file

@ -260,6 +260,28 @@ class Cache extends \Test\TestCase {
$this->assertEquals(\OC\Files\Cache\Cache::COMPLETE, $this->cache->getStatus('foo'));
}
public function putWithAllKindOfQuotesData() {
return [
['`backtick`'],
['´forward´'],
['\'single\''],
];
}
/**
* @dataProvider putWithAllKindOfQuotesData
* @param $fileName
*/
public function testPutWithAllKindOfQuotes($fileName) {
$this->assertEquals(\OC\Files\Cache\Cache::NOT_FOUND, $this->cache->get($fileName));
$this->cache->put($fileName, array('size' => 20, 'mtime' => 25, 'mimetype' => 'foo/file', 'etag' => $fileName));
$cacheEntry = $this->cache->get($fileName);
$this->assertEquals($fileName, $cacheEntry['etag']);
$this->assertEquals($fileName, $cacheEntry['path']);
}
function testSearch() {
$file1 = 'folder';
$file2 = 'folder/foobar';