Jarobe\TaskRunnerBundle\Processor\Processor::process()

Complexity

Conditions	4
Paths	4

Size

Total Lines	36
Code Lines	21

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function process(TaskTypeInterface $task)
    {
        $taskResult = new TaskResult();

        try {
            $driver = $this->driverFactory->getDriverForTask($task);
        } catch (TaskException $e) {
            $exceptionMessage = sprintf("TaskException: %s", $e->getMessage());
            $taskResult->setSuccess(false)
                ->setErrors([$exceptionMessage])
            ;
            return $taskResult;
        }


        //See if there's any issues with the tasks before running.
        $validationErrors = $driver->canRun($task);
        if (count($validationErrors) > 0) {
            $taskResult->setSuccess(false)
                ->setErrors($validationErrors)

It seems like $validationErrors defined by $driver->canRun($task) on line 42 can also be of type null; however, Jarobe\TaskRunnerBundle\Model\TaskResult::setErrors() does only seem to accept array, maybe add an additional type check?

            ;
            return $taskResult;
        }

        //Run the task, and then check for errors
        $errors = $driver->run($task);
        if (count($errors) > 0) {
            $taskResult->setSuccess(false)
                ->setErrors($errors)

It seems like $errors defined by $driver->run($task) on line 51 can also be of type null; however, Jarobe\TaskRunnerBundle\Model\TaskResult::setErrors() does only seem to accept array, maybe add an additional type check?

            ;
            return $taskResult;
        }

        $taskResult->setSuccess(true);
        return $taskResult;
    }
}