LoggerTrait::levelToInt() - Code Metrics - Inspection of "+ WIPPY WikiLogger" - Daimona/BotRiconferme - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( f946ea...c739e9 )

by Daimona

created 2019-11-12 15:55 UTC

LoggerTrait::levelToInt() A

↳ Parent: LoggerTrait

Complexity

Conditions	1
Paths	1

Size

Total Lines	13
Code Lines	10

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

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

<?php declare( strict_types=1 );

namespace BotRiconferme\Logger;

use Psr\Log\LogLevel;

trait LoggerTrait {
	/**
	 * Translate a LogLevel constant to an integer
	 *
	 * @param string $level
	 * @return int
	 */
	protected 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 );

	}

	/**
	 * @param string $level
	 * @param string $message
	 * @return string
	 */
	protected function getFormattedMessage( string $level, string $message ) {
		return sprintf( "%s [%s] - %s\n", date( 'd M H:i:s' ), $level, $message );
	}
}


1			<?php declare( strict_types=1 );
2
3			namespace BotRiconferme\Logger;
4
5			use Psr\Log\LogLevel;
6
7			trait LoggerTrait {
8			/**
9			* Translate a LogLevel constant to an integer
10			*
11			* @param string $level
12			* @return int
13			*/
14			protected function levelToInt( string $level ) : int {
15			// Order matters
16			$mapping = [
17			LogLevel::DEBUG,
18			LogLevel::INFO,
19			LogLevel::NOTICE,
20			LogLevel::WARNING,
21			LogLevel::ERROR,
22			LogLevel::CRITICAL,
23			LogLevel::ALERT,
24			LogLevel::EMERGENCY
25			];
26			return array_search( $level, $mapping );
			1 ignored issue – show Bug Best Practice introduced 2019-05-03 09:09 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...
27			}
28
29			/**
30			* @param string $level
31			* @param string $message
32			* @return string
33			*/
34			protected function getFormattedMessage( string $level, string $message ) {
35			return sprintf( "%s [%s] - %s\n", date( 'd M H:i:s' ), $level, $message );
36			}
37			}
38

Daimona / BotRiconferme

Push — master ( f946ea...c739e9 )

LoggerTrait::levelToInt() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like