? sites/all/modules ? sites/default/files ? sites/default/settings.php Index: misc/form.js =================================================================== RCS file: /cvs/drupal/drupal/misc/form.js,v retrieving revision 1.2 diff -u -p -r1.2 form.js --- misc/form.js 29 Oct 2008 10:01:26 -0000 1.2 +++ misc/form.js 13 Jan 2009 13:19:58 -0000 @@ -8,5 +8,10 @@ Drupal.behaviors.multiselectSelector = { $('.multiselect input:radio[value="'+ this.id.substr(5) +'"]') .attr('checked', true); }); + $('.multiselect input:not(.multiselectSelector-processed)', context) + .addClass('multiselectSelector-processed').change(function() { + $('.multiselect input:radio[value="'+ this.id.substr(5) +'"]') + .attr('checked', true); + }); } }; Index: modules/statistics/statistics.admin.inc =================================================================== RCS file: /cvs/drupal/drupal/modules/statistics/statistics.admin.inc,v retrieving revision 1.16 diff -u -p -r1.16 statistics.admin.inc --- modules/statistics/statistics.admin.inc 11 Jan 2009 21:19:18 -0000 1.16 +++ modules/statistics/statistics.admin.inc 13 Jan 2009 13:19:59 -0000 @@ -10,6 +10,9 @@ * Menu callback; presents the "recent hits" page. */ function statistics_recent_hits() { + $filtertype = 'hits'; + $filter = statistics_build_filter_query($filtertype); + $header = array( array('data' => t('Timestamp'), 'field' => 'a.timestamp', 'sort' => 'desc'), array('data' => t('Page'), 'field' => 'a.path'), @@ -17,9 +20,11 @@ function statistics_recent_hits() { array('data' => t('Operations')) ); - $sql = 'SELECT a.aid, a.path, a.title, a.uid, u.name, a.timestamp FROM {accesslog} a LEFT JOIN {users} u ON u.uid = a.uid' . tablesort_sql($header); + $sql = 'SELECT a.aid, a.path, a.title, a.uid, u.name, a.timestamp FROM {accesslog} a LEFT JOIN {users} u ON u.uid = a.uid' . $filter['join'] . ' WHERE 1 ' . $filter['where']; + $sql .= tablesort_sql($header); + $query_count = 'SELECT COUNT(DISTINCT a.aid) FROM {accesslog} a LEFT JOIN {users} u ON u.uid = a.uid' . $filter['join'] . ' WHERE 1 ' . $filter['where']; + $result = pager_query($sql, 30, 0, $query_count, $filter['args']); - $result = pager_query($sql, 30); $rows = array(); while ($log = db_fetch_object($result)) { $rows[] = array( @@ -33,7 +38,8 @@ function statistics_recent_hits() { $rows[] = array(array('data' => t('No statistics available.'), 'colspan' => 4)); } - $output = theme('table', $header, $rows); + $output = drupal_get_form('statistics_filter_form', 'hits'); + $output .= theme('table', $header, $rows); $output .= theme('pager', NULL, 30, 0); return $output; } @@ -42,18 +48,21 @@ function statistics_recent_hits() { * Menu callback; presents the "top pages" page. */ function statistics_top_pages() { - // MAX(title) avoids having empty node titles which otherwise causes duplicates in the top pages list - $sql = "SELECT COUNT(path) AS hits, path, MAX(title) AS title, AVG(timer) AS average_time, SUM(timer) AS total_time FROM {accesslog} GROUP BY path"; - $sql_cnt = "SELECT COUNT(DISTINCT(path)) FROM {accesslog}"; + $filtertype = 'pages'; + $filter = statistics_build_filter_query($filtertype); $header = array( array('data' => t('Hits'), 'field' => 'hits', 'sort' => 'desc'), array('data' => t('Page'), 'field' => 'path'), array('data' => t('Average page generation time'), 'field' => 'average_time'), array('data' => t('Total page generation time'), 'field' => 'total_time') - ); + ) + ; + // MAX(title) avoids having empty node titles which otherwise causes duplicates in the top pages list + $sql = 'SELECT COUNT(path) AS hits, path, MAX(title) AS title, AVG(timer) AS average_time, SUM(timer) AS total_time FROM {accesslog}' . $filter['join'] . ' WHERE 1 ' . $filter['where'] . ' GROUP BY path'; $sql .= tablesort_sql($header); - $result = pager_query($sql, 30, 0, $sql_cnt); + $sql_cnt = 'SELECT COUNT(DISTINCT(path)) FROM {accesslog}' . $filter['join'] . ' WHERE 1 ' . $filter['where']; + $result = pager_query($sql, 30, 0, $sql_cnt, $filter['args']); $rows = array(); while ($page = db_fetch_object($result)) { @@ -65,7 +74,8 @@ function statistics_top_pages() { } drupal_set_title(t('Top pages in the past %interval', array('%interval' => format_interval(variable_get('statistics_flush_accesslog_timer', 259200)))), PASS_THROUGH); - $output = theme('table', $header, $rows); + $output = drupal_get_form('statistics_filter_form', $filtertype); + $output .= theme('table', $header, $rows); $output .= theme('pager', NULL, 30, 0); return $output; } @@ -74,6 +84,8 @@ function statistics_top_pages() { * Menu callback; presents the "top visitors" page. */ function statistics_top_visitors() { + $filtertype = 'visitors'; + $filter = statistics_build_filter_query($filtertype); $header = array( array('data' => t('Hits'), 'field' => 'hits', 'sort' => 'desc'), @@ -82,9 +94,11 @@ function statistics_top_visitors() { array('data' => user_access('block IP addresses') ? t('Operations') : '', 'colspan' => 2), ); - $sql = "SELECT COUNT(a.uid) AS hits, a.uid, u.name, a.hostname, SUM(a.timer) AS total, bl.iid FROM {accesslog} a LEFT JOIN {blocked_ips} bl ON a.hostname = bl.ip LEFT JOIN {users} u ON a.uid = u.uid GROUP BY a.hostname, a.uid, u.name, bl.iid" . tablesort_sql($header); - $sql_cnt = "SELECT COUNT(DISTINCT(CONCAT(CAST(uid AS char), hostname))) FROM {accesslog}"; - $result = pager_query($sql, 30, 0, $sql_cnt); + $sql = 'SELECT COUNT(a.uid) AS hits, a.uid, u.name, a.hostname, SUM(a.timer) AS total, bl.iid FROM {accesslog} a LEFT JOIN {blocked_ips} bl ON a.hostname = bl.ip LEFT JOIN {users} u ON a.uid = u.uid'; + $sql .= $filter['join'] . ' WHERE 1 ' . $filter['where'] . 'GROUP BY a.hostname, a.uid, u.name, bl.iid'; + $sql .= tablesort_sql($header); + $sql_cnt = 'SELECT COUNT(DISTINCT(CONCAT(CAST(a.uid AS char), hostname))) FROM {accesslog} a LEFT JOIN {users} u ON u.uid = a.uid' . $filter['join'] . ' WHERE 1 ' . $filter['where']; + $result = pager_query($sql, 30, 0, $sql_cnt, $filter['args']); $rows = array(); while ($account = db_fetch_object($result)) { @@ -98,7 +112,8 @@ function statistics_top_visitors() { } drupal_set_title(t('Top visitors in the past %interval', array('%interval' => format_interval(variable_get('statistics_flush_accesslog_timer', 259200)))), PASS_THROUGH); - $output = theme('table', $header, $rows); + $output = drupal_get_form('statistics_filter_form', $filtertype); + $output .= theme('table', $header, $rows); $output .= theme('pager', NULL, 30, 0); return $output; } @@ -107,9 +122,8 @@ function statistics_top_visitors() { * Menu callback; presents the "referrer" page. */ function statistics_top_referrers() { - $query = "SELECT url, COUNT(url) AS hits, MAX(timestamp) AS last FROM {accesslog} WHERE url NOT LIKE :host AND url <> '' GROUP BY url"; - $query_cnt = "SELECT COUNT(DISTINCT(url)) FROM {accesslog} WHERE url <> '' AND url NOT LIKE :host"; - drupal_set_title(t('Top referrers in the past %interval', array('%interval' => format_interval(variable_get('statistics_flush_accesslog_timer', 259200)))), PASS_THROUGH); + $filtertype = 'referrers'; + $filter = statistics_build_filter_query($filtertype); $header = array( array('data' => t('Hits'), 'field' => 'hits', 'sort' => 'desc'), @@ -117,8 +131,12 @@ function statistics_top_referrers() { array('data' => t('Last visit'), 'field' => 'last'), ); + $query = 'SELECT url, COUNT(url) AS hits, MAX(timestamp) AS last FROM {accesslog}' . $filter['join'] . " WHERE url NOT LIKE '%s' AND url <> ''" . $filter['where'] . ' GROUP BY url'; + $query_cnt = 'SELECT COUNT(DISTINCT(url)) FROM {accesslog}' . $filter['join'] . " WHERE url <> '' AND url NOT LIKE '%s' " . $filter['where']; + drupal_set_title(t('Top referrers in the past %interval', array('%interval' => format_interval(variable_get('statistics_flush_accesslog_timer', 259200)))), PASS_THROUGH); + $query .= tablesort_sql($header); - $result = pager_query($query, 30, 0, $query_cnt, array(':host' => '%'. $_SERVER['HTTP_HOST'] .'%')); + $result = pager_query($query, 30, 0, $query_cnt, array_merge(array('%'. $_SERVER['HTTP_HOST'] .'%'), $filter['args'])); $rows = array(); while ($referrer = db_fetch_object($result)) { @@ -129,7 +147,8 @@ function statistics_top_referrers() { $rows[] = array(array('data' => t('No statistics available.'), 'colspan' => 3)); } - $output = theme('table', $header, $rows); + $output = drupal_get_form('statistics_filter_form', $filtertype); + $output .= theme('table', $header, $rows); $output .= theme('pager', NULL, 30, 0); return $output; } @@ -175,6 +194,111 @@ function statistics_access_log($aid) { } /** + * Form builder; Return form for logging filters. + * + * @param $type Type of logging. + * + * @ingroup forms + * @see statistics_filter_form_submit() + */ +function statistics_filter_form($form_state, $type) { + $session = isset($_SESSION['statistics_filter']) ? $_SESSION['statistics_filter'] : array(); + $session = (is_array($session) && isset($session[$type]) ) ? $session[$type] : array(); + $filters = statistics_filters(); + + $i = 0; + $form['filters'] = array( + '#type' => 'fieldset', + '#title' => t('Hide !type where', array('!type' => $filters[$type]['#title'])), + '#theme' => 'statistics_filters', + '#collapsible' => TRUE, + '#collapsed' => FALSE, + ); + + $form['filters']['type'] = array( + '#type' => 'value', + '#value' => $type, + ); + + // get only type specific filters. + $filters = $filters[$type]['#filters']; + + foreach ($session as $filter) { + list($filtertype, $filtervalue) = $filter; + $params = array('%property' => $filters[$filtertype]['title'] , '%value' => $filtervalue); + if ($i++ > 0) { + $form['filters']['current'][] = array('#markup' => t('and where %property is %value', $params)); + } + else { + $form['filters']['current'][] = array('#markup' => t('%property is %value', $params)); + } + } + + foreach ($filters as $key => $filter) { + $names[$key] = $filter['title']; + $form['filters']['status'][$key] = array( + '#type' => 'textfield', + '#size' => 50, + ); + } + + $form['filters']['filter'] = array( + '#type' => 'radios', + '#options' => $names, + ); + + $form['filters']['buttons']['submit'] = array( + '#type' => 'submit', + '#value' => (count($session) ? t('Refine') : t('Filter')), + ); + if (count($session)) { + $form['filters']['buttons']['undo'] = array( + '#type' => 'submit', + '#value' => t('Undo'), + ); + $form['filters']['buttons']['reset'] = array( + '#type' => 'submit', + '#value' => t('Reset'), + ); + } + + drupal_add_js('misc/form.js'); + + return $form; +} + +/** + * Process result from statistics filter form. + */ +function statistics_filter_form_submit($form, &$form_state) { + $op = $form_state['values']['op']; + $type = $form_state['values']['type']; + $filters = statistics_filters(); + // get only type specific filters. + $filters = $filters[$type]['#filters']; + switch ($op) { + case t('Filter'): + case t('Refine'): + if (isset($form_state['values']['filter'])) { + $filter = $form_state['values']['filter']; + if (isset($form_state['values'][$filter])) { + $_SESSION['statistics_filter'][$type][] = array($filter, $form_state['values'][$filter]); + } + } + break; + case t('Undo'): + array_pop($_SESSION['statistics_filter'][$type]); + break; + case t('Reset'): + $_SESSION['statistics_filter'][$type] = array(); + break; + } + + $form_state['redirect'] = 'admin/reports/' . $type; + return; +} + +/** * Form builder; Configure access logging. * * @ingroup forms @@ -213,3 +337,49 @@ function statistics_access_logging_setti return system_settings_form($form, TRUE); } + +/** + * Theme user administration filter form. + * + * @ingroup themeable + */ +function theme_statistics_filter_form(&$form) { + $output = '
'; + $output .= drupal_render($form['filters']); + $output .= '
'; + $output .= drupal_render($form); + return $output; +} + +/** + * Theme statistics filter selector. + * + * @ingroup themeable + */ +function theme_statistics_filters(&$form) { + $output = ''; + + return $output; +} Index: modules/statistics/statistics.module =================================================================== RCS file: /cvs/drupal/drupal/modules/statistics/statistics.module,v retrieving revision 1.293 diff -u -p -r1.293 statistics.module --- modules/statistics/statistics.module 8 Jan 2009 08:42:13 -0000 1.293 +++ modules/statistics/statistics.module 13 Jan 2009 13:19:59 -0000 @@ -38,6 +38,22 @@ function statistics_help($path, $arg) { } /** + * Implementation of hook_theme(). + */ +function statistics_theme() { + return array( + 'statistics_filter_form' => array( + 'arguments' => array('form' => NULL), + 'file' => 'statistics.admin.inc', + ), + 'statistics_filters' => array( + 'arguments' => array('form' => NULL), + 'file' => 'statistics.admin.inc', + ), + ); +} + +/** * Implementation of hook_exit(). * * This is where statistics are gathered on page accesses. @@ -322,6 +338,109 @@ function statistics_block_view($delta = } /** + * List statistics filters that can be applied. + */ +function statistics_filters() { + // Regular filters + $filters = array(); + + $filters['hits'] = array( + '#title' => t('recent hits'), + '#filters' => array( + 'page' => array( + 'title' => t('page'), + 'where' => 'a.path NOT LIKE \'%s\'', + 'join' => '', + ), + 'uid' => array( + 'title' => t('userid'), + 'where' => 'a.uid != %d', + 'join' => '', + ), + 'user' => array( + 'title' => t('username'), + 'where' => 'u.name NOT LIKE \'%s\'', + 'join' => '', + ), + ), + ); + $filters['pages'] = array( + '#title' => t('top pages'), + '#filters' => array( + 'path' => array( + 'title' => t('path'), + 'where' => 'path NOT LIKE \'%s\'', + 'join' => '', + ), + 'title' => array( + 'title' => t('title'), + 'where' => 'title NOT LIKE \'%s\'', + 'join' => '', + ), + ), + ); + $filters['referrers'] = array( + '#title' => t('top referrers'), + '#filters' => array( + 'url' => array( + 'title' => t('url'), + 'where' => 'url NOT LIKE \'%s\'', + 'join' => '', + ), + ), + ); + $filters['visitors'] = array( + '#title' => t('top visitors'), + '#filters' => array( + 'path' => array( + 'title' => t('username'), + 'where' => 'u.name NOT LIKE \'%s\'', + 'join' => '', + ), + 'hostname' => array( + 'title' => t('hostname'), + 'where' => 'a.hostname NOT LIKE \'%s\'', + 'join' => '', + ), + ), + ); + + return $filters; +} + +/** + * Build query for statistic filters based on session and statistic type. + * + * @param $type + * Type of filter. + */ +function statistics_build_filter_query($type) { + $filters = statistics_filters(); + // get type specific filters. + $filters = $filters[$type]['#filters']; + + if (!isset($_SESSION['statistics_filter'][$type])) { + return array('where' => '', 'join' => '', 'args' => array()); + } + + // Build query. + $where = $args = $join = array(); + foreach ($_SESSION['statistics_filter'][$type] as $filter) { + list($key, $value) = $filter; + $where[] = $filters[$key]['where']; + $join[] = $filters[$key]['join']; + $args[] = $value; + } + $where = !empty($where) ? ' AND ' . implode(' AND ', $where) : ''; + $join = !empty($join) ? ' ' . implode(' ', array_unique($join)) : ''; + + return array('where' => $where, + 'join' => $join, + 'args' => $args, + ); +} + +/** * It is possible to adjust the width of columns generated by the * statistics module. */