Changeset 7628 for trunk/bp-notifications/bp-notifications-buddybar.php

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.

File:

• trunk/bp-notifications/bp-notifications-buddybar.php (modified) (2 diffs)

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