#55096, don't show online users in block if nobody online, patch by tenrapid

modules/user.module

@@ -601,13 +601,15 @@ function user_block($op = 'list', $delta = 0, $edit = array()) {
 
           // Display a list of currently online users.
           $max_users = variable_get('user_block_max_list_count', 10);
-          $items = array();
+          if ($total_users && $max_users) {
+            $items = array();
 
-          while ($max_users-- && $account = db_fetch_object($users)) {
-            $items[] = $account;
-          }
+            while ($max_users-- && $account = db_fetch_object($users)) {
+              $items[] = $account;
+            }
 
-          $output .= theme('user_list', $items, t('Online users'));
+            $output .= theme('user_list', $items, t('Online users'));
+          }
 
           $block['subject'] = t('Who\'s online');
           $block['content'] = $output;

modules/user/user.module

@@ -601,13 +601,15 @@ function user_block($op = 'list', $delta = 0, $edit = array()) {
 
           // Display a list of currently online users.
           $max_users = variable_get('user_block_max_list_count', 10);
-          $items = array();
+          if ($total_users && $max_users) {
+            $items = array();
 
-          while ($max_users-- && $account = db_fetch_object($users)) {
-            $items[] = $account;
-          }
+            while ($max_users-- && $account = db_fetch_object($users)) {
+              $items[] = $account;
+            }
 
-          $output .= theme('user_list', $items, t('Online users'));
+            $output .= theme('user_list', $items, t('Online users'));
+          }
 
           $block['subject'] = t('Who\'s online');
           $block['content'] = $output;