Jabe\Impl\Cmd\DeleteJobsCmd::execute()

Complexity

Conditions	4
Paths	4

Size

Total Lines	20
Code Lines	12

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function execute(CommandContext $commandContext)
    {
        $jobToDelete = null;

The assignment to $jobToDelete is dead and can be removed.

        foreach ($this->jobIds as $jobId) {
            $jobToDelete = Context::getCommandContext()
            ->getJobManager()
            ->findJobById($jobId);

            if ($jobToDelete !== null) {
                // When given job doesn't exist, ignore
                $jobToDelete->delete();

                if ($this->cascade) {
                    $commandContext
                    ->getHistoricJobLogManager()
                    ->deleteHistoricJobLogByJobId($this->jobId);

The property jobId does not exist on Jabe\Impl\Cmd\DeleteJobsCmd. Did you mean jobIds?

                }
            }
        }
        return null;
    }