Ticket #3769: 3769.02.patch

bp-core/bp-core-component.php

@@ -89 +89 @@
          *  - id: Unique ID (for internal identification). Letters, numbers, and underscores only
          *  - name: Unique name. This should be a translatable name, eg __( 'Groups', 'buddypress' )
          *  - path: The file path for the component's files. Used by BP_Component::includes()
+         *  - params: Array of additional parameters used for the component loader. Currently accepts 'adminbar_position'.
          * @uses bp_Component::setup_actions() Setup the hooks and actions
          */
-        function start( $id, $name, $path ) {
+        function start( $id, $name, $path, $params = false ) {
                 // Internal identifier of component
                 $this->id   = $id;
 
@@ -101 +102 @@
                 // Path for includes
                 $this->path = $path;
 
+                // Miscellaneous component params
+                if ( ! empty( $params ) ) {
+                        // set the priority of the component's menu under the WP Toolbar's "My Account" menu
+                        if ( ! empty( $params['adminbar_position'] ) && is_numeric( $params['adminbar_position'] ) )
+                                $this->adminbar_position = $params['adminbar_position'];
+                }
+                // Set defaults if not passed
+                else {
+                        $this->adminbar_position = 10;
+                }
+
                 // Move on to the next step
                 $this->setup_actions();
         }
@@ -250 +262 @@
                 add_action( 'bp_setup_nav',              array ( $this, 'setup_nav'              ), 10 );
 
                 // Setup WP Toolbar menus
-                add_action( 'bp_setup_admin_bar',        array ( $this, 'setup_admin_bar'        ), 10 );
+                add_action( 'bp_setup_admin_bar',        array ( $this, 'setup_admin_bar'        ), $this->adminbar_position );
 
                 // Setup component title
                 add_action( 'bp_setup_title',            array ( $this, 'setup_title'            ), 10 );

bp-friends/bp-friends-loader.php

@@ -22 +22 @@
                 parent::start(
                         'friends',
                         __( 'Friend Connections', 'buddypress' ),
-                        BP_PLUGIN_DIR
+                        BP_PLUGIN_DIR,
+                        array(
+                                'adminbar_position' => 30
+                        )
                 );
         }