[buddypress-trac] [BuddyPress] #1925: '@' in username causes strange issues

buddypress-trac at lists.automattic.com buddypress-trac at lists.automattic.com
Mon Feb 15 03:39:45 UTC 2010


#1925: '@' in username causes strange issues
----------------------------------------------------+-----------------------
Reporter:  Zooney                                   |       Owner:     
    Type:  defect                                   |      Status:  new
Priority:  major                                    |   Milestone:  1.2
Keywords:  needs-patch, needs-testing, 2nd-opinion  |  
----------------------------------------------------+-----------------------
Changes (by junsuijin):

  * keywords:  @, username => needs-patch, needs-testing, 2nd-opinion


Comment:

 Looking at wp-includes/formatting.php (
 http://core.trac.wordpress.org/browser/trunk/wp-
 includes/formatting.php#L730 ) I see that {{{sanitize_user}}} allows
 characters that bp-activity/bp-activity-filters.php
 {{{bp_activity_at_name_filter}}} does not handle ({{{_ .-@}}}). I suspect
 this may have something to do with these problems since the activity
 streams get updated after the group creation and profile post. If this
 isn't the cause of the issue, it should be fixed in a separate ticket. I'm
 not sure how we can allow spaces in usernames...maybe using + character in
 activity content (like @some+guy), since it cannot be part of a username?
 Strict mode or further filter could also fix the issue but I don't think
 those are valid means of solving the problem(s).

-- 
Ticket URL: <http://trac.buddypress.org/ticket/1925#comment:3>
BuddyPress <http://buddypress.org/>
BuddyPress


More information about the buddypress-trac mailing list