chillerlan / php-log

src/Output/FileLog.php

<?php
/**
 * Class FileLog
 *
 * @filesource   FileLog.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\Settings\SettingsContainerInterface;

class FileLog extends LogOutputAbstract{

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

	/**
	 * FileLog constructor.
	 *
	 * @param \chillerlan\Settings\SettingsContainerInterface|null $options
	 *
	 * @throws \chillerlan\Logger\LogException
	 */
	public function __construct(SettingsContainerInterface $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;
	}

}