Yoshi2889\Tasks\CallbackTask

Complexity

Total Complexity

9

Size/Duplication

Total Lines	89
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Test Coverage

Coverage

100%

Importance

Changes

0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	run()	0	9	2
A	cancel()	0	5	1
A	getCallback()	0	4	1
A	getExpiryTime()	0	4	1
A	setExpiryTime()	0	4	1
A	getStoredArguments()	0	4	1
A	setStoredArguments()	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 CallbackTask implements TaskInterface
{
	/**
	 * @var callable
	 */
	protected $callback = null;

Protected member variable "callback" must contain a leading underscore

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

Protected member variable "expiryTime" must contain a leading underscore

	/**
	 * @var array
	 */
	protected $storedArguments = [];

Protected member variable "storedArguments" must contain a leading underscore

	/**
	 * CallbackTask constructor.
	 *
	 * @param callable $callback
	 * @param int $time
	 * @param array $args
	 */
	public function __construct(callable $callback, int $time, array $args = [])
	{
		$this->callback = $callback;
		$this->setExpiryTime(time() + $time);
		$this->setStoredArguments($args);
	}

	public function run(): ?TaskInterface
	{
		$result = call_user_func_array($this->getCallback(), $this->getStoredArguments());

		if (!($result instanceof TaskInterface))

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

			return null;

		return $result;
	}

	public function cancel(): void
	{
		$this->callback = function () {};
		$this->setExpiryTime(0);
	}

	/**
	 * @return callable
	 */
	public function getCallback(): callable
	{
		return $this->callback;
	}

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

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

	/**
	 * @return array
	 */
	public function getStoredArguments(): array
	{
		return $this->storedArguments;
	}

	/**
	 * @param array $storedArguments
	 */
	public function setStoredArguments(array $storedArguments)
	{
		$this->storedArguments = $storedArguments;
	}


}