OCA\FaceRecognition\BackgroundJob\FaceRecognitionLogger

Complexity

Total Complexity

11

Size/Duplication

Total Lines	43
Duplicated Lines	0 %

Test Coverage

Coverage

30%

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	getLogger()	0	2	1
A	__construct()	0	7	4
A	logInfo()	0	7	3
A	logDebug()	0	7	3

<?php
/**
 * @copyright Copyright (c) 2017, Matias De lellis <mati86dl@gmail.com>
 * @copyright Copyright (c) 2018, Branko Kokanovic <branko@kokanovic.org>
 *
 * @author Branko Kokanovic <branko@kokanovic.org>
 *
 * @license GNU AGPL version 3 or any later version
 *
 * This program is free software: you can redistribute it and/or modify
 * it under the terms of the GNU Affero General Public License as
 * published by the Free Software Foundation, either version 3 of the
 * License, or (at your option) any later version.
 *
 * This program is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 * GNU Affero General Public License for more details.
 *
 * You should have received a copy of the GNU Affero General Public License
 * along with this program.  If not, see <http://www.gnu.org/licenses/>.
 *
 */
namespace OCA\FaceRecognition\BackgroundJob;

use Psr\Log\LoggerInterface;

use Symfony\Component\Console\Output\OutputInterface;

/**
 * Logger class that encapsulates logging for background tasks. Reason for this
 * class is because background processing can be called as either command, as
 * well as background job, so we need a way to unify logging for both.
 */
class FaceRecognitionLogger {
	/** @var LoggerInterface */
	private $logger;

	/** @var OutputInterface */
	private $output;

	public function __construct($logger) {
		if (method_exists($logger, 'info') && method_exists($logger, 'debug')) {
			$this->logger = $logger;

$logger is of type object, but the property $logger was declared to be of type Psr\Log\LoggerInterface. Are you sure that you always receive this specific sub-class here, or does it make sense to add an instanceof check?

		} else if ($logger instanceof OutputInterface) {
			$this->output = $logger;
		} else {
			throw new \InvalidArgumentException("Logger must be either instance of LoggerInterface or OutputInterface");
		}
	}

	/**
	 * Returns logger, if it is set
	 *
	 * @return LoggerInterface|null Logger, if it is set.
	 */
	public function getLogger() {
		return $this->logger;
	}

	public function logInfo(string $message): void {
		if (!is_null($this->logger)) {
			$this->logger->info($message);
		} else if (!is_null($this->output)) {
			$this->output->writeln($message);
		} else {
			throw new \RuntimeException("There are no configured loggers. Please file an issue at https://github.com/matiasdelellis/facerecognition/issues");
		}
	}

	public function logDebug(string $message): void {
		if (!is_null($this->logger)) {
			$this->logger->debug($message);
		} else if (!is_null($this->output)) {
			$this->output->writeln($message);
		} else {
			throw new \RuntimeException("There are no configured loggers. Please file an issue at https://github.com/matiasdelellis/facerecognition/issues");
		}
	}
}