Cache::save() with closure is deprecated (BC break)

It was internal usage

Author: dg

src/Caching/Cache.php

@@ -169,6 +169,7 @@ public function save($key, $data, array $dependencies = null)
 		$key = $this->generateKey($key);
 
 		if ($data instanceof \Closure) {
+			trigger_error(__METHOD__ . '() closure argument is deprecated.', E_USER_WARNING);
 			$this->storage->lock($key);
 			try {
 				$data = $data(...[&$dependencies]);

tests/Caching/Cache.save.phpt

@@ -31,7 +31,7 @@ Assert::same($dependencies, $res['dependencies']);
 $storage = new testStorage;
 $cache = new Cache($storage, 'ns');
 
-$cache->save('key', function () {
+@$cache->save('key', function () { // @ deprecated
 	return 'value';
 });
 
@@ -45,7 +45,7 @@ $storage = new testStorage;
 $cache = new Cache($storage, 'ns');
 $dependencies = [Cache::TAGS => ['tag']];
 
-$cache->save('key', function () {
+@$cache->save('key', function () { // @ deprecated
 	return 'value';
 }, $dependencies);
 
@@ -59,7 +59,7 @@ $storage = new testStorage;
 $cache = new Cache($storage, 'ns');
 $dependencies = [Cache::EXPIRATION => new DateTime];
 
-$res = $cache->save('key', function () {
+@$res = $cache->save('key', function () { // @ deprecated
 	return 'value';
 }, $dependencies);
 Assert::same('value', $res);

tests/Storages/FileStorage.closure.phpt

@@ -24,7 +24,7 @@ Assert::null($cache->load($key));
 
 
 // Writing cache using Closure...
-$res = $cache->save($key, function () use ($value) {
+$res = @$cache->save($key, function () use ($value) { // @ deprecated
 	return $value;
 });
 
@@ -34,7 +34,7 @@ Assert::same($cache->load($key), $value);
 
 
 // Removing from cache using null callback...
-$cache->save($key, function () {
+@$cache->save($key, function () { // @ deprecated
 	return null;
 });