Changeset 7628

Timestamp:

11/30/2013 06:47:24 AM (12 years ago)

Author:

johnjamesjacoby

Message:

Swap old Core notifications functions for new native Notifications components ones. Props r-a-y. See #5266.

Location:

trunk/bp-notifications

Files:

• bp-notifications-adminbar.php (modified) (1 diff)
• bp-notifications-buddybar.php (modified) (2 diffs)

trunk/bp-notifications/bp-notifications-adminbar.php

@@ -25 +25 @@
     }
 
-    $notifications = bp_core_get_notifications_for_user( bp_loggedin_user_id(), 'object' );
+    $notifications = bp_notifications_get_notifications_for_user( bp_loggedin_user_id(), 'object' );
     $count         = ! empty( $notifications ) ? count( $notifications ) : 0;
     $alert_class   = (int) $count > 0 ? 'pending-count alert' : 'count no-alert';
-    $menu_title    = '<span id="ab-pending-notifications" class="' . $alert_class . '">' . $count . '</span>';
+    $menu_title    = '<span id="ab-pending-notifications" class="' . $alert_class . '">' . number_format_i18n( $count ) . '</span>';
     $menu_link     = trailingslashit( bp_loggedin_user_domain() . bp_get_notifications_slug() );
 

trunk/bp-notifications/bp-notifications-buddybar.php

@@ -25 +25 @@
     }
 
-    echo '<li id="bp-adminbar-notifications-menu"><a href="' . bp_loggedin_user_domain() . '">';
+    echo '<li id="bp-adminbar-notifications-menu"><a href="' . esc_url( bp_loggedin_user_domain() ) . '">';
     _e( 'Notifications', 'buddypress' );
 
-    if ( $notifications = bp_core_get_notifications_for_user( bp_loggedin_user_id() ) ) : ?>
-        <span><?php echo count( $notifications ) ?></span>
+    if ( $notifications = bp_notifications_get_notifications_for_user( bp_loggedin_user_id() ) ) : ?>
+        <span><?php echo number_format_i18n( $notifications ); ?></span>
     <?php
     endif;
@@ -47 +47 @@
     } else { ?>
 
-        <li><a href="<?php echo bp_loggedin_user_domain() ?>"><?php _e( 'No new notifications.', 'buddypress' ); ?></a></li>
+        <li><a href="<?php echo esc_url( bp_loggedin_user_domain() ); ?>"><?php _e( 'No new notifications.', 'buddypress' ); ?></a></li>
 
     <?php