Changeset 7521 for trunk/bp-members/bp-members-buddybar.php

Timestamp:

11/07/2013 05:15:51 PM (13 years ago)

Author:

johnjamesjacoby

Message:

First pass at bp-notifications component. Includes:

• Backwards compatibility for old core functions.
• Screens, functions, classes, and actions for Notifications.
• Improved class methods for getting, updating, and deleting notifications.
• Template parts in bp-legacy for theme compatibility.
• A few basic unit tests.

@todo's:

• Improve template output with dedicated functions, markup, classes, et all.
• More unit tests.
• Pagination links.
• Auto-activate on update, so existing installations do not lose previously core functionality.

See #5148. Props johnjamesjacoby, boonebgorges, r-a-y.

File:

• trunk/bp-members/bp-members-buddybar.php (modified) (1 diff)

trunk/bp-members/bp-members-buddybar.php

@@ -18 +18 @@
 function bp_adminbar_notifications_menu() {
 
-        if ( !is_user_logged_in() )
+        // Bail if notifications is not active
+        if ( ! bp_is_active( 'notifications' ) ) {
                 return false;
-
-        echo '<li id="bp-adminbar-notifications-menu"><a href="' . 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>
-        <?php
         }
 
-        echo '</a>';
-        echo '<ul>';
-
-        if ( $notifications ) {
-                $counter = 0;
-                for ( $i = 0, $count = count( $notifications ); $i < $count; ++$i ) {
-                        $alt = ( 0 == $counter % 2 ) ? ' class="alt"' : ''; ?>
-
-                        <li<?php echo $alt ?>><?php echo $notifications[$i] ?></li>
-
-                        <?php $counter++;
-                }
-        } else { ?>
-
-                <li><a href="<?php echo bp_loggedin_user_domain() ?>"><?php _e( 'No new notifications.', 'buddypress' ); ?></a></li>
-
-        <?php
-        }
-
-        echo '</ul>';
-        echo '</li>';
+        bp_notifications_buddybar_menu();
 }
 add_action( 'bp_adminbar_menus', 'bp_adminbar_notifications_menu', 8 );