Yoshi2889\Tasks\RepeatableTask

Complexity

Total Complexity

9

Size/Duplication

Total Lines	78
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

100%

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	run()	0	13	4
A	cancel()	0	6	1
A	getRepeatInterval()	0	4	1
A	setRepeatInterval()	0	4	1
A	getExpiryTime()	0	4	1

<?php

/**
 * Copyright 2017 NanoSector
 *
 * You should have received a copy of the MIT license with the project.
 * See the LICENSE file for more information.
 */

namespace Yoshi2889\Tasks;


class RepeatableTask implements TaskInterface
{
	/**
	 * @var int
	 */
	protected $repeatInterval = 0;

Protected member variable "repeatInterval" must contain a leading underscore

	/**
	 * @var int
	 */
	protected $expiryTime = 0;

Protected member variable "expiryTime" must contain a leading underscore

	/**
	 * @var TaskInterface
	 */
	protected $childTask;

Protected member variable "childTask" must contain a leading underscore

	/**
	 * RepeatableTask constructor.
	 *
	 * @param TaskInterface $childTask
	 * @param int $repeatInterval
	 */
	public function __construct(TaskInterface $childTask, int $repeatInterval)
	{
		$this->expiryTime = time() + $repeatInterval;
		$this->repeatInterval = $repeatInterval;
		$this->childTask = $childTask;
	}

	/**
	 * @return null|TaskInterface
	 */
	public function run(): ?TaskInterface
	{
		if (time() >= $this->childTask->getExpiryTime())
		{
			$result = $this->childTask->run();

			if ($result instanceof TaskInterface)

It is generally a best practice to always use braces with control structures.

				$this->childTask = $result;
		}

		$this->expiryTime = $this->getExpiryTime() + $this->getRepeatInterval();
		return $this->expiryTime > 0 ? $this : null;
	}

	public function cancel(): void
	{
		$this->childTask->cancel();
		$this->repeatInterval = 0;
		$this->expiryTime = 0;
	}

	/**
	 * @return int
	 */
	public function getRepeatInterval(): int
	{
		return $this->repeatInterval;
	}

	/**
	 * @param int $repeatInterval
	 */
	public function setRepeatInterval(int $repeatInterval)
	{
		$this->repeatInterval = $repeatInterval;
	}

	/**
	 * @return int
	 */
	public function getExpiryTime(): int
	{
		return $this->expiryTime;
	}
}