Skip to content

Commit 13afec7

Browse files
Jirka Fdg
authored andcommitted
added Cache::NAMESPACES and support for cleaning namespace in FileStorage [Closes #52]
1 parent aebaa0e commit 13afec7

File tree

3 files changed

+111
-0
lines changed

3 files changed

+111
-0
lines changed

src/Caching/Cache.php

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,7 @@ class Cache
3131
ITEMS = 'items',
3232
CONSTS = 'consts',
3333
CALLBACKS = 'callbacks',
34+
NAMESPACES = 'namespaces',
3435
ALL = 'all';
3536

3637
/** @internal */
@@ -198,6 +199,11 @@ private function completeDependencies(?array $dp): array
198199
$dp[self::TAGS] = array_values((array) $dp[self::TAGS]);
199200
}
200201

202+
// make list from NAMESPACES
203+
if (isset($dp[self::NAMESPACES])) {
204+
$dp[self::NAMESPACES] = array_values((array) $dp[self::NAMESPACES]);
205+
}
206+
201207
// convert FILES into CALLBACKS
202208
if (isset($dp[self::FILES])) {
203209
foreach (array_unique((array) $dp[self::FILES]) as $item) {

src/Caching/Storages/FileStorage.php

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -249,6 +249,7 @@ public function clean(array $conditions): void
249249
{
250250
$all = !empty($conditions[Cache::ALL]);
251251
$collector = empty($conditions);
252+
$namespaces = $conditions[Cache::NAMESPACES] ?? null;
252253

253254
// cleaning using file iterator
254255
if ($all || $collector) {
@@ -284,6 +285,17 @@ public function clean(array $conditions): void
284285
$this->journal->clean($conditions);
285286
}
286287
return;
288+
289+
} elseif ($namespaces) {
290+
foreach ($namespaces as $namespace) {
291+
$dir = $this->dir . '/_' . urlencode($namespace);
292+
if (is_dir($dir)) {
293+
foreach (Nette\Utils\Finder::findFiles('_*')->in($dir) as $entry) {
294+
$this->delete((string) $entry);
295+
}
296+
@rmdir($dir); // may already contain new files
297+
}
298+
}
287299
}
288300

289301
// cleaning using journal
Lines changed: 93 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,93 @@
1+
<?php
2+
3+
/**
4+
* Test: Nette\Caching\Storages\FileStorage clean with Cache::NAMESPACE
5+
*/
6+
7+
declare(strict_types=1);
8+
9+
use Nette\Caching\Cache;
10+
use Nette\Caching\Storages\FileStorage;
11+
use Tester\Assert;
12+
13+
14+
require __DIR__ . '/../bootstrap.php';
15+
16+
$storage = new FileStorage(TEMP_DIR);
17+
18+
/*
19+
* Create filestorage cache without namespace and some with namespaces
20+
*/
21+
$cacheA = new Cache($storage);
22+
$cacheB = new Cache($storage, 'C' . Cache::NAMESPACE_SEPARATOR . 'A');
23+
$cacheC = new Cache($storage, 'C');
24+
$cacheD = new Cache($storage, 'D');
25+
26+
/*
27+
* Fill with data
28+
*/
29+
$cacheA->save('test1', 'David');
30+
$cacheA->save('test2', 'Grudl');
31+
32+
$cacheB->save('test1', 'Barry');
33+
$cacheB->save('test2', 'Allen');
34+
35+
$cacheC->save('test1', 'Oliver');
36+
$cacheC->save('test2', 'Queen');
37+
38+
$cacheD->save('test1', 'Bruce');
39+
$cacheD->save('test2', 'Wayne');
40+
41+
42+
/*
43+
* Check if fill wass successfull
44+
*/
45+
Assert::same('David', $cacheA->load('test1'));
46+
Assert::same('Grudl', $cacheA->load('test2'));
47+
48+
Assert::same('Barry', $cacheB->load('test1'));
49+
Assert::same('Allen', $cacheB->load('test2'));
50+
51+
Assert::same('Oliver', $cacheC->load('test1'));
52+
Assert::same('Queen', $cacheC->load('test2'));
53+
54+
Assert::same('Bruce', $cacheD->load('test1'));
55+
Assert::same('Wayne', $cacheD->load('test2'));
56+
57+
58+
/*
59+
* Clean one namespace
60+
*/
61+
$storage->clean([Cache::NAMESPACES => [$cacheB->getNamespace()]]);
62+
63+
Assert::same('David', $cacheA->load('test1'));
64+
Assert::same('Grudl', $cacheA->load('test2'));
65+
66+
// Only these should be null now
67+
Assert::null($cacheB->load('test1'));
68+
Assert::null($cacheB->load('test2'));
69+
70+
Assert::same('Oliver', $cacheC->load('test1'));
71+
Assert::same('Queen', $cacheC->load('test2'));
72+
73+
Assert::same('Bruce', $cacheD->load('test1'));
74+
Assert::same('Wayne', $cacheD->load('test2'));
75+
76+
77+
/*
78+
* Test cleaning multiple namespaces
79+
*/
80+
$storage->clean([Cache::NAMESPACES => [$cacheC->getNamespace(), $cacheD->getNamespace()]]);
81+
82+
Assert::same('David', $cacheA->load('test1'));
83+
Assert::same('Grudl', $cacheA->load('test2'));
84+
85+
// All other should be null
86+
Assert::null($cacheB->load('test1'));
87+
Assert::null($cacheB->load('test2'));
88+
89+
Assert::null($cacheC->load('test1'));
90+
Assert::null($cacheC->load('test2'));
91+
92+
Assert::null($cacheD->load('test1'));
93+
Assert::null($cacheD->load('test2'));

0 commit comments

Comments
 (0)