Changeset 1187 for sites/trunk/api.wordpress.org/public_html/dotorg/trac/mentions-handler.php

Timestamp:

01/23/2015 11:49:51 PM (10 years ago)

Author:

nacin

Message:

Avoid sending mention notifications for people specifically watching or already subscribed.

File:

• sites/trunk/api.wordpress.org/public_html/dotorg/trac/mentions-handler.php (modified) (1 diff)

sites/trunk/api.wordpress.org/public_html/dotorg/trac/mentions-handler.php

@@ -59 +59 @@
 
 add_filter( 'wporg_notifications_notify_username', function( $notify, $username ) use ( $type, $payload, $wpdb ) {
-    // Core Trac has notifications configured, see if the user has blocked the ticket.
+    // Core Trac has notifications configured.
     if ( $payload->trac === 'core' ) {
+        // See if the user has blocked the ticket, are watching it already, or have already been mentioned.
         $status = wporg_get_trac_ticket_subscription_status( $username, $payload->ticket_id );
-        if ( BLOCKED === $status ) {
+        if ( BLOCKED === $status || SUBSCRIBED === $status || MENTIONED === $status ) {
             return false;
         }