Changeset 1182

Timestamp:

01/23/2015 09:45:46 PM (8 years ago)

Author:

nacin

Message:

Trac mentions: Restrict notifications to individuals who have posted to a Trac before. see #830.

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

@@ -23 +23 @@
 $user        = get_user_by( 'login', $user_login );
 
-if ( ! $wpdb->get_var( $wpdb->prepare( "SELECT user_login FROM trac_users WHERE user_login = %s", $user_login ) ) ) {
+function wporg_user_has_visited_trac( $user_login ) {
+    global $wpdb;
+    return (bool) $wpdb->get_var( $wpdb->prepare( "SELECT user_login FROM trac_users WHERE user_login = %s", $user_login ) );
+}
+
+if ( ! wporg_user_has_visited_trac( $user_login ) ) {
     $wpdb->insert( 'trac_users', compact( 'user_login' ) );
 }
+
+add_filter( 'wporg_notifications_notify_username', function( $notify, $username ) use ( $type, $payload, $wpdb ) {
+    if ( $type === 'ticket' ) {
+        // Don't need a query to say we can notify the owner and reporter.
+        if ( $username === $payload->owner || $username === $payload->reporter ) {
+            return true;
+        }
+    }
+
+    if ( wporg_user_has_visited_trac( $username ) ) {
+        return true;
+    }
+    return $notify;
+}, 10, 2 );
 
 $notif->match_notify( array(