[feature/system-cron] Fixes to make cron actually run.

PHPBB3-9596

phpBB/common.php

@@ -243,5 +243,5 @@ foreach ($cache->obtain_hooks() as $hook)
 if (!$config['use_system_cron'])
 {
 	include($phpbb_root_path . 'includes/cron/cron_manager.' . $phpEx);
-	$cron = new cron();
+	$cron = new cron_manager();
 }

phpBB/cron.php

@@ -48,12 +48,16 @@ function do_cron($run_tasks)
 	garbage_collection();
 }
 
+$cron_lock = new cron_lock;
 if ($cron_lock->lock())
 {
 	if ($config['use_system_cron'])
 	{
 		$use_shutdown_function = false;
 
+		include($phpbb_root_path . 'includes/cron/cron_manager.' . $phpEx);
+		$cron = new cron_manager;
+
 		$run_tasks = $cron->find_all_ready_tasks();
 	}
 	else
@@ -63,13 +67,19 @@ if ($cron_lock->lock())
 
 		output_image();
 
+		// If invalid task is specified, empty $run_tasks is passed to do_cron which then does nothing
+		$run_tasks = array();
 		$task = $cron->find_task($cron_type);
-		if ($task) {
+		if ($task)
-			if ($task->is_parametrized()) {
+		{
+			if ($task->is_parametrized())
+			{
 				$task->parse_parameters($_GET);
 			}
-			if ($task->is_ready()) {
+			if ($task->is_ready())
-				if ($use_shutdown_function && !$task->is_shutdown_function_safe()) {
+			{
+				if ($use_shutdown_function && !$task->is_shutdown_function_safe())
+				{
 					$use_shutdown_function = false;
 				}
 				$run_tasks = array($task);

phpBB/includes/cron/cron_manager.php

@@ -61,7 +61,7 @@ class cron_manager
 		{
 			// ignore ., .. and dot directories
 			// todo: change is_dir to account for symlinks
-			if ($entry[0] == '.' || !is_dir($entry))
+			if ($entry[0] == '.' || !is_dir("$tasks_root_path/$entry"))
 			{
 				continue;
 			}
@@ -76,10 +76,13 @@ class cron_manager
 		{
 			$path = $phpbb_root_path . 'includes/cron/tasks/' . $task_dir;
 			$dir = opendir($path);
-			while (($entry = readdir($dir)) !== false && substr($entry, -$ext_length) == $ext)
+			while (($entry = readdir($dir)) !== false)
 			{
-				$task_file = substr($entry, 0, -$ext_length);
+				if (substr($entry, -$ext_length) == $ext)
-				$task_files[] = array($task_dir, $task_file);
+				{
+					$task_file = substr($entry, 0, -$ext_length);
+					$task_files[] = array($task_dir, $task_file);
+				}
 			}
 			closedir($dir);
 		}
@@ -106,7 +109,7 @@ class cron_manager
 				$class = "cron_task_${mod}_${filename}";
 				if (!class_exists($class))
 				{
-					include($phpbb_root_path . "includes/cron/$mod/$filename.$phpEx");
+					include($phpbb_root_path . "includes/cron/tasks/$mod/$filename.$phpEx");
 				}
 				$object = new $class;
 				$wrapper = new cron_task_wrapper($object);

phpBB/includes/cron/cron_task_base.php

@@ -34,11 +34,6 @@ if (!class_exists('cron_task'))
 */
 abstract class cron_task_base implements cron_task
 {
-	/**
-	* Runs this cron task.
-	*/
-	abstract public function run();
-
 	/**
 	* Returns whether this cron task can run, given current board configuration.
 	*

phpBB/includes/cron/tasks/core/tidy_database.php

@@ -33,6 +33,7 @@ class cron_task_core_tidy_database extends cron_task_base
 	*/
 	public function run()
 	{
+		global $phpbb_root_path, $phpEx;
 		if (!function_exists('tidy_database'))
 		{
 			include($phpbb_root_path . 'includes/functions_admin.' . $phpEx);

phpBB/includes/cron/tasks/core/tidy_search.php

@@ -28,7 +28,7 @@ if (!class_exists('cron_task_base'))
 *
 * @package phpBB3
 */
-class cron_task_core_tidy_sessions extends cron_task_base
+class cron_task_core_tidy_search extends cron_task_base
 {
 	/**
 	* Runs this cron task.