sonrac\WAMP\Commands\RunCommandInBackground

Complexity

Total Complexity

10

Size/Duplication

Total Lines	86
Duplicated Lines	0 %

Test Coverage

Coverage

91.67%

Importance

Changes	2
Bugs	1	Features	1

6 Methods

Rating	Name	Duplication	Size	Complexity
A	composeForRun()	0	13	3
A	factory()	0	3	1
A	getArtisan()	0	3	2
A	__construct()	0	6	1
A	composeForRunInBackground()	0	3	1
A	runInBackground()	0	5	2

<?php
/**
 * @author Donii Sergii <doniysa@gmail.com>
 */

namespace sonrac\WAMP\Commands;

use Symfony\Component\Process\PhpExecutableFinder;

/**
 * Class RunCommandInBackground
 *
 * @package sonrac\WAMP\Commands
 *
 * @see     https://github.com/dmitry-ivanov/laravel-helper-functions
 */
class RunCommandInBackground
{
    protected $command = null;
    protected $before = null;
    protected $after = null;
    protected $phpBinary = null;

    /**
     * RunCommandInBackground constructor.
     *
     * @param string $command
     * @param null   $before

Are you sure the doc-type for parameter $before is correct as it would always require null to be passed?

     * @param null   $after

Are you sure the doc-type for parameter $after is correct as it would always require null to be passed?

     */
    public function __construct($command, $before = null, $after = null)
    {
        $this->command = $command;
        $this->before = $before;
        $this->after = $after;
        $this->phpBinary = (new PhpExecutableFinder())->find();
    }

    /**
     * Create new command
     *
     * @param string      $command
     * @param string|null $before
     * @param string|null $after
     *
     * @return \sonrac\WAMP\Commands\RunCommandInBackground
     */
    public static function factory($command, $before = null, $after = null)
    {
        return new self($command, $before, $after);

It seems like $after can also be of type string; however, parameter $after of sonrac\WAMP\Commands\RunCommandInBackground::__construct() does only seem to accept null, maybe add an additional type check?

It seems like $before can also be of type string; however, parameter $before of sonrac\WAMP\Commands\RunCommandInBackground::__construct() does only seem to accept null, maybe add an additional type check?

    }

    /**
     * Run command in background and return process pid
     *
     * @return int|null
     */
    public function runInBackground()
    {
        exec($this->composeForRunInBackground(), $out, $pid);

        return count($out) ? (int) $out[0] : null;
    }

    /**
     * Prepare command for run ib background
     *
     * @return string
     */
    protected function composeForRunInBackground()
    {
        return "({$this->composeForRun()}) > /dev/null 2>&1 & echo $!";
    }

    /**
     * Prepare command for run
     *
     * @return string
     */
    protected function composeForRun()
    {
        $parts = [];
        if (!empty($this->before)) {
            $parts[] = (string) $this->before;
        }
        $parts[] = 'cd '.base_path();

The function base_path was not found. Maybe you did not declare it correctly or list all dependencies?

        $parts[] = "{$this->phpBinary} {$this->getArtisan()} {$this->command}";
        if (!empty($this->after)) {
            $parts[] = (string) $this->after;
        }

        return implode(' && ', $parts);
    }

    /**
     * Get artisan
     *
     * @return string
     */
    protected function getArtisan()
    {
        return defined('ARTISAN_BINARY') ? ARTISAN_BINARY : 'artisan';

The constant sonrac\WAMP\Commands\ARTISAN_BINARY was not found. Maybe you did not declare it correctly or list all dependencies?

    }
}