[buddypress-trac] [BuddyPress Trac] #6681: Delete all of a user's notifications after they're deleted

buddypress-trac noreply at wordpress.org
Mon Oct 26 18:50:21 UTC 2015


#6681: Delete all of a user's notifications after they're deleted
----------------------------------------+--------------------
 Reporter:  henry.wright                |       Owner:
     Type:  enhancement                 |      Status:  new
 Priority:  normal                      |   Milestone:  2.5
Component:  Component - Notifications   |     Version:  2.3.3
 Severity:  normal                      |  Resolution:
 Keywords:  has-patch needs-unit-tests  |
----------------------------------------+--------------------

Comment (by boonebgorges):

 > Do functions in bp-notifications fire if the notifications component is
 disabled?

 No. But I think we shouldn't worry about that. Components can't be
 expected to clean up after themselves if they've been turned off. Moving
 all of this logic inside of bp-core is going to create all sorts of
 `bp_is_active()` headaches that I'd like to avoid. In any case, it's not
 going to break anyone's site if this data is not cleared up. So let's keep
 it in bp-notifications.

--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/6681#comment:6>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac


More information about the buddypress-trac mailing list