Enable/Disable online listing + time online

git-svn-id: file:///svn/phpbb/trunk@3841 89ea8834-ac86-4346-8a33-228a782c2dd0
author: Paul S. Owen <psotfx@users.sourceforge.net> 2003-04-15 22:57:05 +0000
committer: Paul S. Owen <psotfx@users.sourceforge.net> 2003-04-15 22:57:05 +0000
commit: dce7e4c2b82ce4cc99ac2042f765e68c0c97e92b (patch)
tree: 5e0e17f505b9c7bfe7c9b0add4f5af63419383b0
parent: d9dcb502a96744ce19127f92f08572a56a7bdf2a (diff)
download: forums-dce7e4c2b82ce4cc99ac2042f765e68c0c97e92b.tar
forums-dce7e4c2b82ce4cc99ac2042f765e68c0c97e92b.tar.gz
forums-dce7e4c2b82ce4cc99ac2042f765e68c0c97e92b.tar.bz2
forums-dce7e4c2b82ce4cc99ac2042f765e68c0c97e92b.tar.xz
forums-dce7e4c2b82ce4cc99ac2042f765e68c0c97e92b.zip
3 files changed, 111 insertions, 103 deletions
diff --git a/phpBB/includes/page_header.php b/phpBB/includes/page_header.php
index dd37c03761..fc9d7a1bfa 100644
--- a/phpBB/includes/page_header.php
+++ b/phpBB/includes/page_header.php
@@ -50,129 +50,132 @@ else
 // Last visit date/time
 $s_last_visit = ($user->data['user_id'] != ANONYMOUS) ? $user->format_date($user->data['session_last_visit']) : '';
 
-// Get users online list
-$userlist_ary = array();
-$userlist_visible = array();
-
-$logged_visible_online = 0;
-$logged_hidden_online = 0;
-$guests_online = 0;
-$online_userlist = '';
-
-$prev_user_id = 0;
-$prev_user_ip = $reading_sql = '';
-if (!empty($_REQUEST['f']))
+
+// Get users online list ... if required
+$l_online_users = $online_userlist = $l_online_record = '';
+if (!empty($config['load_online']) && !empty($config['load_online_time']))
 {
-	$reading_sql = 'AND s.session_page LIKE \'%f=' . intval($_REQUEST['f']) . '%\'';
-}
+	$userlist_ary = $userlist_visible = array();
+	$logged_visible_online = $logged_hidden_online = $guests_online = 0;
+
+	$prev_user_id = 0;
+	$prev_user_ip = $reading_sql = '';
+	if (!empty($_REQUEST['f']))
+	{
+		$reading_sql = "AND s.session_page LIKE '%f=" . intval($_REQUEST['f']) . "%'";
+	}
 
-$sql = "SELECT u.username, u.user_id, u.user_allow_viewonline, u.user_colour, s.session_ip, s.session_allow_viewonline
-	FROM " . USERS_TABLE . " u, " . SESSIONS_TABLE ." s
-	WHERE s.session_time >= " . (time() - 300) . "
-		$reading_sql
-		AND u.user_id = s.session_user_id
-	ORDER BY u.username ASC, s.session_ip ASC";
-$result = $db->sql_query($sql, false);
+	$sql = "SELECT u.username, u.user_id, u.user_allow_viewonline, u.user_colour, s.session_ip, s.session_allow_viewonline
+		FROM " . USERS_TABLE . " u, " . SESSIONS_TABLE ." s
+		WHERE s.session_time >= " . (time() - (intval($config['load_online_time']) * 60)) . "
+			$reading_sql
+			AND u.user_id = s.session_user_id
+		ORDER BY u.username ASC, s.session_ip ASC";
+	$result = $db->sql_query($sql, false);
 
-while ($row = $db->sql_fetchrow($result))
-{
-	// User is logged in and therefor not a guest
-	if ($row['user_id'] != ANONYMOUS)
+	while ($row = $db->sql_fetchrow($result))
 	{
-		// Skip multiple sessions for one user
-		if ($row['user_id'] != $prev_user_id)
+		// User is logged in and therefor not a guest
+		if ($row['user_id'] != ANONYMOUS)
 		{
-			if ($row['user_colour'])
-			{
-				$row['username'] = '<b style="color:#' . $row['user_colour'] . '">' . $row['username'] . '</b>';
-			}
-
-			if ($row['user_allow_viewonline'] && $row['session_allow_viewonline'])
-			{
-				$user_online_link = $row['username'];
-				$logged_visible_online++;
-			}
-			else
+			// Skip multiple sessions for one user
+			if ($row['user_id'] != $prev_user_id)
 			{
-				$user_online_link = '<i>' . $row['username'] . '</i>';
-				$logged_hidden_online++;
+				if ($row['user_colour'])
+				{
+					$row['username'] = '<b style="color:#' . $row['user_colour'] . '">' . $row['username'] . '</b>';
+				}
+
+				if ($row['user_allow_viewonline'] && $row['session_allow_viewonline'])
+				{
+					$user_online_link = $row['username'];
+					$logged_visible_online++;
+				}
+				else
+				{
+					$user_online_link = '<i>' . $row['username'] . '</i>';
+					$logged_hidden_online++;
+				}
+
+				if ($row['user_allow_viewonline'] || $auth->acl_get('a_'))
+				{
+					$user_online_link = '<a href="' . "memberlist.$phpEx$SID&amp;mode=viewprofile&amp;u=" . $row['user_id'] . '">' . $user_online_link . '</a>';
+					$online_userlist .= ($online_userlist != '') ? ', ' . $user_online_link : $user_online_link;
+				}
 			}
 
-			if ($row['user_allow_viewonline'] || $auth->acl_get('a_'))
+			$prev_user_id = $row['user_id'];
+		}
+		else
+		{
+			// Skip multiple sessions for one user
+			if ($row['session_ip'] != $prev_session_ip)
 			{
-				$user_online_link = '<a href="' . "memberlist.$phpEx$SID&amp;mode=viewprofile&amp;u=" . $row['user_id'] . '">' . $user_online_link . '</a>';
-				$online_userlist .= ($online_userlist != '') ? ', ' . $user_online_link : $user_online_link;
+				$guests_online++;
 			}
 		}
 
-		$prev_user_id = $row['user_id'];
+		$prev_session_ip = $row['session_ip'];
 	}
-	else
+
+	if ($online_userlist == '')
 	{
-		// Skip multiple sessions for one user
-		if ($row['session_ip'] != $prev_session_ip)
-		{
-			$guests_online++;
-		}
+		$online_userlist = $user->lang['NONE'];
 	}
 
-	$prev_session_ip = $row['session_ip'];
-}
-
-if ($online_userlist == '')
-{
-	$online_userlist = $user->lang['NONE'];
-}
-
-if (empty($_REQUEST['f']))
-{
-	$online_userlist = $user->lang['Registered_users'] . ' ' . $online_userlist;
-}
-else
-{
-	$l_online = ($guests_online == 1) ? $user->lang['Browsing_forum_guest'] : $user->lang['Browsing_forum_guests'];
-	$online_userlist = sprintf($l_online, $online_userlist, $guests_online);
-}
+	if (empty($_REQUEST['f']))
+	{
+		$online_userlist = $user->lang['Registered_users'] . ' ' . $online_userlist;
+	}
+	else
+	{
+		$l_online = ($guests_online == 1) ? $user->lang['Browsing_forum_guest'] : $user->lang['Browsing_forum_guests'];
+		$online_userlist = sprintf($l_online, $online_userlist, $guests_online);
+	}
 
-$total_online_users = $logged_visible_online + $logged_hidden_online + $guests_online;
+	$total_online_users = $logged_visible_online + $logged_hidden_online + $guests_online;
 
-if ($total_online_users > $config['record_online_users'])
-{
-	set_config('record_online_users', $total_online_users);
-	set_config('record_online_date', time());
-}
+	if ($total_online_users > $config['record_online_users'])
+	{
+		set_config('record_online_users', $total_online_users);
+		set_config('record_online_date', time());
+	}
 
-// Build online listing
-$vars_online = array(
-	'ONLINE'=> array('total_online_users', 'l_t_user_s'),
-	'REG'	=> array('logged_visible_online', 'l_r_user_s'),
-	'HIDDEN'=> array('logged_hidden_online', 'l_h_user_s'),
-	'GUEST'	=> array('guests_online', 'l_g_user_s')
-);
+	// Build online listing
+	$vars_online = array(
+		'ONLINE'=> array('total_online_users', 'l_t_user_s'),
+		'REG'	=> array('logged_visible_online', 'l_r_user_s'),
+		'HIDDEN'=> array('logged_hidden_online', 'l_h_user_s'),
+		'GUEST'	=> array('guests_online', 'l_g_user_s')
+	);
 
-foreach ($vars_online as $l_prefix => $var_ary)
-{
-	switch ($$var_ary[0])
+	foreach ($vars_online as $l_prefix => $var_ary)
 	{
-		case 0:
-			$$var_ary[1] = $user->lang[$l_prefix . '_USERS_ZERO_TOTAL'];
-			break;
+		switch ($$var_ary[0])
+		{
+			case 0:
+				$$var_ary[1] = $user->lang[$l_prefix . '_USERS_ZERO_TOTAL'];
+				break;
 
-		case 1:
-			$$var_ary[1] = $user->lang[$l_prefix . '_USER_TOTAL'];
-			break;
+			case 1:
+				$$var_ary[1] = $user->lang[$l_prefix . '_USER_TOTAL'];
+				break;
 
-		default:
-			$$var_ary[1] = $user->lang[$l_prefix . '_USERS_TOTAL'];
-			break;
+			default:
+				$$var_ary[1] = $user->lang[$l_prefix . '_USERS_TOTAL'];
+				break;
+		}
 	}
+	unset($vars_online);
+
+	$l_online_users = sprintf($l_t_user_s, $total_online_users);
+	$l_online_users .= sprintf($l_r_user_s, $logged_visible_online);
+	$l_online_users .= sprintf($l_h_user_s, $logged_hidden_online);
+	$l_online_users .= sprintf($l_g_user_s, $guests_online);
+	$l_online_record = sprintf($user->lang['RECORD_ONLINE_USERS'], $config['record_online_users'], $user->format_date($config['record_online_date']));
+	$l_online_time = ($config['load_online_time'] == 1) ? 'VIEW_ONLINE_TIME' : 'VIEW_ONLINE_TIMES';
+	$l_online_time = sprintf($user->lang[$l_online_time], $config['load_online_time']);
 }
-unset($vars_online);
-
-$l_online_users = sprintf($l_t_user_s, $total_online_users);
-$l_online_users .= sprintf($l_r_user_s, $logged_visible_online);
-$l_online_users .= sprintf($l_h_user_s, $logged_hidden_online);
-$l_online_users .= sprintf($l_g_user_s, $guests_online);
 
 
 // Obtain number of new private messages if user is logged in
@@ -248,13 +251,14 @@ $template->assign_vars(array(
 	'CURRENT_TIME' 					=> sprintf($user->lang['CURRENT_TIME'], $user->format_date(time())),
 	'TOTAL_USERS_ONLINE' 			=> $l_online_users,
 	'LOGGED_IN_USER_LIST' 			=> $online_userlist,
-	'RECORD_USERS' 					=> sprintf($user->lang['RECORD_ONLINE_USERS'], $config['record_online_users'], $user->format_date($config['record_online_date'])),
+	'RECORD_USERS' 					=> $l_online_record,
 	'PRIVATE_MESSAGE_INFO' 			=> $l_privmsgs_text,
 	'PRIVATE_MESSAGE_NEW_FLAG'		=> $s_privmsg_new,
 	'PRIVATE_MESSAGE_INFO_UNREAD' 	=> $l_privmsgs_text_unread,
 
 	'L_LOGIN_LOGOUT' 	=> $l_login_logout,
-	'L_INDEX' 			=> $user->lang['FORUM_INDEX'],
+	'L_INDEX' 			=> $user->lang['FORUM_INDEX'], 
+	'L_ONLINE_EXPLAIN'	=> $l_online_time, 
 
 	'U_PRIVATEMSGS'	=> 'ucp.'.$phpEx.$SID.'&amp;mode=pm&amp;folder=inbox',
 	'U_MEMBERLIST' 	=> 'memberlist.'.$phpEx.$SID,
@@ -279,7 +283,8 @@ $template->assign_vars(array(
 	'S_CONTENT_ENCODING' 	=> $user->lang['ENCODING'],
 	'S_CONTENT_DIR_LEFT' 	=> $user->lang['LEFT'],
 	'S_CONTENT_DIR_RIGHT' 	=> $user->lang['RIGHT'],
-	'S_TIMEZONE' 			=> ($user->data['user_dst'] || ($user->data['user_id'] == ANONYMOUS && $config['board_dst'])) ? sprintf($user->lang['ALL_TIMES'], $user->lang[$tz], $user->lang['tz']['dst']) : sprintf($user->lang['ALL_TIMES'], $user->lang[$tz], ''),
+	'S_TIMEZONE' 			=> ($user->data['user_dst'] || ($user->data['user_id'] == ANONYMOUS && $config['board_dst'])) ? sprintf($user->lang['ALL_TIMES'], $user->lang[$tz], $user->lang['tz']['dst']) : sprintf($user->lang['ALL_TIMES'], $user->lang[$tz], ''), 
+	'S_DISPLAY_ONLINE_LIST'	=> (!empty($config['load_online'])) ? 1 : 0, 
 
 	'T_STYLESHEET_DATA'	=> $user->theme['css_data'],
 	'T_STYLESHEET_LINK' => 'templates/' . $user->theme['css_external'],
diff --git a/phpBB/language/en/lang_main.php b/phpBB/language/en/lang_main.php
index 852a5d0136..22428c84f2 100644
--- a/phpBB/language/en/lang_main.php
+++ b/phpBB/language/en/lang_main.php
@@ -848,7 +848,8 @@ $lang = array(
 	'Guest_users_zero_online' => 'There are 0 Guest users online',
 	'Guest_user_online' => 'There is %d Guest user online',
 	'No_users_browsing' => 'There are no users currently browsing this forum',
-	'ONLINE_EXPLAIN' => 'This data is based on users active over the past five minutes',
+	'VIEW_ONLINE_TIME'	=> 'This data is based on users active over the past %d minute', // 1 minute
+	'VIEW_ONLINE_TIMES' => 'This data is based on users active over the past %d minutes', // 2 minutes, etc.
 	'Forum_Location' => 'Forum Location',
 	'Last_updated' => 'Last Updated',
 	'Forum_index' => 'Forum index',
diff --git a/phpBB/templates/subSilver/index_body.html b/phpBB/templates/subSilver/index_body.html
index faab6f0f67..f4ab35de3e 100644
--- a/phpBB/templates/subSilver/index_body.html
+++ b/phpBB/templates/subSilver/index_body.html
@@ -40,7 +40,7 @@
 	<!-- ENDIF -->
 	<!-- BEGINELSE -->
 	<tr>
-		<td class="row1" colspan="5" height="28" align="center">{L_NO_FORUMS}</td>
+		<td class="row1" colspan="5" height="28" align="center"><span class="gensmall">{L_NO_FORUMS}</span></td>
 	</tr>
 	<!-- END forumrow -->
 </table>
@@ -54,6 +54,7 @@
 
 <br clear="all" />
 
+<!-- IF S_DISPLAY_ONLINE_LIST -->
 <table class="tablebg" width="100%" cellspacing="1" cellpadding="2" border="0">
 	<tr>
 		<td class="cat" colspan="2" height="28"><a class="cattitle" href="{U_VIEWONLINE}">{L_WHO_IS_ONLINE}</a></td>
@@ -68,6 +69,7 @@
 	</tr>
 	<!-- ENDIF -->
 </table>
+<!-- ENDIF -->
 
 <!-- IF not S_USER_LOGGED_IN -->
 <form method="post" action="{S_LOGIN_ACTION}"><table class="tablebg" width="100%" cellspacing="1" cellpadding="3" border="0">
author	Paul S. Owen <psotfx@users.sourceforge.net>	2003-04-15 22:57:05 +0000
committer	Paul S. Owen <psotfx@users.sourceforge.net>	2003-04-15 22:57:05 +0000
commit	dce7e4c2b82ce4cc99ac2042f765e68c0c97e92b (patch)
tree	5e0e17f505b9c7bfe7c9b0add4f5af63419383b0
parent	d9dcb502a96744ce19127f92f08572a56a7bdf2a (diff)
download	forums-dce7e4c2b82ce4cc99ac2042f765e68c0c97e92b.tar forums-dce7e4c2b82ce4cc99ac2042f765e68c0c97e92b.tar.gz forums-dce7e4c2b82ce4cc99ac2042f765e68c0c97e92b.tar.bz2 forums-dce7e4c2b82ce4cc99ac2042f765e68c0c97e92b.tar.xz forums-dce7e4c2b82ce4cc99ac2042f765e68c0c97e92b.zip