MadWeb\Initializer\Run

Complexity

Total Complexity

12

Size/Duplication

Total Lines	64
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes

0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	artisan()	0	6	1
A	external()	0	6	1
A	callable()	0	6	1
A	dispatch()	0	6	1
A	dispatchNow()	0	6	1
A	pushCommand()	0	4	1
A	getCommands()	0	4	1
B	publish()	0	14	5

<?php

namespace MadWeb\Initializer;

use MadWeb\Initializer\Contracts\Runner;

class Run implements Runner
{
    protected $commands = [];

    public function artisan(string $command, array $arguments = []): Runner
    {
        $this->pushCommand(__FUNCTION__, $command, $arguments);

        return $this;
    }

    public function external(string $command, ...$arguments): Runner
    {
        $this->pushCommand(__FUNCTION__, $command, $arguments);

        return $this;
    }

    public function callable(callable $function, ...$arguments): Runner
    {
        $this->pushCommand(__FUNCTION__, $function, $arguments);

        return $this;
    }

    public function dispatch($job): Runner
    {
        $this->pushCommand(__FUNCTION__, $job);

        return $this;
    }

    public function dispatchNow($job): Runner
    {
        $this->pushCommand(__FUNCTION__, $job);

        return $this;
    }

    public function publish(array $providers): Runner
    {
        foreach ($providers as $provider => $tag) {
            $arguments['--provider'] = is_numeric($provider) ? $tag : $provider;

$arguments was never initialized. Although not strictly required by PHP, it is generally a good practice to add $arguments = array(); before regardless.

            if (! is_numeric($provider) and is_string($tag)) {

Using logical operators such as and instead of && is generally not recommended.

                $arguments['--tag'] = $tag;
            }

            $this->artisan('vendor:publish', $arguments);
        }

        return $this;
    }

    protected function pushCommand(string $type, $command, array $arguments = [])
    {
        $this->commands[] = compact('type', 'command', 'arguments');
    }

    public function getCommands(): array
    {
        return $this->commands;
    }
}