[buddypress-trac] [BuddyPress] #4814: spammer can create group
    buddypress-trac 
    noreply at wordpress.org
       
    Wed May  1 18:12:26 UTC 2013
    
    
  
#4814: spammer can create group
--------------------------+-----------------------
 Reporter:  intimez       |       Owner:  r-a-y
     Type:  defect (bug)  |      Status:  reopened
 Priority:  normal        |   Milestone:  1.8
Component:  Groups        |     Version:  1.7
 Severity:  critical      |  Resolution:
 Keywords:  dev-feedback  |
--------------------------+-----------------------
Comment (by r-a-y):
 > Something tells me the approach is probably fine, though it is a drastic
 behavioral change to make without having discussed it in a dev chat
 together.
 Sorry for not discussing this in a dev chat!
 I don't mind how we go about this issue, but we do have to address how a
 live user that is marked as a spammer is still able to do things as
 mentioned at the beginning of this ticket.
 My two cents can be
 [https://buddypress.trac.wordpress.org/ticket/4814#comment:5 found above].
 ----
 > The wp_die() approach gets the point across, but maybe it should be a
 more informative message/page within the theme? Like a 404 but for logged
 in users.
 That could work as well.  Perhaps redirect to homepage and use
 `bp_core_add_message()` saying the user is a spammer?
 ----
 > If we are going to kill the page like this, we should use the existing
 bp_is_user_spammer() and bp_is_user_deleted() functions, rather than off-
 load the logic.
 I decided not to use `bp_is_user_spammer()`, because we've already queried
 that data for the logged-in user in the `$bp->loggedin_user` object.
 Thanks to your feedback, I've added that logic directly in the function
 now.  See r6893.
-- 
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/4814#comment:11>
BuddyPress <http://buddypress.org/>
BuddyPress
    
    
More information about the buddypress-trac
mailing list