Even more rewrites/fixes from EL branch (please report any broken part after you...

[mailer.git] / inc / modules / admin / what-surfbar_stats.php

diff --git a/inc/modules/admin/what-surfbar_stats.php b/inc/modules/admin/what-surfbar_stats.php
index 7db1aaf511aef397877bd8471ebd0ae3d0d15fd2..62a5ca2c7351ab76fccdcf423746a8feaef1da99 100644 (file)
--- a/inc/modules/admin/what-surfbar_stats.php
+++ b/inc/modules/admin/what-surfbar_stats.php
@@ -1,7 +1,7 @@
 <?php
 /************************************************************************
- * MXChange v0.2.1                                    Start: 09/09/2008 *
- * ================                             Last change: 09/09/2008 *
+ * Mailer v0.2.1-FINAL                                Start: 09/09/2008 *
+ * ===================                          Last change: 09/09/2008 *
  *                                                                      *
  * -------------------------------------------------------------------- *
  * File              : what-surfbar_stats.php                           *
@@ -17,7 +17,8 @@
  * Needs to be in all Files and every File needs "svn propset           *
  * svn:keywords Date Revision" (autoprobset!) at least!!!!!!            *
  * -------------------------------------------------------------------- *
- * Copyright (c) 2003 - 2008 by Roland Haeder                           *
+ * Copyright (c) 2003 - 2009 by Roland Haeder                           *
+ * Copyright (c) 2009, 2010 by Mailer Developer Team                    *
  * For more information visit: http://www.mxchange.org                  *
  *                                                                      *
  * This program is free software; you can redistribute it and/or modify *
@@ -37,39 +38,44 @@
  ************************************************************************/
 
 // Some security stuff...
-if ((!defined('__SECURITY')) || (!IS_ADMIN())) {
-       $INC = substr(dirname(__FILE__), 0, strpos(dirname(__FILE__), '/inc') + 4) . '/security.php';
-       require($INC);
+if ((!defined('__SECURITY')) || (!isAdmin())) {
+       die();
 }
 
 // Add description as navigation point
-ADD_DESCR('admin', __FILE__);
+addMenuDescription('admin', __FILE__);
 
 // Is the 'url_id' set?
-if (REQUEST_ISSET_GET(('url_id'))) {
+if (isGetRequestParameterSet(('url_id'))) {
        // Generate general statistics
-       $result = SQL_QUERY_ESC("SELECT `userid`,`count` AS `total_visits`, UNIX_TIMESTAMP(`last_online`) AS `last_online`
-FROM `{!_MYSQL_PREFIX!}_surfbar_stats`
-WHERE `url_id`=%s
-ORDER BY `userid` ASC",
-       array(bigintval(REQUEST_GET('url_id'))), __FILE__, __LINE__);
+       $result = SQL_QUERY_ESC("SELECT
+       `stats_userid`,
+       `stats_count` AS `total_visits`,
+       UNIX_TIMESTAMP(`stats_last_surfed`) AS `last_surfed`
+FROM
+       `{?_MYSQL_PREFIX?}_surfbar_stats`
+WHERE
+       `stats_url_id`=%s
+ORDER BY
+       `stats_userid` ASC",
+               array(bigintval(getRequestParameter('url_id'))), __FILE__, __LINE__);
 
        // Entries found?
        if (SQL_NUMROWS($result) > 0) {
                // Get all rows
-               $OUT = ''; $SW = 2; $cnt = 0; $visits = 0;
+               $OUT = ''; $SW = 2; $cnt = '0'; $visits = '0';
                while ($content = SQL_FETCHARRAY($result)) {
                        // Count totals
                        $visits += $content['total_visits'];
 
                        // "Translate" some data
-                       $content['userid']       = generateUserProfileLink($content['userid']);
+                       $content['stats_userid'] = generateUserProfileLink($content['stats_userid']);
                        $content['total_visits'] = translateComma($content['total_visits']);
-                       $content['last_online']  = generateDateTime($content['last_online'], '2');
+                       $content['last_surfed']  = generateDateTime($content['last_surfed'], 2);
                        $content['sw']           = $SW;
 
                        // Load row template
-                       $OUT .= LOAD_TEMPLATE("admin_surfbar_url_stats_row", true, $content);
+                       $OUT .= loadTemplate('admin_surfbar_url_stats_row', true, $content);
                        $cnt++; $SW = 3 - $SW;
                } // END - while
 
@@ -82,51 +88,58 @@ ORDER BY `userid` ASC",
 
 
                // Load main template
-               LOAD_TEMPLATE("admin_surfbar_url_stats", false, $content);
+               loadTemplate('admin_surfbar_url_stats', false, $content);
        } else {
                // No statistics so far
-               LOAD_TEMPLATE('admin_settings_saved', false, getMessage('ADMIN_SURFBAR_NO_URL_STATS'));
+               loadTemplate('admin_settings_saved', false, getMessage('ADMIN_SURFBAR_NO_URL_STATS'));
        }
 } else {
        // Generate general statistics
-       $result = SQL_QUERY("SELECT `userid`, COUNT(`url_id`) AS `visited_urls`, SUM(`count`) AS `total_visits`
-FROM `{!_MYSQL_PREFIX!}_surfbar_stats`
-GROUP BY `userid`
-ORDER BY `userid` ASC", __FILE__, __LINE__);
+       $result = SQL_QUERY("SELECT
+       `stats_userid`,
+       COUNT(`stats_url_id`) AS `visited_urls`,
+       SUM(`stats_count`) AS `total_visits`
+FROM
+       `{?_MYSQL_PREFIX?}_surfbar_stats`
+GROUP BY
+       `stats_userid`
+ORDER BY
+       `stats_userid` ASC", __FILE__, __LINE__);
 
        // Entries found?
        if (SQL_NUMROWS($result) > 0) {
                // Get all rows
-               $OUT = ''; $SW = 2; $urls = 0; $visits = 0;
+               $OUT = ''; $SW = 2; $urlArray = '0'; $visits = '0';
                while ($content = SQL_FETCHARRAY($result)) {
                        // Count totals
-                       $urls   += $content['visited_urls'];
+                       $urlArray   += $content['visited_urls'];
                        $visits += $content['total_visits'];
 
                        // "Translate" some data
-                       $content['userid']       = generateUserProfileLink($content['userid']);
+                       // @TODO Can't all these translation blocks moved into templates? This code looks ugly.
+                       $content['stats_userid'] = generateUserProfileLink($content['stats_userid']);
                        $content['visited_urls'] = translateComma($content['visited_urls']);
                        $content['total_visits'] = translateComma($content['total_visits']);
                        $content['sw']           = $SW;
 
                        // Load row template
-                       $OUT .= LOAD_TEMPLATE("admin_surfbar_stats_row", true, $content);
+                       $OUT .= loadTemplate('admin_surfbar_stats_row', true, $content);
                        $SW = 3 - $SW;
                } // END - while
 
                // Prepare content
                $content = array(
                        'rows'   => $OUT,
-                       'urls'   => $urls,
+                       'urls'   => $urlArray,
                        'visits' => $visits
                );
 
 
                // Load main template
-               LOAD_TEMPLATE("admin_surfbar_stats", false, $content);
+               loadTemplate('admin_surfbar_stats', false, $content);
        } else {
                // No statistics so far
-               LOAD_TEMPLATE('admin_settings_saved', false, getMessage('ADMIN_SURFBAR_NO_STATS'));
+               loadTemplate('admin_settings_saved', false, getMessage('ADMIN_SURFBAR_NO_STATS'));
        }
 }