coding style: fixes in code

Author: dg

src/Caching/Cache.php

@@ -176,7 +176,7 @@ public function save($key, $data, array $dependencies = null)
 			$this->storage->remove($key);
 		} else {
 			$dependencies = $this->completeDependencies($dependencies);
-			if (isset($dependencies[Cache::EXPIRATION]) && $dependencies[Cache::EXPIRATION] <= 0) {
+			if (isset($dependencies[self::EXPIRATION]) && $dependencies[self::EXPIRATION] <= 0) {
 				$this->storage->remove($key);
 			} else {
 				$this->storage->write($key, $data, $dependencies);

tests/Caching/Cache.php

@@ -41,7 +41,7 @@ public function clean(array $conditions): void
 
 class BulkReadTestStorage extends TestStorage implements IBulkReader
 {
-	function bulkRead(array $keys): array
+	public function bulkRead(array $keys): array
 	{
 		$result = [];
 		foreach ($keys as $key) {

tests/Storages/FileStorage.call.phpt

@@ -16,14 +16,14 @@ require __DIR__ . '/../bootstrap.php';
 
 class Mock
 {
-	function mockFunction($x, $y)
+	public function mockFunction($x, $y)
 	{
 		$GLOBALS['called'] = true;
 		return $x + $y;
 	}
 
 
-	function __sleep()
+	public function __sleep()
 	{
 		throw new Exception;
 	}

tests/Storages/FileStorage.tags.phpt

@@ -57,8 +57,8 @@ $cache->clean([
 		0 => 'non-existent1',
 		1 => 'non-existent2',
 		3 => 'one',
-		5 => 'non-existent3'
-	]
+		5 => 'non-existent3',
+	],
 ]);
 
 Assert::null($cache->load('key1'));

tests/Storages/FileStorage.wrap.phpt

@@ -23,7 +23,7 @@ function mockFunction($x, $y)
 
 class Test
 {
-	function mockMethod($x, $y)
+	public function mockMethod($x, $y)
 	{
 		$GLOBALS['called'] = true;
 		return $x + $y;

tests/Storages/IJournalTestCase.php

@@ -164,7 +164,7 @@ final public function testDifferentCleaning()
 	final public function testSpecialChars()
 	{
 		$this->journal->write('ok_test7ščřžýáíé', [
-			Cache::TAGS => ['čšřýýá', 'ýřžčýž']
+			Cache::TAGS => ['čšřýýá', 'ýřžčýž'],
 		]);
 
 		Assert::same([
@@ -176,10 +176,10 @@ final public function testSpecialChars()
 	final public function testDuplicatedSameTags()
 	{
 		$this->journal->write('ok_test_a', [
-			Cache::TAGS => ['homepage']
+			Cache::TAGS => ['homepage'],
 		]);
 		$this->journal->write('ok_test_a', [
-			Cache::TAGS => ['homepage']
+			Cache::TAGS => ['homepage'],
 		]);
 		Assert::same([
 			'ok_test_a',
@@ -190,11 +190,11 @@ final public function testDuplicatedSameTags()
 	final public function testDuplicatedSamePriority()
 	{
 		$this->journal->write('ok_test_b', [
-			Cache::PRIORITY => 12
+			Cache::PRIORITY => 12,
 		]);
 
 		$this->journal->write('ok_test_b', [
-			Cache::PRIORITY => 12
+			Cache::PRIORITY => 12,
 		]);
 
 		Assert::same([
@@ -206,11 +206,11 @@ final public function testDuplicatedSamePriority()
 	final public function testDuplicatedDifferentTags()
 	{
 		$this->journal->write('ok_test_ba', [
-			Cache::TAGS => ['homepage']
+			Cache::TAGS => ['homepage'],
 		]);
 
 		$this->journal->write('ok_test_ba', [
-			Cache::TAGS => ['homepage2']
+			Cache::TAGS => ['homepage2'],
 		]);
 
 		Assert::same([
@@ -225,11 +225,11 @@ final public function testDuplicatedDifferentTags()
 	final public function testDuplicatedTwoDifferentTags()
 	{
 		$this->journal->write('ok_test_baa', [
-			Cache::TAGS => ['homepage', 'aąa']
+			Cache::TAGS => ['homepage', 'aąa'],
 		]);
 
 		$this->journal->write('ok_test_baa', [
-			Cache::TAGS => ['homepage2', 'aaa']
+			Cache::TAGS => ['homepage2', 'aaa'],
 		]);
 
 		Assert::same([
@@ -244,11 +244,11 @@ final public function testDuplicatedTwoDifferentTags()
 	final public function testDuplicatedDifferentPriorities()
 	{
 		$this->journal->write('ok_test_bb', [
-			Cache::PRIORITY => 10
+			Cache::PRIORITY => 10,
 		]);
 
 		$this->journal->write('ok_test_bb', [
-			Cache::PRIORITY => 20
+			Cache::PRIORITY => 20,
 		]);
 
 		Assert::same([
@@ -263,7 +263,7 @@ final public function testDuplicatedDifferentPriorities()
 	final public function testCleanAll()
 	{
 		$this->journal->write('ok_test_all_tags', [
-			Cache::TAGS => ['test:all', 'test:all']
+			Cache::TAGS => ['test:all', 'test:all'],
 		]);
 
 		$this->journal->write('ok_test_all_priority', [