CuyZ\Notiz\Service\Scheduler\Scheduler

Complexity

Total Complexity

3

Size/Duplication

Total Lines	48
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	executeTask()	0	22	2
A	__construct()	0	5	1

<?php
declare(strict_types=1);

/*
 * Copyright (C)
 * Nathan Boiron <nathan.boiron@gmail.com>
 * Romain Canon <romain.hydrocanon@gmail.com>
 *
 * This file is part of the TYPO3 NotiZ project.
 * It is free software; you can redistribute it and/or modify it
 * under the terms of the GNU General Public License, either
 * version 3 of the License, or any later version.
 *
 * For the full copyright and license information, see:
 * http://www.gnu.org/licenses/gpl-3.0.html
 */

namespace CuyZ\Notiz\Service\Scheduler;

use CuyZ\Notiz\Service\Container;
use Throwable;
use TYPO3\CMS\Extbase\SignalSlot\Dispatcher;

The type TYPO3\CMS\Extbase\SignalSlot\Dispatcher was not found. Maybe you did not declare it correctly or list all dependencies?

use TYPO3\CMS\Scheduler\Task\AbstractTask;

The type TYPO3\CMS\Scheduler\Task\AbstractTask was not found. Maybe you did not declare it correctly or list all dependencies?

/**
 * Overrides the TYPO3 core class in order to send signals on which NotiZ can
 * bind events.
 *
 * Two signals are sent:
 *
 * Signal: task was executed
 * -------------------------
 *
 * @see \CuyZ\Notiz\Service\Scheduler\Scheduler::SIGNAL_TASK_EXECUTED
 *
 * When a task was successfully executed.
 *
 * Two arguments are given to the slot:
 *
 * - `$task` - @see \TYPO3\CMS\Scheduler\Task\AbstractTask
 *   The executed task instance containing useful information.
 *
 * - `$result` - bool
 *   Result returned by the task execution process.
 *
 * Signal: task execution failed
 * -----------------------------
 *
 * @see \CuyZ\Notiz\Service\Scheduler\Scheduler::SIGNAL_TASK_FAILED
 *
 * When something has gone wrong during the execution of the task (an exception
 * has been thrown).
 *
 * Two arguments are given to the slot:
 *
 * - `$task` - @see \TYPO3\CMS\Scheduler\Task\AbstractTask
 *   The executed task instance containing useful information.
 *
 * - `$exception` - @see \Throwable
 *   The exception that was thrown during the execution.
 */
class Scheduler extends \TYPO3\CMS\Scheduler\Scheduler

The type TYPO3\CMS\Scheduler\Scheduler was not found. Maybe you did not declare it correctly or list all dependencies?

{
    const SIGNAL_TASK_EXECUTED = 'taskWasExecuted';

    const SIGNAL_TASK_FAILED = 'taskExecutionFailed';

    /**
     * @var Dispatcher
     */
    protected $dispatcher;

    public function __construct()
    {
        parent::__construct();

        $this->dispatcher = Container::get(Dispatcher::class);
    }

    /**
     * See class description for more information.
     *
     * @param AbstractTask $task
     * @return bool
     *
     * @throws Throwable
     */
    public function executeTask(AbstractTask $task): bool
    {
        $exception = null;

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

        try {
            $result = parent::executeTask($task);

            $this->dispatcher->dispatch(
                __CLASS__,
                self::SIGNAL_TASK_EXECUTED,
                [$task, $result]
            );

            return $result;
        } catch (Throwable $exception) {
            $this->dispatcher->dispatch(
                __CLASS__,
                self::SIGNAL_TASK_FAILED,
                [$task, $exception]
            );

            throw $exception;
        }
    }
}