Sign upLogin

Mezzle/queuejitsu-scheduler-cli

View on GitHub
Star
src/Command/Scheduler.php

Summary

Maintainability
A
0 mins
Test Coverage

Avoid unused parameters such as '$output'.
Open

    protected function execute(InputInterface $input, OutputInterface $output): int
Severity: Minor
Found in src/Command/Scheduler.php by phpmd

UnusedFormalParameter

Since: 0.2

Avoid passing parameters to methods or constructors and then not using those parameters.

Example

class Foo
{
    private function bar($howdy)
    {
        // $howdy is not used
    }
}

Source https://phpmd.org/rules/unusedcode.html#unusedformalparameter

The method workInBackground() contains an exit expression.
Wontfix

            die('Could not fork worker');
Severity: Minor
Found in src/Command/Scheduler.php by phpmd

ExitExpression

Since: 0.2

An exit-expression within regular code is untestable and therefore it should be avoided. Consider to move the exit-expression into some kind of startup script where an error/exception code is returned to the calling environment.

Example

class Foo {
    public function bar($param)  {
        if ($param === 42) {
            exit(23);
        }
    }
}

Source https://phpmd.org/rules/design.html#exitexpression

The method writePidFile() contains an exit expression.
Open

            die(sprintf('Could not write PID information to %s', $pidfile));
Severity: Minor
Found in src/Command/Scheduler.php by phpmd

ExitExpression

Since: 0.2

An exit-expression within regular code is untestable and therefore it should be avoided. Consider to move the exit-expression into some kind of startup script where an error/exception code is returned to the calling environment.

Example

class Foo {
    public function bar($param)  {
        if ($param === 42) {
            exit(23);
        }
    }
}

Source https://phpmd.org/rules/design.html#exitexpression

There are no issues that match your filters.

Category
Status