diff --git a/core/modules/search/search.test b/core/modules/search/search.test index 619055c..9e5173e 100644 --- a/core/modules/search/search.test +++ b/core/modules/search/search.test @@ -450,7 +450,7 @@ class SearchRankingTestCase extends SearchWebTestCase { $this->drupalPost(NULL, $edit, t('Save')); // Enable counting of statistics. - variable_set('statistics_count_content_views', 1); + config('statistics.settings')->set('count_content_views', 1)->save(); // Then View one of the nodes a bunch of times. // Manually calling statistics.php, simulating ajax behavior. diff --git a/core/modules/statistics/config/statistics.settings.xml b/core/modules/statistics/config/statistics.settings.xml new file mode 100644 index 0000000..6b2f355 --- /dev/null +++ b/core/modules/statistics/config/statistics.settings.xml @@ -0,0 +1,20 @@ + + + + 0 + 259200 + + 0 + + + + 0 + + + 0 + + + 0 + + + diff --git a/core/modules/statistics/statistics.admin.inc b/core/modules/statistics/statistics.admin.inc index 87ae6a5..3458228 100644 --- a/core/modules/statistics/statistics.admin.inc +++ b/core/modules/statistics/statistics.admin.inc @@ -93,7 +93,7 @@ function statistics_top_pages() { $rows[] = array($page->hits, _statistics_format_item($page->title, $page->path), t('%time ms', array('%time' => round($page->average_time))), format_interval(round($page->total_time / 1000))); } - drupal_set_title(t('Top pages in the past %interval', array('%interval' => format_interval(variable_get('statistics_flush_accesslog_timer', 259200)))), PASS_THROUGH); + drupal_set_title(t('Top pages in the past %interval', array('%interval' => format_interval(config('statistics.settings')->get('access_log.flush_timer')))), PASS_THROUGH); $build['statistics_top_pages_table'] = array( '#theme' => 'table', '#header' => $header, @@ -153,7 +153,7 @@ function statistics_top_visitors() { $rows[] = array($account->hits, ($account->uid ? theme('username', array('account' => $account)) : $account->hostname), format_interval(round($account->total / 1000)), (user_access('block IP addresses') && !$account->uid) ? $ban_link : ''); } - drupal_set_title(t('Top visitors in the past %interval', array('%interval' => format_interval(variable_get('statistics_flush_accesslog_timer', 259200)))), PASS_THROUGH); + drupal_set_title(t('Top visitors in the past %interval', array('%interval' => format_interval(config('statistics.settings')->get('access_log.flush_timer')))), PASS_THROUGH); $build['statistics_top_visitors_table'] = array( '#theme' => 'table', '#header' => $header, @@ -175,7 +175,7 @@ function statistics_top_visitors() { * A render array containing the top referrers information. */ function statistics_top_referrers() { - drupal_set_title(t('Top referrers in the past %interval', array('%interval' => format_interval(variable_get('statistics_flush_accesslog_timer', 259200)))), PASS_THROUGH); + drupal_set_title(t('Top referrers in the past %interval', array('%interval' => format_interval(config('statistics.settings')->get('access_log.flush_timer')))), PASS_THROUGH); $header = array( array('data' => t('Hits'), 'field' => 'hits', 'sort' => 'desc'), @@ -276,6 +276,8 @@ function statistics_access_log($aid) { */ function statistics_settings_form() { // Access log settings. + $config = config('statistics.settings'); + $form['access'] = array( '#type' => 'fieldset', '#title' => t('Access log settings'), @@ -283,13 +285,13 @@ function statistics_settings_form() { $form['access']['statistics_enable_access_log'] = array( '#type' => 'checkbox', '#title' => t('Enable access log'), - '#default_value' => variable_get('statistics_enable_access_log', 0), + '#default_value' => $config->get('access_log.enable'), '#description' => t('Log each page access. Required for referrer statistics.'), ); $form['access']['statistics_flush_accesslog_timer'] = array( '#type' => 'select', '#title' => t('Discard access logs older than'), - '#default_value' => variable_get('statistics_flush_accesslog_timer', 259200), + '#default_value' => $config->get('access_log.flush_timer'), '#options' => array(0 => t('Never')) + drupal_map_assoc(array(3600, 10800, 21600, 32400, 43200, 86400, 172800, 259200, 604800, 1209600, 2419200, 4838400, 9676800), 'format_interval'), '#description' => t('Older access log entries (including referrer statistics) will be automatically discarded. (Requires a correctly configured cron maintenance task.)', array('@cron' => url('admin/reports/status'))), ); @@ -302,9 +304,25 @@ function statistics_settings_form() { $form['content']['statistics_count_content_views'] = array( '#type' => 'checkbox', '#title' => t('Count content views'), - '#default_value' => variable_get('statistics_count_content_views', 0), + '#default_value' => $config->get('count_content_views'), '#description' => t('Increment a counter each time content is viewed.'), ); - return system_settings_form($form); + $form['submit'] = array('#type' => 'submit', '#value' => t('Save configuration')); + return $form; +} + +/** + * Form builder submit handler; Handle submission for statistics settings. + * + * @ingroup forms + * @see system_settings_form() + */ +function statistics_settings_form_submit($form, &$form_state) { + // Set the maintenance mode parameters. + $config = config('statistics.settings'); + $config->set('access_log.enable', $form_state['values']['statistics_enable_access_log']); + $config->set('access_log.flush_timer', $form_state['values']['statistics_flush_accesslog_timer']); + $config->set('count_content_views', $form_state['values']['statistics_count_content_views']); + $config->save(); } diff --git a/core/modules/statistics/statistics.install b/core/modules/statistics/statistics.install index 5ee20dc..55670eb 100644 --- a/core/modules/statistics/statistics.install +++ b/core/modules/statistics/statistics.install @@ -6,20 +6,6 @@ */ /** - * Implements hook_uninstall(). - */ -function statistics_uninstall() { - // Remove variables. - variable_del('statistics_count_content_views'); - variable_del('statistics_enable_access_log'); - variable_del('statistics_flush_accesslog_timer'); - variable_del('statistics_day_timestamp'); - variable_del('statistics_block_top_day_num'); - variable_del('statistics_block_top_all_num'); - variable_del('statistics_block_top_last_num'); -} - -/** * Implements hook_schema(). */ function statistics_schema() { @@ -134,3 +120,21 @@ function statistics_schema() { return $schema; } + +/** + * @defgroup updates-7.x-to-8.x Updates from 7.x to 8.x + * @{ + * Update functions from 7.x to 8.x. + */ + +/** + * Moves statistics settings from variable to config. + */ +function statistics_update_8000() { + update_variables_to_config('statistics.settings'); +} + +/** + * @} End of "defgroup updates-7.x-to-8.x". + * The next series of updates should start at 9000. + */ diff --git a/core/modules/statistics/statistics.module b/core/modules/statistics/statistics.module index ef1bceb..0db3628 100644 --- a/core/modules/statistics/statistics.module +++ b/core/modules/statistics/statistics.module @@ -58,7 +58,7 @@ function statistics_exit() { // in which case we need to bootstrap to the session phase anyway. drupal_bootstrap(DRUPAL_BOOTSTRAP_VARIABLES); - if (variable_get('statistics_enable_access_log', 0)) { + if (config('statistics.settings')->get('access_log.enable')) { drupal_bootstrap(DRUPAL_BOOTSTRAP_SESSION); // For anonymous users unicode.inc will not have been loaded. @@ -230,20 +230,21 @@ function statistics_user_predelete($account) { * Implements hook_cron(). */ function statistics_cron() { - $statistics_timestamp = variable_get('statistics_day_timestamp', ''); + $config = config('statistics.settings'); + $statistics_timestamp = $config->get('statistics_day_timestamp'); if ((REQUEST_TIME - $statistics_timestamp) >= 86400) { // Reset day counts. db_update('node_counter') ->fields(array('daycount' => 0)) ->execute(); - variable_set('statistics_day_timestamp', REQUEST_TIME); + $config->set('statistics_day_timestamp', REQUEST_TIME)->save(); } // Clean up expired access logs (if applicable). - if (variable_get('statistics_flush_accesslog_timer', 259200) > 0) { + if ($config->get('access_log.flush_timer') > 0) { db_delete('accesslog') - ->condition('timestamp', REQUEST_TIME - variable_get('statistics_flush_accesslog_timer', 259200), '<') + ->condition('timestamp', REQUEST_TIME - config('statistics.settings')->get('access_log.flush_timer'), '<') ->execute(); } } @@ -311,8 +312,8 @@ function statistics_get($nid) { */ function statistics_block_info() { $blocks = array(); - - if (variable_get('statistics_count_content_views', 0)) { + $statistics_count_content_views = config('statistics.settings')->get('count_content_views'); + if (!empty($statistics_count_content_views)) { $blocks['popular']['info'] = t('Popular content'); // Too dynamic to cache. $blocks['popular']['cache'] = DRUPAL_NO_CACHE; @@ -325,10 +326,11 @@ function statistics_block_info() { */ function statistics_block_configure($delta = '') { // Popular content block settings + $config = config('statistics.settings'); $numbers = array('0' => t('Disabled')) + drupal_map_assoc(array(1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 15, 20, 25, 30, 40)); - $form['statistics_block_top_day_num'] = array('#type' => 'select', '#title' => t("Number of day's top views to display"), '#default_value' => variable_get('statistics_block_top_day_num', 0), '#options' => $numbers, '#description' => t('How many content items to display in "day" list.')); - $form['statistics_block_top_all_num'] = array('#type' => 'select', '#title' => t('Number of all time views to display'), '#default_value' => variable_get('statistics_block_top_all_num', 0), '#options' => $numbers, '#description' => t('How many content items to display in "all time" list.')); - $form['statistics_block_top_last_num'] = array('#type' => 'select', '#title' => t('Number of most recent views to display'), '#default_value' => variable_get('statistics_block_top_last_num', 0), '#options' => $numbers, '#description' => t('How many content items to display in "recently viewed" list.')); + $form['statistics_block_top_day_num'] = array('#type' => 'select', '#title' => t("Number of day's top views to display"), '#default_value' => $config->get('block.top_day.num'), '#options' => $numbers, '#description' => t('How many content items to display in "day" list.')); + $form['statistics_block_top_all_num'] = array('#type' => 'select', '#title' => t('Number of all time views to display'), '#default_value' => $config->get('block.top_all.num'), '#options' => $numbers, '#description' => t('How many content items to display in "all time" list.')); + $form['statistics_block_top_last_num'] = array('#type' => 'select', '#title' => t('Number of most recent views to display'), '#default_value' => $config->get('block.top_last.num'), '#options' => $numbers, '#description' => t('How many content items to display in "recently viewed" list.')); return $form; } @@ -336,9 +338,11 @@ function statistics_block_configure($delta = '') { * Implements hook_block_save(). */ function statistics_block_save($delta = '', $edit = array()) { - variable_set('statistics_block_top_day_num', $edit['statistics_block_top_day_num']); - variable_set('statistics_block_top_all_num', $edit['statistics_block_top_all_num']); - variable_set('statistics_block_top_last_num', $edit['statistics_block_top_last_num']); + $config = config('statistics.settings'); + $config->set('block.top_day.num', $edit['statistics_block_top_day_num']); + $config->set('block.top_all.num', $edit['statistics_block_top_all_num']); + $config->set('block.top_last.num', $edit['statistics_block_top_last_num']); + $config->save(); } /** @@ -347,20 +351,21 @@ function statistics_block_save($delta = '', $edit = array()) { function statistics_block_view($delta = '') { if (user_access('access content')) { $content = array(); + $config = config('statistics.settings'); - $daytop = variable_get('statistics_block_top_day_num', 0); + $daytop = $config->get('block.top_day.num'); if ($daytop && ($result = statistics_title_list('daycount', $daytop)) && ($node_title_list = node_title_list($result, t("Today's:")))) { $content['top_day'] = $node_title_list; $content['top_day']['#suffix'] = '
'; } - $alltimetop = variable_get('statistics_block_top_all_num', 0); + $alltimetop = $config->get('block.top_all.num'); if ($alltimetop && ($result = statistics_title_list('totalcount', $alltimetop)) && ($node_title_list = node_title_list($result, t('All time:')))) { $content['top_all'] = $node_title_list; $content['top_all']['#suffix'] = '
'; } - $lasttop = variable_get('statistics_block_top_last_num', 0); + $lasttop = $config->get('block.top_last.num'); if ($lasttop && ($result = statistics_title_list('timestamp', $lasttop)) && ($node_title_list = node_title_list($result, t('Last viewed:')))) { $content['top_last'] = $node_title_list; $content['top_last']['#suffix'] = '
'; @@ -423,7 +428,8 @@ function statistics_node_predelete(Node $node) { * Implements hook_ranking(). */ function statistics_ranking() { - if (variable_get('statistics_count_content_views', 0)) { + $statistics_count_content_views = config('statistics.settings')->get('count_content_views'); + if (!empty($statistics_count_content_views)) { return array( 'views' => array( 'title' => t('Number of views'), diff --git a/core/modules/statistics/statistics.php b/core/modules/statistics/statistics.php index 040dc19..6ccc96d 100644 --- a/core/modules/statistics/statistics.php +++ b/core/modules/statistics/statistics.php @@ -15,7 +15,8 @@ define('DRUPAL_ROOT', getcwd()); include_once DRUPAL_ROOT . '/core/includes/bootstrap.inc'; drupal_bootstrap(DRUPAL_BOOTSTRAP_VARIABLES); -if (variable_get('statistics_count_content_views', 0)) { + +if (config('statistics.settings')->get('count_content_views')) { $nid = $_POST['nid']; if (is_numeric($nid)) { db_merge('node_counter') diff --git a/core/modules/statistics/statistics.test b/core/modules/statistics/statistics.test index bb28b01..48e79a1 100644 --- a/core/modules/statistics/statistics.test +++ b/core/modules/statistics/statistics.test @@ -33,8 +33,10 @@ class StatisticsTestCase extends WebTestBase { $this->drupalLogin($this->blocking_user); // Enable access logging. - variable_set('statistics_enable_access_log', 1); - variable_set('statistics_count_content_views', 1); + $config = config('statistics.settings'); + $config->set('access_log.enable', 1); + $config->set('count_content_views', 1); + $config->save(); // Insert dummy access by anonymous user into access log. db_insert('accesslog') @@ -70,6 +72,12 @@ class StatisticsLoggingTestCase extends WebTestBase { function setUp() { parent::setUp(array('statistics', 'block')); + // Enable access logging. + $config = config('statistics.settings'); + $config->set('access_log.enable', 1); + $config->set('count_content_views', 1); + $config->save(); + // Create Basic page node type. if ($this->profile != 'standard') { $this->drupalCreateContentType(array('type' => 'page', 'name' => 'Basic page')); @@ -85,10 +93,6 @@ class StatisticsLoggingTestCase extends WebTestBase { $config->set('cache', 1); $config->save(); - // Enable access logging. - variable_set('statistics_enable_access_log', 1); - variable_set('statistics_count_content_views', 1); - // Clear the logs. db_truncate('accesslog'); db_truncate('node_counter'); @@ -363,8 +367,9 @@ class StatisticsAdminTestCase extends WebTestBase { * Verifies that the statistics settings page works. */ function testStatisticsSettings() { - $this->assertFalse(variable_get('statistics_enable_access_log', 0), t('Access log is disabled by default.')); - $this->assertFalse(variable_get('statistics_count_content_views', 0), t('Count content view log is disabled by default.')); + $config = config('statistics.settings'); + $this->assertFalse($config->get('access_log.enable'), t('Access log is disabled by default.')); + $this->assertFalse($config->get('count_content_views'), t('Count content view log is disabled by default.')); $this->drupalGet('admin/reports/pages'); $this->assertRaw(t('No statistics available.'), t('Verifying text shown when no statistics is available.')); @@ -373,8 +378,8 @@ class StatisticsAdminTestCase extends WebTestBase { $edit['statistics_enable_access_log'] = 1; $edit['statistics_count_content_views'] = 1; $this->drupalPost('admin/config/system/statistics', $edit, t('Save configuration')); - $this->assertTrue(variable_get('statistics_enable_access_log'), t('Access log is enabled.')); - $this->assertTrue(variable_get('statistics_count_content_views'), t('Count content view log is enabled.')); + $this->assertTrue(config('statistics.settings')->get('access_log.enable'), t('Access log is enabled.')); + $this->assertTrue(config('statistics.settings')->get('count_content_views'), t('Count content view log is enabled.')); // Hit the node. $this->drupalGet('node/' . $this->test_node->nid); @@ -404,7 +409,7 @@ class StatisticsAdminTestCase extends WebTestBase { * Tests that when a node is deleted, the node counter is deleted too. */ function testDeleteNode() { - variable_set('statistics_count_content_views', 1); + config('statistics.settings')->set('count_content_views', 1)->save(); $this->drupalGet('node/' . $this->test_node->nid); // Manually calling statistics.php, simulating ajax behavior. @@ -436,7 +441,7 @@ class StatisticsAdminTestCase extends WebTestBase { * Tests that accesslog reflects when a user is deleted. */ function testDeleteUser() { - variable_set('statistics_enable_access_log', 1); + config('statistics.settings')->set('access_log.enable', 1)->save(); variable_set('user_cancel_method', 'user_cancel_delete'); $this->drupalLogout($this->privileged_user); @@ -467,10 +472,12 @@ class StatisticsAdminTestCase extends WebTestBase { * Tests that cron clears day counts and expired access logs. */ function testExpiredLogs() { - variable_set('statistics_enable_access_log', 1); - variable_set('statistics_count_content_views', 1); - variable_set('statistics_day_timestamp', 8640000); - variable_set('statistics_flush_accesslog_timer', 1); + $config = config('statistics.settings'); + $config->set('access_log.enable', 1); + $config->set('count_content_views', 1); + $config->set('statistics_day_timestamp', 8640000); + $config->set('access_log.flush_timer', 1); + $config->save(); $this->drupalGet('node/' . $this->test_node->nid); // Manually calling statistics.php, simulating ajax behavior.