Webtorque\DevTaskRunner\DevTaskRunnerCronTask::process()

Complexity

Conditions	5
Paths	3

Size

Total Lines

35

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function process()
    {
        $nextTask = DevTaskRun::get_next_task();
        if ($nextTask) {
            //create task instance
            $task = Injector::inst()->create($nextTask->Task);

            //get params
            $params = explode(' ', $nextTask->Params);
            $paramList = array();
            if ($params) {

The expression $params of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

                foreach ($params as $param) {
                    $parts = explode('=', $param);
                    if (count($parts) === 2) {
                        $paramList[$parts[0]] = $parts[1];
                    }
                }
            }

            //set starting flag
            $nextTask->Status = 'Running';
            $nextTask->write();
            echo 'Starting task ' . $task->getTitle() . "\n";

            //execute task
            $request = new HTTPRequest('GET', 'dev/tasks/' . $nextTask->Task, $paramList);
            $task->run($request);

            //set finished flag
            $nextTask->Status = 'Finished';
            $nextTask->FinishDate = DBDatetime::now()->getValue();
            $nextTask->write();
            echo 'Finished task ' . $task->getTitle() . "\n";
        }
    }
}