Oliverde8\PhpEtlBundle\Model\ExecutionContext

Complexity

Total Complexity

7

Size/Duplication

Total Lines	27
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	finalise()	0	16	6
A	__construct()	0	5	1

<?php

namespace Oliverde8\PhpEtlBundle\Model;

use Monolog\Logger;
use Oliverde8\Component\PhpEtl\Model\File\FileSystemInterface;
use Oliverde8\Component\PhpEtl\Model\File\LocalFileSystem;
use Psr\Log\LoggerInterface;

class ExecutionContext extends \Oliverde8\Component\PhpEtl\Model\ExecutionContext
{
    protected string $workDir;

    public function __construct(array $parameters, FileSystemInterface $fileSystem, LoggerInterface $logger, string $logPath)
    {
        parent::__construct($parameters, $fileSystem);
        $this->logger = $logger;
        $this->workDir = $logPath;
    }

    protected function finalise(): void
    {
        if ($this->logger instanceof Logger) {
            foreach ($this->logger->getHandlers() as $handler) {

The method getHandlers() does not exist on null.

The method getHandlers() does not exist on Psr\Log\LoggerInterface. It seems like you code against a sub-type of Psr\Log\LoggerInterface such as Monolog\Logger.

                $handler->close();
            }
        }

        if ($this->fileSystem instanceof LocalFileSystem && $this->fileSystem->getRootPath() == $this->workDir) {
            // Local file system needs no moving of the log file.
            return;
        }

        $logPath = $this->workDir . "/execution.log";
        if(file_exists($logPath)) {
            $this->fileSystem->writeStream("execution.log", fopen($logPath, 'r'));
        }
    }
}