[buddypress-trac] [BuddyPress Trac] #5297: Disallow multiple core and email notifications for same item

[buddypress-trac]

#5297: Disallow multiple core and email notifications for same item
---------------------------+------------------
 Reporter:  henrywright    |       Owner:
     Type:  defect (bug)   |      Status:  new
 Priority:  normal         |   Milestone:  2.0
Component:  Notifications  |     Version:
 Severity:  normal         |  Resolution:
 Keywords:                 |
---------------------------+------------------

Comment (by boonebgorges):

 I don't think there's a list readily available, but you could compile one
 by searching our codebase for uses of
 `bp_notifications_add_notification()`.

 Another consideration is to put a check directly into
 `bp_notifications_add_notification()`. I don't know how others feel about
 that, but I'd think that (at least in the majority of cases) there's no
 reason ever to allow for duplicate entries in the notifications table. So
 we could make our API function idempotent by default, with perhaps a flag
 `allow_duplicate` that could be passed to bypass this block.

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