Changeset 1182

Timestamp:

01/23/2015 09:45:46 PM (12 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(