[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 henry.wright):

 I think your suggestion to put a check directly into
 {{{bp_notifications_add_notification()}}} is far better than making
 changes across the codebase:

 1. It will eliminate the problem of duplicate notifications wherever they
 are generated from in BP core.
 2. It'll eliminate the problem for any plugins that may using the
 {{{bp_notifications_add_notification()}}} function.

 The flag is a great idea as it will let
 {{{bp_notifications_add_notification()}}} retain it's flexibility if
 duplicate notifications are ever needed.

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