Shlinkio\Shlink\CLI\Command\Db\AbstractDatabaseCommand

Complexity

Total Complexity

4

Size/Duplication

Total Lines	23
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	runPhpCommand()	0	4	1
A	getLockConfig()	0	3	1
A	__construct()	0	5	2

<?php

declare(strict_types=1);

namespace Shlinkio\Shlink\CLI\Command\Db;

use Shlinkio\Shlink\CLI\Command\Util\AbstractLockedCommand;
use Shlinkio\Shlink\CLI\Command\Util\LockedCommandConfig;
use Symfony\Component\Console\Helper\ProcessHelper;
use Symfony\Component\Console\Output\OutputInterface;
use Symfony\Component\Lock\Factory as Locker;
use Symfony\Component\Process\PhpExecutableFinder;

use function array_unshift;

abstract class AbstractDatabaseCommand extends AbstractLockedCommand
{
    /** @var ProcessHelper */
    private $processHelper;
    /** @var string */
    private $phpBinary;

    public function __construct(Locker $locker, ProcessHelper $processHelper, PhpExecutableFinder $phpFinder)
    {
        parent::__construct($locker);
        $this->processHelper = $processHelper;
        $this->phpBinary = $phpFinder->find(false) ?: 'php';
    }

    protected function runPhpCommand(OutputInterface $output, array $command): void
    {
        array_unshift($command, $this->phpBinary);
        $this->processHelper->mustRun($output, $command);
    }

    protected function getLockConfig(): LockedCommandConfig
    {
        return new LockedCommandConfig($this->getName(), true);

It seems like $this->getName() can also be of type null; however, parameter $lockName of Shlinkio\Shlink\CLI\Command\Util\LockedCommandConfig::__construct() does only seem to accept string, maybe add an additional type check?

    }
}