[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