Wrong users menu settings in backend

[ipm-frommen]

The profile_admin_head function of the BP_Members_Admin class is added to the dynamic admin_head-{$hook_suffix} filter for a few cases. The problem with this is that it manipulates both the global $parent_file and $submenu_file variables, no matter what.

This results in the wrong submenu item (i.e., Your Profile) being highlighted, even if you are on the users.php page.

The attached patch fixes this behavior by improving/fixing the mechanism for setting the is_self_profile property. It does that by restricting the user ID check for the according page only.

[ipm-frommen]

Okay, I just saw on GitHub that there already is a fix for the problem. However, I think my approach is more straightforward than what is currently done. The function is added to a dynamic filter hook with the $hook_suffix global, so it should be this global on which the restriction is based. Just my two cents.

[johnjamesjacoby]

• Does this happen in single or multi-site?
• Site admin or network admin?
• Can you confirm your fix does not affect other conditions?
• Any issues with multiblog setups too?

[ipm-frommen]

As already stated in my comment, the patch is just a (in my opinion) more straightforward version of what you already did in ​this commit. I am working with the current version, 2.1.1, that's why I didn't see this has been taken care of (once the new version will be published).

[johnjamesjacoby]

The current version is 2.2, released Wednesday. If this is an issue in 2.1.1, updating to 2.2 will fix it for you, and should do so for all installation types.

[DJPaul]

I think @ipm-frommen's patch looks a little less brittle than the solution in 2.2, but I don't know if making a change for its own sake is something we want to do.

[DJPaul]

Since this is fixed, let's not fix it again for purely cosmetic gains.