Make function "bp_core_menu_highlight_parent_page" multisite-aware

[martinutopia]

In my WP-multisite I want to loop over posts in a WP-Menu of my main site that is no buddypress-Site. So I'm using switch_to_blog() to get the data from my main site.

I get an error in the file buddypress/bp-core/bp-core-filters.php on line 199 that the second parameter of in_array is not an array. (it's null because the get_post() on line 198 returns null)

My suggested fix is to test if we are in a switch_to_blogs context. For that I suggest following fix on line 171 (add a check of ms_is_switched()):

if ( ! is_buddypress() || ms_is_switched() ) 

Thanks,
Martin

[DJPaul]

Hi @martinutopia

Thanks for the bug report (your first, I think; welcome!).

I think we'll fix this with our bp_is_root_blog function rather than ms_is_switched (the latter function is only present if multisite is enabled). Alternately, BuddyPress could do its own switch_to_blog inside the function. I need to spend more than 5 minutes considering the best fix to be sure, but we can definitely get this in for our next major release (2.8; early next year).

For now, I'd suggest you hook a function to that action with a higher priority, and inside that function callback, unhook the BuddyPress function from that action if ms_is_switched since that apparently works well for you.

bp_is_root_blog check added on bp_core_menu_highlight_parent_page function

Trac ticket: ​7308

@imath can you please review this pull request.
This is my first contribution to buddypress :)

[rudlinkon]

The PR looks good to me according to @DJPaul's suggestions. Thanks @robinwpdeveloper for your first contribution in BuddyPress

@robinwpdeveloper Sure & with great pleasure! Thanks a lot for this first PR 😍. Don't hesitate to pull more requests 👌

[rudlinkon]

return from the function if not bp_is_root_blog()

[rudlinkon]

Replying to imath:

​imath commented on ​PR #22:

@robinwpdeveloper Sure & with great pleasure! Thanks a lot for this first PR 😍. Don't hesitate to pull more requests 👌

Yes, you are absolutely right we have to return from the function if it is not bp_is_root_blog(). I just uploaded a patch you may check it.

FYI, a patch was added to our ​Trac ticket about it, so I'm going to close this PR. I hope you won't be disappointed, no worries about the credit/prop, I'll add your WordPress username to my commit.

[imath]

In 13322:

Only filter pages/menu list CSS classes on the root blog

To make sure a pages list or a menu highlight the current BuddyPress page or the parent/ancestor of a displayed component' single item, we use a filter that should only run on the WordPress site BuddyPress is activated. Taking this precaution will save our users from potential warning notices that were brought to our attention when BuddyPress is network activated.

Props robinwpdeveloper, martinutopia, DJPaul, rudlinkon

Closes ​https://github.com/buddypress/buddypress/pull/22
Fixes #7308

Thank you very much @imath :)