Comodojo\Extender\Socket\Commands\Scheduler\Get

Complexity

Total Complexity

2

Size/Duplication

Total Lines	28
Duplicated Lines	0 %

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
B	execute()	0	26	2

<?php namespace Comodojo\Extender\Socket\Commands\Scheduler;

use \Comodojo\Daemon\Daemon;
use \Comodojo\Extender\Schedule\Manager;
use \Comodojo\Extender\Socket\Messages\Task\Request as TaskRequestMessage;
use \Comodojo\Extender\Socket\Messages\Scheduler\Schedule as ScheduleMessage;
use \Comodojo\RpcServer\Request\Parameters;

class Get {

    public static function execute(Parameters $params, Daemon $daemon) {

        $manager = new Manager(
            $daemon->getConfiguration(),

The method getConfiguration() does not exist on Comodojo\Daemon\Daemon. It seems like you code against a sub-type of Comodojo\Daemon\Daemon such as Comodojo\Extender\ExtenderDaemon.

            $daemon->getLogger(),
            $daemon->getEvents()
        );

        $id = $params->get('id');
        $name = $params->get('name');

        $schedule = empty($id) ? $manager->getByName($name) :
            $manager->get($id);

        $request = $schedule->getRequest();

The method getRequest() does not exist on null.

        $schedule_message = new ScheduleMessage();
        $schedule_message->setId($schedule->getId());
        $schedule_message->setName($schedule->getName());
        $schedule_message->setDescription($schedule->getDescription());
        $schedule_message->setExpression((string)$schedule->getExpression());
        $schedule_message->setEnabled($schedule->getEnabled());

        $request_message = $request->convertToMessage();

        return [$schedule_message->export(), $request_message->export()];

    }

}