diff options
author | Igor Wiedler <igor@wiedler.ch> | 2012-04-09 00:22:55 +0200 |
---|---|---|
committer | Igor Wiedler <igor@wiedler.ch> | 2012-04-09 12:42:31 +0200 |
commit | 2e76620c8824da62f97cfdaee8f9b1014159fd7c (patch) | |
tree | f5868b1b7209a545bda547138f563b82179ea56f /phpBB/includes/cron/task/provider.php | |
parent | 9165a045c5dee06bb5c163281bb817369a1733a3 (diff) | |
download | forums-2e76620c8824da62f97cfdaee8f9b1014159fd7c.tar forums-2e76620c8824da62f97cfdaee8f9b1014159fd7c.tar.gz forums-2e76620c8824da62f97cfdaee8f9b1014159fd7c.tar.bz2 forums-2e76620c8824da62f97cfdaee8f9b1014159fd7c.tar.xz forums-2e76620c8824da62f97cfdaee8f9b1014159fd7c.zip |
[feature/dic] Rewrite cron system to use DIC
PHPBB3-10739
Diffstat (limited to 'phpBB/includes/cron/task/provider.php')
-rw-r--r-- | phpBB/includes/cron/task/provider.php | 37 |
1 files changed, 21 insertions, 16 deletions
diff --git a/phpBB/includes/cron/task/provider.php b/phpBB/includes/cron/task/provider.php index 1482051699..acc3d455ec 100644 --- a/phpBB/includes/cron/task/provider.php +++ b/phpBB/includes/cron/task/provider.php @@ -15,6 +15,8 @@ if (!defined('IN_PHPBB')) exit; } +use Symfony\Component\DependencyInjection\Container; + /** * Provides cron manager with tasks * @@ -22,27 +24,30 @@ if (!defined('IN_PHPBB')) * * @package phpBB3 */ -class phpbb_cron_task_provider extends phpbb_extension_provider +class phpbb_cron_task_provider implements IteratorAggregate { + private $container; + + public function __construct(Container $container) + { + $this->container = $container; + } + /** - * Finds cron task names using the extension manager. - * - * All PHP files in includes/cron/task/core/ are considered tasks. Tasks - * in extensions have to be located in a directory called cron or a subdir - * of a directory called cron. The class and filename must end in a _task - * suffix. Additionally all PHP files in includes/cron/task/core/ are - * tasks. + * Retrieve an iterator over all items * - * @return array List of task names + * @return ArrayIterator An iterator for the array of cron tasks */ - protected function find() + public function getIterator() { - $finder = $this->extension_manager->get_finder(); + $definitions = $this->container->findTaggedServiceIds('cron.task'); + + $tasks = array(); + foreach ($definitions as $name => $definition) + { + $tasks[] = $this->container->get($name); + } - return $finder - ->extension_suffix('_task') - ->extension_directory('/cron') - ->core_path('includes/cron/task/core/') - ->get_classes(); + return new ArrayIterator($tasks); } } |