BotRiconferme\SimpleLogger

Complexity

Total Complexity

4

Size/Duplication

Total Lines	39
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	2	1
A	levelToInt()	0	13	1
A	log()	0	3	2

<?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 );

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.

	}

	/**
	 * @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 );
		}
	}
}