[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