WordPress.org

Making WordPress.org

Changeset 8089


Ignore:
Timestamp:
01/17/2019 02:46:31 AM (21 months ago)
Author:
dd32
Message:

Trac: When performing @mentions, use the users nicename when the username contains a space.

This keeps the current ProperCase usernames and only suggests nicenames when required.

Fixes #2539.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • sites/trunk/wordpress.org/public_html/style/trac/wp-trac.js

    r8055 r8089  
    808808                    $( $( '.change .username' ).get().reverse() ).each( function() {
    809809                        var username = $(this).data( 'username' );
     810
     811                        // Override the username with the nicename if it differs by more than just case (ie. spaces, etc)
     812                        if (
     813                            $(this).data( 'nicename' ) &&
     814                            username.toLowerCase() != $(this).data( 'nicename' ).toLowerCase() &&
     815                            wpTrac.currentUser !== username
     816                        ) {
     817                            username = $(this).data( 'nicename' );
     818                        }
     819
    810820                        if (
    811821                            typeof username !== 'undefined' &&
     
    813823                            -1 === $.inArray( username, exclude )
    814824                        ) {
    815                             users.push( $(this).data( 'username' ) );
     825                            users.push( username );
    816826                        }
    817827                    });
     
    819829                    // Add ticket reporter.
    820830                    var ticketReporter = $.trim( $( '#ticket td[headers="h_reporter"]' ).text() );
     831                    var ticketReporterNicename = $( '#ticket td[headers="h_reporter"] a' ).data( 'nicename' );
     832                    // Override the username with the nicename if it differs by more than just case (ie. spaces, etc)
     833                    if (
     834                        ticketReporter &&
     835                        ticketReporterNicename &&
     836                        ticketReporter !== wpTrac.currentUser &&
     837                        ticketReporter.toLowerCase() !== ticketReporterNicename.toLowerCase()
     838                    ) {
     839                        ticketReporter = ticketReporterNicename;
     840                    }
     841
    821842                    if ( ticketReporter && -1 === $.inArray( ticketReporter, users ) ) {
    822843                        users.push( ticketReporter );
Note: See TracChangeset for help on using the changeset viewer.