[buddypress-trac] [BuddyPress Trac] #5093: Leading whitespace in display names messes with alphabetical sort

buddypress-trac noreply at wordpress.org
Thu Dec 29 03:16:43 UTC 2016


#5093: Leading whitespace in display names messes with alphabetical sort
---------------------------------------+-------------------------
 Reporter:  boonebgorges               |       Owner:
     Type:  enhancement                |      Status:  closed
 Priority:  lowest                     |   Milestone:
Component:  Core                       |     Version:
 Severity:  minor                      |  Resolution:  worksforme
 Keywords:  needs-testing needs-patch  |
---------------------------------------+-------------------------
Changes (by boonebgorges):

 * status:  new => closed
 * resolution:   => worksforme
 * milestone:  2.8 =>


Comment:

 On further testing, it appears that field data is, in fact, stripped on
 save. I can't see at a glance when this change took place, but the stack
 trace is:

 * `xprofile_set_field_data()` causes `BP_XProfile_ProfileData::save()` to
 be run
 * `save()` runs `xprofile_data_value_before_save` callbacks, including
 https://buddypress.trac.wordpress.org/browser/trunk/src/bp-xprofile/bp-
 xprofile-filters.php?marks=55#L51
 * `xprofile_sanitize_data_value_before_save()` runs through the
 'xprofile_filtered_data_value_before_save' filter
 https://buddypress.trac.wordpress.org/browser/trunk/src/bp-xprofile/bp-
 xprofile-filters.php?marks=188#L153
 * which is trimmed https://buddypress.trac.wordpress.org/browser/trunk/src
 /bp-xprofile/bp-xprofile-filters.php?marks=56#L51

--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/5093#comment:4>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac


More information about the buddypress-trac mailing list