Use bp_get_admin_url() in XProfile instead of hard-coded links

[JohnPBloch]

There were several areas in XProfile that used hard-coded URLs instead of the BuddyPress helper function. Replacing hard-coded addresses with addresses generated by the BP helper will improve overall stability and add flexibility to the plugin overall.

[johnjamesjacoby]

I'll test this out in the multitude of environment setups that we support.

I think relying on bp_get_admin_url() is OK, but there may be weirdness with single-site activations on multisite setups, where site-admins wouldn't normally have the option to edit user settings.

[hnla]

@johnjamesjacoby does the patch still need testing or do you want to squeeze it in or punt to 3.0?

[DJPaul]

@johnjamesjacoby I'm inclined to merge this patch -- are you able to spend a little bit of time testing on the scenario you mentioned previously, or at least clarify that configuration so I can test it?

[johnjamesjacoby]

In 11808:

XProfile: use bp_get_admin_url() where appropriate.

This change updates all inline and querystring-based URL references to users.php to instead use the built-in supplied wrapper function. This comes with a few benefits:

• URLs will be consistent across BuddyPress installation configurations (rather than relative to the current admin dashboard area)
• Code is cleaner and easier to understand
• Variable IDs are now cast to integers as needed (where they were not all previously)

Bonus: also updates a few bp_get_admin_url() calls in the about page to use the second parameter of add_query_arg() rather than wrapping it entirely, to match the expected usage format of all add_query_arg() calls.

Fixes #7536. Props JohnPBloch.

[johnjamesjacoby]

Related: we may want to consider adding protected methods to the field and field-group classes, so they can self-generate these URLs. It would reduce code duplication, remove some logic from the local scope of the functions we use to output the admin pages, and make these URLs testable so we can ensure their stability for future enhancements.

Imagine something like:

$group->get_edit_url()

...or...

$group->get_url( 'edit' )