FileFetcher\PsrCacheFileFetcher

Complexity

Total Complexity

7

Size/Duplication

Total Lines	46
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

100%

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	fetchFile()	0	9	2
A	getFileContentsFromCache()	0	8	2
A	retrieveAndCacheFile()	0	11	2

<?php

declare( strict_types = 1 );

namespace FileFetcher;

use Psr\SimpleCache\CacheException;
use Psr\SimpleCache\CacheInterface;

/**
 * Decorator that caches files using psr/simple-cache.
 * https://packagist.org/packages/psr/simple-cache
 *
 * @since 5.0
 *
 * @licence BSD-3-Clause
 * @author Jeroen De Dauw < jeroendedauw@gmail.com >
 */
class PsrCacheFileFetcher implements FileFetcher {

	private $fileFetcher;
	private $cache;

	public function __construct( FileFetcher $fileFetcher, CacheInterface $cache ) {
		$this->fileFetcher = $fileFetcher;
		$this->cache = $cache;
	}

	/**
	 * @see FileFetcher::fetchFile
	 * @throws FileFetchingException
	 */
	public function fetchFile( string $fileUrl ): string {
		$fileContents = $this->getFileContentsFromCache( $fileUrl );

		if ( $fileContents === null ) {
			return $this->retrieveAndCacheFile( $fileUrl );
		}

		return $fileContents;
	}

	private function getFileContentsFromCache( string $fileUrl ): ?string {
		try {
			return $this->cache->get( $fileUrl );
		}
		catch ( CacheException $ex ) {

The class Psr\SimpleCache\CacheException does not exist. Did you forget a USE statement, or did you not list all dependencies?

			return null;
		}
	}

	private function retrieveAndCacheFile( $fileUrl ): string {
		$fileContents = $this->fileFetcher->fetchFile( $fileUrl );

		try {
			$this->cache->set( $fileUrl, $fileContents );
		}
		catch ( CacheException $ex ) {

Consider adding a comment why this CATCH block is empty.

The class Psr\SimpleCache\CacheException does not exist. Did you forget a USE statement, or did you not list all dependencies?

		}

		return $fileContents;
	}

}