Changeset 5696 for trunk/bp-messages/bp-messages-loader.php

Timestamp:

02/10/2012 09:02:53 PM (15 years ago)

Author:

johnjamesjacoby

Message:

Remove unneeded globals and clean up some code in Messages component. See #3989.

File:

• trunk/bp-messages/bp-messages-loader.php (modified) (3 diffs)

trunk/bp-messages/bp-messages-loader.php

@@ -1 +1 @@
 <?php
+
 /**
- * BuddyPress Private Messages Loader
+ * BuddyPress Messages Loader
  *
  * A private messages component, for users to send messages to each other
  *
  * @package BuddyPress
- * @subpackage Messages Core
+ * @subpackage MessagesLoader
  */
 
@@ -93 +94 @@
                 global $bp;
 
-                $name = sprintf( __( 'Messages <span>%s</span>', 'buddypress' ), bp_get_total_unread_messages_count() );
+                $sub_nav = array();
+                $name    = sprintf( __( 'Messages <span>%s</span>', 'buddypress' ), bp_get_total_unread_messages_count() );
 
                 // Add 'Messages' to the main navigation
@@ -174 +176 @@
 
                         // Unread message count
-                        if ( $count = messages_get_unread_count() ) {
-                                $title = sprintf( __( 'Messages <span class="count">%s</span>', 'buddypress' ), $count );
-                                $inbox = sprintf( __( 'Inbox <span class="count">%s</span>',    'buddypress' ), $count );
+                        $count = messages_get_unread_count();
+                        if ( !empty( $count ) ) {
+                                $title = sprintf( __( 'Messages <span class="count">%s</span>', 'buddypress' ), number_format_i18n( $count ) );
+                                $inbox = sprintf( __( 'Inbox <span class="count">%s</span>',    'buddypress' ), number_format_i18n( $count ) );
                         } else {
                                 $title = __( 'Messages', 'buddypress' );