'. t('The Poormanscron module runs cron jobs without the need of the cron application.') .'
'; - case 'admin/settings/poormanscron': - return ''. t('The settings provided here allow you to administer Poormancron.') .'
'; - } +if (!defined('DRUPAL_CRON_DEFAULT_THRESHOLD')) { + define('DRUPAL_CRON_DEFAULT_THRESHOLD', 10800); +} + +if (!defined('REQUEST_TIME')) { + define('REQUEST_TIME', time()); } /** - * Implementation of hook_exit(). - * - * Checks if poormanscron needs to be run. If this is the case, it invokes - * the cron hooks of all enabled modules, which are executed after - * all HTML is returned to the browser. So the user who kicks off the cron - * jobs should not notice any delay. - */ -function poormanscron_exit() { - - // Calculate when the next poormanscron run is due. - $lastrun = variable_get('poormanscron_lastrun', 0); - $nextrun = $lastrun + 60 * variable_get('poormanscron_interval', 60); - - // If the configured time has passed, start the next poormanscron run. - if (time() > $nextrun) { - - // If this cron run fails to complete, wait a few minutes before retrying. - variable_set('poormanscron_lastrun', - $lastrun + 60 * variable_get('poormanscron_retry_interval', 10)); - - // Get the current Drupal messages. Use drupal_set_message() so that - // the messages aren't deleted in case the cron run fails and - // we don't get a chance to restore them below. - $saved_messages = drupal_set_message(); - - // Invoke the cron hooks of all enabled modules. - if (drupal_cron_run()) { - $message = 'Cron run completed (via poormanscron).'; - } - else { - $message = 'Cron run unsuccessful (via poormanscron).'; - } + * Implementation of hook_menu(). + */ +function poormanscron_menu() { + $items['poormanscron/run-cron-check'] = array( + 'title' => 'Execute cron', + 'page callback' => 'poormanscron_run_cron_check', + 'access callback' => 'poormanscron_run_cron_check_access', + 'type' => MENU_CALLBACK, + ); + return $items; +} - // Write a message to the logs if the user wants us to do so. - if (variable_get('poormanscron_log_cron_runs', 1) == 1) { - watchdog('cron', $message); - } +/** + * Implementation of hook_init(). + */ +function poormanscron_init() { + if (poormanscron_run_cron_check_access()) { + drupal_add_js(drupal_get_path('module', 'poormanscron') . '/poormanscron.js'); + drupal_add_js(array('cronCheck' => REQUEST_TIME + variable_get('cron_safe_threshold', DRUPAL_CRON_DEFAULT_THRESHOLD)), 'setting'); + } +} - $t = time(); +/** + * Checks if the feature to automatically run cron is enabled. + * + * Also used as a menu access callback for this feature. + * + * @return + * TRUE if cron threshold is enabled, FALSE otherwise. + * + * @see poormanscron_run_cron_check() + */ +function poormanscron_run_cron_check_access() { + return variable_get('cron_safe_threshold', DRUPAL_CRON_DEFAULT_THRESHOLD) > 0; +} - // Update the time of the last poormanscron run (this one). - variable_set('poormanscron_lastrun', $t); +/** + * Menu callback; executes cron via an image callback. + * + * This callback runs cron in a separate HTTP request to prevent "mysterious" + * slow-downs of regular HTTP requests. It is invoked via an AJAX request + * (if the client's browser supports JavaScript). + * + * @see poormanscron_run_cron_check_access() + */ +function poormanscron_run_cron_check() { + //drupal_page_is_cacheable(FALSE); + $_SERVER['REQUEST_METHOD'] = 'POST'; + + $cron_run = FALSE; + + // Cron threshold semaphore is used to avoid errors every time the image + // callback is displayed when a previous cron is still running. + $threshold_semaphore = variable_get('cron_threshold_semaphore', FALSE); + if ($threshold_semaphore) { + if (REQUEST_TIME - $threshold_semaphore > 3600) { + // Either cron has been running for more than an hour or the semaphore + // was not reset due to a database error. + watchdog('cron', 'Cron has been running for more than an hour and is most likely stuck.', array(), WATCHDOG_ERROR); - // Delete any messages added during the cron run (and existing prior - // messages). - drupal_get_messages(); - - // Restore any prior messages. - if (isset($saved_messages)) { - foreach ($saved_messages as $type => $types_messages) { - foreach ($types_messages as $message) { - drupal_set_message($message, $type); - } - } + // Release the cron threshold semaphore. + variable_del('cron_threshold_semaphore'); + } + } + else { + // Run cron automatically if it has never run or threshold was crossed. + $cron_last = variable_get('cron_last', NULL); + $cron_threshold = variable_get('cron_safe_threshold', DRUPAL_CRON_DEFAULT_THRESHOLD); + if (!isset($cron_last) || (REQUEST_TIME - $cron_last > $cron_threshold) || TRUE) { + // Lock cron threshold semaphore. + variable_set('cron_threshold_semaphore', REQUEST_TIME); + $cron_run = drupal_cron_run(); + // Release the cron threshold semaphore. + variable_del('cron_threshold_semaphore'); } } + + drupal_json(array('cron_run' => $cron_run)); + exit; } /** - * Implmentation of hook_menu(). + * Implementation of hook_form_FORM_ID_alter(). */ -function poormanscron_menu() { - $items = array(); - $items['admin/settings/poormanscron'] = array( - 'title' => 'Poormanscron', - 'description' => 'A module which runs Drupal cron jobs without the cron application.', - 'page callback' => 'drupal_get_form', - 'page arguments' => array('poormanscron_admin_settings'), - 'access arguments' => array('administer site configuration'), - 'file' => 'poormanscron.admin.inc', +function poormanscron_form_system_site_information_settings_alter(&$form, &$form_state) { + $form['cron_safe_threshold'] = array( + '#type' => 'select', + '#title' => t('Automatically run cron'), + '#default_value' => variable_get('cron_safe_threshold', DRUPAL_CRON_DEFAULT_THRESHOLD), + '#options' => array(0 => t('Never')) + drupal_map_assoc(array(3600, 10800, 21600, 43200, 86400, 604800), 'format_interval'), + '#description' => t('When enabled, the site will check whether cron has been run in the configured interval and automatically run it upon the next page request. For more information visit the status report page.', array('@status-report-url' => url('admin/reports/status'))), ); - return $items; + $form['buttons'] += array('#weight' => 30); }