SimpleLogger::__construct() - Code Metrics - Inspection of "Rename Logger -> SimpleLogger" - Daimona/BotRiconferme - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( c84974...1a8235 )

by Daimona

created 2019-11-11 14:57 UTC

SimpleLogger::__construct() A

↳ Parent: SimpleLogger

Complexity

Conditions	1
Paths	1

Size

Total Lines	2
Code Lines	1

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	1
eloc	1
c	1
b	0
f	0
nc	1
nop	1
dl	0
loc	2
rs	10

<?php declare( strict_types=1 );

namespace BotRiconferme;

use Psr\Log\AbstractLogger;
use Psr\Log\LogLevel;

/**
 * Implementation for a PSR-3 logger
 */
class SimpleLogger extends AbstractLogger {
	/** @var int */
	private $minLevel;

	/**
	 * @param string $minlevel
	 */
	public function __construct( $minlevel = LogLevel::INFO ) {
		$this->minLevel = $this->levelToInt( $minlevel );
	}

	/**
	 * Translate a LogLevel constant to an integer
	 *
	 * @param string $level
	 * @return int
	 */
	private function levelToInt( string $level ) : int {
		// Order matters
		$mapping = [
			LogLevel::DEBUG,
			LogLevel::INFO,
			LogLevel::NOTICE,
			LogLevel::WARNING,
			LogLevel::ERROR,
			LogLevel::CRITICAL,
			LogLevel::ALERT,
			LogLevel::EMERGENCY
		];
		return array_search( $level, $mapping );

	}

	/**
	 * @inheritDoc
	 * @suppress PhanUnusedPublicMethodParameter
	 */
	public function log( $level, $message, array $context = [] ) {
		if ( $this->levelToInt( $level ) >= $this->minLevel ) {
			printf( "%s [%s] - %s\n", date( 'd M H:i:s' ), $level, $message );
		}
	}
}


1			<?php declare( strict_types=1 );
2
3			namespace BotRiconferme;
4
5			use Psr\Log\AbstractLogger;
6			use Psr\Log\LogLevel;
7
8			/**
9			* Implementation for a PSR-3 logger
10			*/
11			class SimpleLogger extends AbstractLogger {
12			/** @var int */
13			private $minLevel;
14
15			/**
16			* @param string $minlevel
17			*/
18			public function __construct( $minlevel = LogLevel::INFO ) {
19			$this->minLevel = $this->levelToInt( $minlevel );
20			}
21
22			/**
23			* Translate a LogLevel constant to an integer
24			*
25			* @param string $level
26			* @return int
27			*/
28			private function levelToInt( string $level ) : int {
29			// Order matters
30			$mapping = [
31			LogLevel::DEBUG,
32			LogLevel::INFO,
33			LogLevel::NOTICE,
34			LogLevel::WARNING,
35			LogLevel::ERROR,
36			LogLevel::CRITICAL,
37			LogLevel::ALERT,
38			LogLevel::EMERGENCY
39			];
40			return array_search( $level, $mapping );
			0 ignored issues – show Bug Best Practice introduced 2019-11-11 14:59 UTC by Report Bug Copy Issue Report The expression `return array_search($level, $mapping)` could return the type `false\|string` which is incompatible with the type-hinted return `integer`. Consider adding an additional type-check to rule them out. Loading history...
41			}
42
43			/**
44			* @inheritDoc
45			* @suppress PhanUnusedPublicMethodParameter
46			*/
47			public function log( $level, $message, array $context = [] ) {
48			if ( $this->levelToInt( $level ) >= $this->minLevel ) {
49			printf( "%s [%s] - %s\n", date( 'd M H:i:s' ), $level, $message );
50			}
51			}
52			}
53

Daimona / BotRiconferme

Push — master ( c84974...1a8235 )

SimpleLogger::__construct() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like