[buddypress-trac] [BuddyPress] #4622: Spaces allowed in screen names since 1.6 update

buddypress-trac noreply at wordpress.org
Sat Feb 16 00:28:12 UTC 2013


#4622: Spaces allowed in screen names since 1.6 update
------------------------------+-----------------------------
 Reporter:  oceanwidedesigns  |       Owner:
     Type:  defect (bug)      |      Status:  new
 Priority:  normal            |   Milestone:  Future Release
Component:  Members           |     Version:  1.6
 Severity:  normal            |  Resolution:
 Keywords:  dev-feedback      |
------------------------------+-----------------------------

Comment (by boonebgorges):

 The WooCommerce issue sounds like a nasty bug, though as you note, it's
 not really ours to fix.

 If you're having problems with an infinite loop, you can fix it like this:

 {{{
 function my_sanitizing_function() {
     remove_filter( 'sanitize_user', 'my_sanitizing_function' );
     $foo = bp_core_strip_username_spaces( $bar );
     /// etc
 }
 add_filter( 'sanitize_user', 'my_sanitizing_function' );
 }}}

 I'll reiterate the core team's belief that BP has no business intervening
 in the username validation process. If you want to impose specific
 username requirements, there should be core filters in place to allow you
 to do so. I haven't seen any compelling reason why BP needs to add any
 global restrictions on top of WP's.

 That said, it does look like there's still an issue to be addressed here,
 which is that sometimes BP replaces spaces with hyphens but doesn't tell
 the user about it. We need to clear up this situation with better error
 reporting. As far as I can see, that's the reason the ticket is still
 open.

-- 
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/4622#comment:14>
BuddyPress <http://buddypress.org/>
BuddyPress


More information about the buddypress-trac mailing list