Startwind\Inventorio\Exec\Runner

Complexity

Total Complexity

9

Size/Duplication

Total Lines	53
Duplicated Lines	0 %

Importance

Changes	3
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	getInstance()	0	7	2
A	__construct()	0	7	2
A	run()	0	13	2
A	outputToArray()	0	3	1
A	commandExists()	0	8	2

<?php

namespace Startwind\Inventorio\Exec;

use Symfony\Component\Process\Process;
use function RectorPrefix202308\Symfony\Component\String\s;

class Runner
{
    private bool $timeout;

    private static ?Runner $instance = null;

    static public function getInstance(): Runner
    {
        if (self::$instance === null) {
            self::$instance = new self();
        }

        return self::$instance;

The expression return self::instance could return the type null which is incompatible with the type-hinted return Startwind\Inventorio\Exec\Runner. Consider adding an additional type-check to rule them out.

    }

    public function __construct()
    {
        $process = new Process(['timeout', '--version']);
        if ($process->isSuccessful()) {
            $this->timeout = true;
        } else {
            $this->timeout = false;
        }
    }

    public function run($command, $killAfterSeconds = 5): Process
    {
        if ($this->timeout) {
            $shellCommandLine = "timeout --kill-after=" . $killAfterSeconds . "s 1m " . $command;
        } else {
            $shellCommandLine = $command;
        }

        $process = Process::fromShellCommandline($shellCommandLine);

        $process->run();

        return $process;
    }

    public function commandExists(string $command): bool
    {
        $which = $this->run('which ' . $command)->getOutput();
        if (empty($which)) {
            return false;
        }

        return true;
    }

    public static function outputToArray(string $output): array
    {
        return explode("\n", trim($output));
    }
}