Skip to content

Commit f66b8f9

Browse files
dgdg
committed
coding style
1 parent e5d4189 commit f66b8f9

File tree

9 files changed

+25
-57
lines changed

9 files changed

+25
-57
lines changed

‎src/Bridges/CacheLatte/CacheMacro.php‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,7 @@ public function nodeOpened(Latte\MacroNode $node)
6363
->write(
6464
'<?php if (Nette\Bridges\CacheLatte\CacheMacro::createCache($this->global->cacheStorage, %var, $this->global->cacheStack, %node.array?)) /* line %var */ try { ?>',
6565
Nette\Utils\Random::generate(),
66-
$node->startLine
66+
$node->startLine,
6767
);
6868
}
6969

@@ -81,7 +81,7 @@ public function nodeClosed(Latte\MacroNode $node)
8181
} catch (\Throwable $ʟ_e) {
8282
Nette\Bridges\CacheLatte\CacheMacro::rollback($this->global->cacheStack); throw $ʟ_e;
8383
} ?>',
84-
$node->startLine
84+
$node->startLine,
8585
);
8686
}
8787

@@ -108,7 +108,7 @@ public static function createCache(
108108
Nette\Caching\Storage $cacheStorage,
109109
string $key,
110110
?array &$parents,
111-
?array $args = null
111+
?array $args = null,
112112
) {
113113
if ($args) {
114114
if (array_key_exists('if', $args) && !$args['if']) {

‎src/Caching/Cache.php‎

Lines changed: 4 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -124,10 +124,8 @@ public function bulkLoad(array $keys, ?callable $generator = null): array
124124
$result[$key] = $this->load(
125125
$key,
126126
$generator
127-
? function (&$dependencies) use ($key, $generator) {
128-
return $generator(...[$key, &$dependencies]);
129-
}
130-
: null
127+
? fn(&$dependencies) => $generator(...[$key, &$dependencies])
128+
: null,
131129
);
132130
}
133131

@@ -141,9 +139,7 @@ public function bulkLoad(array $keys, ?callable $generator = null): array
141139
if (isset($cacheData[$storageKey])) {
142140
$result[$key] = $cacheData[$storageKey];
143141
} elseif ($generator) {
144-
$result[$key] = $this->load($key, function (&$dependencies) use ($key, $generator) {
145-
return $generator(...[$key, &$dependencies]);
146-
});
142+
$result[$key] = $this->load($key, fn(&$dependencies) => $generator(...[$key, &$dependencies]));
147143
} else {
148144
$result[$key] = null;
149145
}
@@ -285,9 +281,7 @@ public function call(callable $function)
285281
$key[0][0] = get_class($function[0]);
286282
}
287283

288-
return $this->load($key, function () use ($function, $key) {
289-
return $function(...array_slice($key, 1));
290-
});
284+
return $this->load($key, fn() => $function(...array_slice($key, 1)));
291285
}
292286

293287

‎src/Caching/Storages/MemcachedStorage.php‎

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ public function __construct(
4949
string $host = 'localhost',
5050
int $port = 11211,
5151
string $prefix = '',
52-
?Journal $journal = null
52+
?Journal $journal = null,
5353
) {
5454
if (!static::isAvailable()) {
5555
throw new Nette\NotSupportedException("PHP extension 'memcached' is not loaded.");
@@ -110,9 +110,7 @@ public function read(string $key)
110110

111111
public function bulkRead(array $keys): array
112112
{
113-
$prefixedKeys = array_map(function ($key) {
114-
return urlencode($this->prefix . $key);
115-
}, $keys);
113+
$prefixedKeys = array_map(fn($key) => urlencode($this->prefix . $key), $keys);
116114
$keys = array_combine($prefixedKeys, $keys);
117115
$metas = $this->memcached->getMulti($prefixedKeys);
118116
$result = [];

‎tests/Bridges.Latte/CacheMacro.cache.phpt‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -23,16 +23,16 @@ $params['id'] = 456;
2323

2424
Assert::matchFile(
2525
__DIR__ . '/expected/CacheMacro.cache.phtml',
26-
$latte->compile(__DIR__ . '/templates/cache.latte')
26+
$latte->compile(__DIR__ . '/templates/cache.latte'),
2727
);
2828
Assert::matchFile(
2929
__DIR__ . '/expected/CacheMacro.cache.html',
3030
$latte->renderToString(
3131
__DIR__ . '/templates/cache.latte',
32-
$params
33-
)
32+
$params,
33+
),
3434
);
3535
Assert::matchFile(
3636
__DIR__ . '/expected/CacheMacro.cache.inc.phtml',
37-
file_get_contents($latte->getCacheFile(__DIR__ . strtr('/templates/include.cache.latte', '/', DIRECTORY_SEPARATOR)))
37+
file_get_contents($latte->getCacheFile(__DIR__ . strtr('/templates/include.cache.latte', '/', DIRECTORY_SEPARATOR))),
3838
);

‎tests/Bridges.Latte/CacheMacro.createCache.phpt‎

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -26,9 +26,7 @@ test('', function () {
2626
test('', function () {
2727
$parents = [];
2828
$dp = [Cache::TAGS => ['rum', 'cola']];
29-
$dpFallback = function () use ($dp) {
30-
return $dp;
31-
};
29+
$dpFallback = fn() => $dp;
3230
$outputHelper = CacheMacro::createCache(new DevNullStorage, 'test', $parents);
3331
CacheMacro::endCache($parents, ['dependencies' => $dpFallback]);
3432
Assert::same($dp + [Cache::EXPIRATION => '+ 7 days'], $outputHelper->dependencies);
@@ -40,9 +38,7 @@ test('', function () {
4038
Cache::TAGS => ['rum', 'cola'],
4139
Cache::EXPIRATION => '+ 1 days',
4240
];
43-
$dpFallback = function () use ($dp) {
44-
return $dp;
45-
};
41+
$dpFallback = fn() => $dp;
4642
$outputHelper = CacheMacro::createCache(new DevNullStorage, 'test', $parents);
4743
CacheMacro::endCache($parents, ['dependencies' => $dpFallback]);
4844
Assert::same($dp, $outputHelper->dependencies);

‎tests/Caching/Cache.bulkLoad.phpt‎

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -19,9 +19,7 @@ test('storage without bulk load support', function () {
1919
$cache = new Cache($storage, 'ns');
2020
Assert::same([1 => null, 2 => null], $cache->bulkLoad([1, 2]), 'data');
2121

22-
Assert::same([1 => 1, 2 => 2], $cache->bulkLoad([1, 2], function ($key) {
23-
return $key;
24-
}));
22+
Assert::same([1 => 1, 2 => 2], $cache->bulkLoad([1, 2], fn($key) => $key));
2523

2624
$data = $cache->bulkLoad([1, 2]);
2725
Assert::same(1, $data[1]['data']);
@@ -33,9 +31,7 @@ test('storage with bulk load support', function () {
3331
$cache = new Cache($storage, 'ns');
3432
Assert::same([1 => null, 2 => null], $cache->bulkLoad([1, 2]));
3533

36-
Assert::same([1 => 1, 2 => 2], $cache->bulkLoad([1, 2], function ($key) {
37-
return $key;
38-
}));
34+
Assert::same([1 => 1, 2 => 2], $cache->bulkLoad([1, 2], fn($key) => $key));
3935

4036
$data = $cache->bulkLoad([1, 2]);
4137
Assert::same(1, $data[1]['data']);

‎tests/Caching/Cache.load.phpt‎

Lines changed: 3 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -19,14 +19,10 @@ require __DIR__ . '/Cache.php';
1919
$storage = new TestStorage;
2020
$cache = new Cache($storage, 'ns');
2121

22-
$value = $cache->load('key', function () {
23-
return 'value';
24-
});
22+
$value = $cache->load('key', fn() => 'value');
2523
Assert::same('value', $value);
2624

27-
$data = $cache->load('key', function () {
28-
return "won't load this value"; // will read from storage
29-
});
25+
$data = $cache->load('key', fn() => "won't load this value");
3026
Assert::same('value', $data['data']);
3127

3228

@@ -41,9 +37,7 @@ $value = $cache->load('key', function (&$deps) use ($dependencies) {
4137
});
4238
Assert::same('value', $value);
4339

44-
$data = $cache->load('key', function () {
45-
return "won't load this value"; // will read from storage
46-
});
40+
$data = $cache->load('key', fn() => "won't load this value");
4741
Assert::same('value', $data['data']);
4842
Assert::same($dependencies, $data['dependencies']);
4943

‎tests/Caching/Cache.save.phpt‎

Lines changed: 3 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -31,9 +31,7 @@ Assert::same($dependencies, $res['dependencies']);
3131
$storage = new testStorage;
3232
$cache = new Cache($storage, 'ns');
3333

34-
$cache->save('key', function () {
35-
return 'value';
36-
});
34+
$cache->save('key', fn() => 'value');
3735

3836
$res = $cache->load('key');
3937
Assert::same('value', $res['data']);
@@ -45,9 +43,7 @@ $storage = new testStorage;
4543
$cache = new Cache($storage, 'ns');
4644
$dependencies = [Cache::TAGS => ['tag']];
4745

48-
$cache->save('key', function () {
49-
return 'value';
50-
}, $dependencies);
46+
$cache->save('key', fn() => 'value', $dependencies);
5147

5248
$res = $cache->load('key');
5349
Assert::same('value', $res['data']);
@@ -59,9 +55,7 @@ $storage = new testStorage;
5955
$cache = new Cache($storage, 'ns');
6056
$dependencies = [Cache::EXPIRATION => new DateTime];
6157

62-
$res = $cache->save('key', function () {
63-
return 'value';
64-
}, $dependencies);
58+
$res = $cache->save('key', fn() => 'value', $dependencies);
6559
Assert::same('value', $res);
6660

6761
$res = $cache->load('key');

‎tests/Storages/FileStorage.closure.phpt‎

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -24,18 +24,14 @@ Assert::null($cache->load($key));
2424

2525

2626
// Writing cache using Closure...
27-
$res = $cache->save($key, function () use ($value) {
28-
return $value;
29-
});
27+
$res = $cache->save($key, fn() => $value);
3028

3129
Assert::same($res, $value);
3230

3331
Assert::same($cache->load($key), $value);
3432

3533

3634
// Removing from cache using null callback...
37-
$cache->save($key, function () {
38-
return null;
39-
});
35+
$cache->save($key, fn() => null);
4036

4137
Assert::null($cache->load($key));

0 commit comments

Comments
 (0)