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

Timestamp:

11/24/2015 10:55:03 PM (9 years ago)

Author:

coffee2code

Message:

Trac mentions: Adapt to notifications now dealing in user_nicename instead of user_login.

File:

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

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

@@ -60 +60 @@
 function wporg_mentions_limit_to_security_team( $usernames, $data ) {
     require_once dirname( __DIR__ ) . '/slack/security-team.php';
-    $team = \Dotorg\Slack\Security_Team\get_security_team();
+    $team = \Dotorg\Slack\Security_Team\get_security_team( 'user_nicename' );
 
     if ( ! empty( $data['object']->cc ) ) {
@@ -80 +80 @@
 
 add_filter( 'wporg_notifications_notify_username', function( $notify, $username ) use ( $type, $payload, $wpdb ) {
+    // $username is user_nicename, but this function expects user_login.
+    $user = get_user_by( 'slug', $username );
+    if ( ! $user ) {
+        return false;
+    }
+    $username = $user->user_login;
+
     // Core Trac has notifications configured.
     if ( $payload->trac === 'core' ) {