for testing and deploying your application
for finding and fixing issues
for empowering human code reviews
<?php
namespace MadWeb\Initializer\ExecutorActions;
use Illuminate\Console\Command;
use Illuminate\Container\Container;
use Illuminate\Contracts\Bus\Dispatcher;
class Dispatch
{
private $artisanCommand;
private $job;
private $runNow;
public function __construct(Command $artisanCommand, $job, bool $runNow = false)
$this->artisanCommand = $artisanCommand;
$this->job = $job;
$this->runNow = $runNow;
}
private function title()
return '<comment>Dispatching job:</comment> '.get_class($this->job);
public function __invoke(): bool
return $this->artisanCommand->task($this->title(), function () {
$result = null;
$result
This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently.
$myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; }
Both the $myVar assignment in line 1 and the $higher assignment in line 2 are dead. The first because $myVar is never used and the second because $higher is always overwritten for every possible time line.
$myVar
$higher
if ($this->runNow) {
$result = Container::getInstance()->make(Dispatcher::class)->dispatchNow($this->job);
} else {
$result = Container::getInstance()->make(Dispatcher::class)->dispatch($this->job);
if ($this->artisanCommand->getOutput()->isVerbose()) {
$this->artisanCommand->getOutput()->newLine();
$this->artisanCommand->info($result);
});
mad-web /
laravel-initializer
This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently.
Both the
$myVarassignment in line 1 and the$higherassignment in line 2 are dead. The first because$myVaris never used and the second because$higheris always overwritten for every possible time line.