ErrorLoggingFileFetcher::fetchFile() - Code Metrics - Inspection of "Added error logging file fetcher" - JeroenDeDauw/FileFetcher - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#9)

by Jeroen De

created 2017-05-09 16:46 UTC

ErrorLoggingFileFetcher::fetchFile() A

↳ Parent: ErrorLoggingFileFetcher

Complexity

Conditions	2
Paths	2

Size

Total Lines	14
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	8
CRAP Score	2

Importance

Changes

Metric	Value
dl	0
loc	14
ccs	8
cts	8
cp	1
rs	9.4285
c	0
b	0
f	0
cc	2
eloc	9
nc	2
nop	1
crap	2

<?php

declare( strict_types = 1 );

namespace FileFetcher;

use Psr\Log\LoggerInterface;
use Psr\Log\LogLevel;

/**
 * @license GNU GPL v2+
 * @author Gabriel Birke < [email protected] >
 */
class ErrorLoggingFileFetcher implements FileFetcher {

	private $wrappedFileFetcher;
	private $logger;
	private $logLevel;

	public function __construct( FileFetcher $fileFetcher, LoggerInterface $logger, string $logLevel = LogLevel::ERROR ) {
		$this->wrappedFileFetcher = $fileFetcher;
		$this->logger = $logger;
		$this->logLevel = $logLevel;
	}

	/**
	 * @see FileFetcher::fetchFile
	 * @throws FileFetchingException
	 */
	public function fetchFile( string $fileUrl ): string {
		try {
			return $this->wrappedFileFetcher->fetchFile( $fileUrl );
		} catch ( FileFetchingException $e ) {
			$this->logger->log(
				$this->logLevel,
				$e->getMessage(),
				[
					'exception' => $e
				]
			);
			throw $e;
		}
	}

}


1		<?php
2
3		declare( strict_types = 1 );
4
5		namespace FileFetcher;
6
7		use Psr\Log\LoggerInterface;
8		use Psr\Log\LogLevel;
9
10		/**
11		* @license GNU GPL v2+
12		* @author Gabriel Birke < [email protected] >
13		*/
14		class ErrorLoggingFileFetcher implements FileFetcher {
15
16		private $wrappedFileFetcher;
17		private $logger;
18		private $logLevel;
19
20	4	public function __construct( FileFetcher $fileFetcher, LoggerInterface $logger, string $logLevel = LogLevel::ERROR ) {
21	4	$this->wrappedFileFetcher = $fileFetcher;
22	4	$this->logger = $logger;
23	4	$this->logLevel = $logLevel;
24	4	}
25
26		/**
27		* @see FileFetcher::fetchFile
28		* @throws FileFetchingException
29		*/
30	4	public function fetchFile( string $fileUrl ): string {
31		try {
32	4	return $this->wrappedFileFetcher->fetchFile( $fileUrl );
33	3	} catch ( FileFetchingException $e ) {
34	3	$this->logger->log(
35	3	$this->logLevel,
36	3	$e->getMessage(),
37		[
38	3	'exception' => $e
39		]
40		);
41	3	throw $e;
42		}
43		}
44
45		}
46

JeroenDeDauw / FileFetcher

Pull Request — master (#9)

ErrorLoggingFileFetcher::fetchFile() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like