ovidiul / XCloner-Wordpress

includes/class-xcloner-logger.php

<?php

use Monolog\Logger;
use Monolog\Handler\StreamHandler;
use Monolog\Handler\RotatingFileHandler;

class Xcloner_Logger extends Logger {

	private $logger_path;
	private $max_logger_files = 7;
	private $main_logger_url;

	/**
	 * Xcloner_Logger constructor.
	 * @param Xcloner $xcloner_container
	 * @param string $logger_name
	 * @throws Exception
	 */
	public function __construct(Xcloner $xcloner_container, $logger_name = "xcloner_logger") {
		if (!$xcloner_container->get_xcloner_settings()) {
			$xcloner_settings = new Xcloner_Settings($xcloner_container);
		} else {
			$xcloner_settings = $xcloner_container->get_xcloner_settings();
		}

		$hash = $xcloner_settings->get_hash();
		if ($hash == "-".$xcloner_settings->get_server_unique_hash(5)) {
			$hash = "";
		}

		$logger_path     = $xcloner_settings->get_xcloner_store_path().DS.$xcloner_settings->get_logger_filename();
		$logger_path_tmp = "";

		if ($hash) {
			$logger_path_tmp = $xcloner_settings->get_xcloner_tmp_path().DS.$xcloner_settings->get_logger_filename(1);
		}

		$this->logger_path = $logger_path;

		if (!is_dir($xcloner_settings->get_xcloner_store_path()) or !is_writable($xcloner_settings->get_xcloner_store_path())) {
			$logger_path     = 'php://stderr';
			$logger_path_tmp = "";
		}

		if (!$xcloner_settings->get_xcloner_option('xcloner_enable_log')) {
			$logger_path     = 'php://stderr';
			$logger_path_tmp = "";
		}

		// create a log channel
		parent::__construct($logger_name);

		$debug_level = Logger::INFO;

		if (WP_DEBUG) {
			$debug_level = Logger::DEBUG;
		}


		if ($logger_path) {
			if (!$xcloner_settings->get_xcloner_option('xcloner_enable_log')) {
				$stream = new StreamHandler($logger_path, $debug_level);
			} else {
				$stream = new RotatingFileHandler($logger_path, $this->max_logger_files, $debug_level);
			}

			$this->pushHandler($stream);

			$this->main_logger_url = $stream->getUrl();
		}

		if ($hash and $logger_path_tmp) {
			$this->pushHandler(new StreamHandler($logger_path_tmp, $debug_level));
		}

		//return $this;
	}

	/**
	 * @return string|null
	 */
	function get_main_logger_url() {

It is generally recommended to explicitly declare the visibility for methods.

		return $this->main_logger_url;
	}

	/**
	 * @param int $totalLines
	 * @return array|bool
	 */
	function getLastDebugLines($totalLines = 200) {

It is generally recommended to explicitly declare the visibility for methods.

		$lines = array();

		if (!file_exists($this->main_logger_url) or !is_readable($this->main_logger_url)) {
			return false;
		}

		$fp = fopen($this->main_logger_url, 'r');
		fseek($fp, - 1, SEEK_END);

It seems like $fp can also be of type false; however, parameter $handle of fseek() does only seem to accept resource, maybe add an additional type check?

		$pos      = ftell($fp);

It seems like $fp can also be of type false; however, parameter $handle of ftell() does only seem to accept resource, maybe add an additional type check?

		$lastLine = "";

		// Loop backword until we have our lines or we reach the start
		while ($pos > 0 && count($lines) < $totalLines) {

			$C = fgetc($fp);

It seems like $fp can also be of type false; however, parameter $handle of fgetc() does only seem to accept resource, maybe add an additional type check?

			if ($C == "\n") {
				// skip empty lines
				if (trim($lastLine) != "") {
					$lines[] = $lastLine;
				}
				$lastLine = '';
			} else {
				$lastLine = $C.$lastLine;
			}
			fseek($fp, $pos--);
		}

		$lines = array_reverse($lines);

		return $lines;
	}
}