Slashes in Group description on Profile Administration screen

[imath]

A screenshot to describe the issue

The group description in Profile Field administration site.url/wp-admin/users.php?page=bp-profile-setup is not "stripslashed".

[DJPaul]

I think using the bp_get_group_description filter here would be more appropriate and inline with how the group description would normally be rendered.

[mahype]

Parts patched on Contributor day at the WordCamp. Not finished yet.

[DJPaul]

Okay, this was fun. Maybe not a good-first-bug due to its complexity, but @mahype and I had fun exploring this issue at Wordcamp Europe contributor day. :)

1) Patch adds filters to parts of the xprofile wp-admin templates so we're running group description and name through the same output filters that are used in the front-end template parts (for stripslashes).

2) This change revealed another bug where we are storing slashes in the database (this is a known issue and we have somewhere a very old ticket), but in this case, after applying the fix in 1), editing a profile group name or description multiple times would show the extra slashes again in the templates (only stripping the first slash).

So the patch adds a stripslashes to profile group name and description before_save filters to fix the root issue.
Keeping the existing on-display stripslashes maintains backwards compatibility.

3) Which revealed a fun bug where, if you save a profile group without changing *any* details, it didn't work and we threw an error. It looks like someone misinterpreted the return values of WPDB's query and get_results functions. So this was corrected.

No unit tests, but it's now time for a siesta. :)

[DJPaul]

The third part of this issue has been separately fixed by @boonebgorges in r10002

[djpaul]

In 10239:

xprofile admin: fix handling of slashes in xprofile group name/desc

Add filters to parts of the xprofile wp-admin templates so we're
running group description and name through the same output filters that
are used in the front-end template parts (for stripslashes).

Also adds a stripslashes to profile group name and description
before_save filters to fix the root issue. By keeping the existing
on-display stripslashes, we maintain backwards compatibility.

Fixes #6081

Props mahype, djpaul