Deleting profile fields - UX issue

[rogercoathup]

In wp-admin (users.. profile fields).

If I delete a profile field from a profile field group, it always redirects me back to the base profile field group tab.

For a better UX experience, it should redirect you back to the field group you were working on.

[mercime]

Confirming that this is still an issue as of BP 2.7 trunk. Needs fix for better UX and Accessibility.

[DJPaul]

Eeep, scary old code! :) Rather than tack a new part on the end, let's improve the whole thing. While there isn't an output escaping issue as such, the way it has been done already is less than perfect.

We need to esc_url the entire value for the href attribute, and use either add_query_arg or sprintf to concatenate the hardcoded URL and the variables together. Think also about how to escape the values you're concatenating (strings would usually be passed through urlencode but it looks like these are integers, so you could do a simple cast to make that clear).

I think you'd do this above the HTML part of the function, because trying to do all that inline would make the line length really long.

[slaffik]

In 11345:

Profile: redirect to group after profile field deleted.

When deleting a profile field we should redirect back to a fields group, where the field was deleted from. That's a UX improvement.

Props rogercoathup, DJPaul.
Fixes #4646.