chillerlan\Logger\Output\Logfile

Complexity

Total Complexity

6

Size/Duplication

Total Lines	50
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__log()	0	2	1
A	__construct()	0	10	2
A	close()	0	3	1
A	__destruct()	0	5	2

<?php
/**
 * Class Logfile
 *
 * @filesource   Logfile.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\Logger\LogException;
use chillerlan\Traits\ContainerInterface;

/**
 */
class Logfile extends LogOutputAbstract{

	/**
	 * @var bool|resource
	 */
	protected $fh;

	/**
	 * Logfile constructor.
	 *
	 * @param \chillerlan\Traits\ContainerInterface|null $options
	 *
	 * @throws \chillerlan\Logger\LogException
	 */
	public function __construct(ContainerInterface $options = null){
		parent::__construct($options);

		if(!is_writable($this->options->logfileDir)){
			throw new LogException('log file directory inaccessible');
		}

		$logfile = $this->options->logfileDir.'/'.$this->options->logfileName.'-'.date('Y-m-d').'.'.$this->options->logfileExt;

		$this->fh = fopen($logfile, 'a+');
	}

	/**
	 * @inheritdoc
	 */
	public function __destruct(){
		$this->close();

		if(gettype($this->fh) === 'resource'){
			fclose($this->fh);
		}
	}

	/**
	 * @inheritdoc
	 */
	protected function __log(string $level, string $message, array $context = null){
		fwrite($this->fh, $this->message($level, $message, $context));

It seems like $this->fh can also be of type boolean; however, parameter $handle of fwrite() does only seem to accept resource, maybe add an additional type check?

	}

	/**
	 * @inheritdoc
	 */
	public function close():LogOutputInterface{
		$this->__log('log closed', '~~~');
		return $this;
	}

}