chillerlan\Logger\Output\DatabaseLog

Complexity

Total Complexity

3

Size/Duplication

Total Lines	46
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	close()	0	4	1
A	__construct()	0	5	1
A	__log()	0	12	1

<?php
/**
 * Class DatabaseLog
 *
 * @filesource   DatabaseLog.php
 * @created      04.01.2018
 * @package      chillerlan\Logger\Output
 * @author       Smiley <smiley@chillerlan.net>
 * @copyright    2018 Smiley
 * @license      MIT
 */

namespace chillerlan\Logger\Output;

use chillerlan\Database\Database;

The type chillerlan\Database\Database was not found. Maybe you did not declare it correctly or list all dependencies?

use chillerlan\Settings\SettingsContainerInterface;

class DatabaseLog extends LogOutputAbstract{

	/**
	 * @var \chillerlan\Database\Database
	 */
	protected $db;

	/** @noinspection PhpMissingParentConstructorInspection */
	/**
	 * DatabaseLog constructor.
	 *
	 * @param \chillerlan\Settings\SettingsContainerInterface|null $options
	 * @param \chillerlan\Database\Database    $db
	 */
	public function __construct(SettingsContainerInterface $options, Database $db){
		$this->options = $options;

$options is of type chillerlan\Settings\SettingsContainerInterface, but the property $options was declared to be of type chillerlan\Logger\LogOptions. Are you sure that you always receive this specific sub-class here, or does it make sense to add an instanceof check?

		$this->db      = $db;

		$this->db->connect();
	}

	/**
	 * @inheritdoc
	 */
	protected function __log(string $level, string $message, array $context = []){

		// TODO: $context, tests
		$this->db->insert
			->into($this->options->dbLogTable)
			->values([
				'level'   => $level,
				'message' => $message,
				'context' => json_encode($context),
				'time'    => time(),
			])
			->query()
		;
	}

	/**
	 * @inheritdoc
	 */
	public function close():LogOutputInterface{
		$this->db->disconnect();

		return $this;
	}

}