FondBot\Toolbelt\Commands\ServerRun

Complexity

Total Complexity

5

Size/Duplication

Total Lines	54
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	4

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	configure()	0	8	1
A	port()	0	4	1
A	host()	0	4	1
A	serverCommand()	0	9	1
A	handle()	0	6	1

<?php

declare(strict_types=1);

namespace FondBot\Toolbelt\Commands;

use FondBot\Toolbelt\Command;
use Symfony\Component\Console\Input\InputArgument;
use Symfony\Component\Process\PhpExecutableFinder;
use Symfony\Component\Process\ProcessUtils;

class ServerRun extends Command
{
    protected function configure(): void
    {
        $this
            ->setName('serve')
            ->setDescription('Run server')
            ->addOption('host', null, InputArgument::OPTIONAL, 'The host address to serve the application on.', '127.0.0.1')
            ->addOption('port', null, InputArgument::OPTIONAL, 'The port to serve the application on.', '8000');
    }

    /**
     * Get the port for the command.
     *
     * @return string
     */
    protected function port()
    {
        return $this->input->getOption('port');
    }

    /**
     * Get the host for the command.
     *
     * @return string
     */
    protected function host()
    {
        return $this->input->getOption('host');
    }

    /**
     * Get the full server command.
     *
     * @return string
     */
    protected function serverCommand() : string
    {
        return sprintf('%s -S %s:%s %s/public/index.php',
            ProcessUtils::escapeArgument((new PhpExecutableFinder)->find(false)),

It seems like (new \Symfony\Component\Process\PhpExecutableFinder())->find(false) targeting Symfony\Component\Process\PhpExecutableFinder::find() can also be of type false; however, Symfony\Component\Process\ProcessUtils::escapeArgument() does only seem to accept string, did you maybe forget to handle an error condition?

The method Symfony\Component\Process\ProcessUtils::escapeArgument() has been deprecated with message: since version 3.3, to be removed in 4.0. Use a command line array or give env vars to the `Process::start/run()` method instead.

            $this->host(),
            $this->port(),
            ProcessUtils::escapeArgument(path())

The method Symfony\Component\Process\ProcessUtils::escapeArgument() has been deprecated with message: since version 3.3, to be removed in 4.0. Use a command line array or give env vars to the `Process::start/run()` method instead.

        );
    }

    public function handle(): void
    {
        chdir(path());
        $this->line("<info>Fondbot development server started:</info> <http://{$this->host()}:{$this->port()}>");
        passthru($this->serverCommand());
    }

}