phpbb\mediaembed\cron\purge_cache

Complexity

Total Complexity

5

Size/Duplication

Total Lines	53
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	run()	0	12	2
A	is_runnable()	0	4	1
A	should_run()	0	4	1

<?php
/**
 *
 * phpBB Media Embed PlugIn extension for the phpBB Forum Software package.
 *
 * @copyright (c) 2020 phpBB Limited <https://www.phpbb.com>
 * @license GNU General Public License, version 2 (GPL-2.0)
 *
 */

namespace phpbb\mediaembed\cron;

use \phpbb\config\config;
use \phpbb\mediaembed\cache\cache;

/**
 * Mediaembed cron task.
 */
class purge_cache extends \phpbb\cron\task\base
{
	/** @var config $config */
	protected $config;

	/** @var purge $mediaembed_cache */
	protected $mediaembed_cache;

	/**
	 * Constructor
	 *
	 * @param config       $config           Config object
	 * @param purge        $mediaembed_cache Mediaembed cache object

Should the type for parameter $mediaembed_cache not be cache?

	 * @access public
	 */
	public function __construct(config $config, cache $mediaembed_cache)
	{
		$this->config = $config;
		$this->mediaembed_cache = $mediaembed_cache;

It seems like $mediaembed_cache of type object<phpbb\mediaembed\cache\cache> is incompatible with the declared type object<phpbb\mediaembed\cron\purge> of property $mediaembed_cache.

	}

	/**
	 * {@inheritDoc}
	 */
	public function run()
	{
		try
		{
			$this->mediaembed_cache->purge();
			$this->config->set('mediaembed_last_gc', time(), false);
		}
		catch (\RuntimeException $e)
		{
			return;
		}
	}

	/**
	 * {@inheritDoc}
	 */
	public function is_runnable()
	{
		return (bool) $this->config['media_embed_enable_cache'];
	}

	/**
	 * {@inheritDoc}
	 */
	public function should_run()
	{
		return $this->config['mediaembed_last_gc'] < strtotime('24 hours ago');
	}
}