WordPress.org
Get WordPress

Make WordPress Core

Ticket #26017: 26016.diff

File 26016.diff, 5.7 KB (added by dougwollison, 11 years ago)

Missed a couple spacing fixes

  • src/wp-admin/js/user-profile.js

     
    1 (function($){
     1/* global ajaxurl:false */
     2/* global pwsL10n:false */
     3/* global svgPainter:false */
    24
     5(function( $ ) {
     6
    37        function check_pass_strength() {
    4                 var pass1 = $('#pass1').val(), user = $('#user_login').val(), pass2 = $('#pass2').val(), strength;
     8                var pass1 = $( '#pass1' ).val(), pass2 = $( '#pass2' ).val(), strength;
    59
    6                 $('#pass-strength-result').removeClass('short bad good strong');
     10                $( '#pass-strength-result' ).removeClass( 'short bad good strong' );
    711                if ( ! pass1 ) {
    8                         $('#pass-strength-result').html( pwsL10n.empty );
     12                        $( '#pass-strength-result' ).html( pwsL10n.empty );
    913                        return;
    1014                }
    1115
    … …  
    1317
    1418                switch ( strength ) {
    1519                        case 2:
    16                                 $('#pass-strength-result').addClass('bad').html( pwsL10n['bad'] );
    17                                 break;
     20                                $( '#pass-strength-result' ).addClass( 'bad' ).html( pwsL10n.bad );
     21                        break;
    1822                        case 3:
    19                                 $('#pass-strength-result').addClass('good').html( pwsL10n['good'] );
    20                                 break;
     23                                $( '#pass-strength-result' ).addClass( 'good' ).html( pwsL10n.good );
     24                        break;
    2125                        case 4:
    22                                 $('#pass-strength-result').addClass('strong').html( pwsL10n['strong'] );
    23                                 break;
     26                                $( '#pass-strength-result' ).addClass( 'strong' ).html( pwsL10n.strong );
     27                        break;
    2428                        case 5:
    25                                 $('#pass-strength-result').addClass('short').html( pwsL10n['mismatch'] );
    26                                 break;
     29                                $( '#pass-strength-result' ).addClass( 'short' ).html( pwsL10n.mismatch );
     30                        break;
    2731                        default:
    28                                 $('#pass-strength-result').addClass('short').html( pwsL10n['short'] );
     32                                $( '#pass-strength-result' ).addClass( 'short' ).html( pwsL10n['short'] );
    2933                }
    3034        }
    3135
    32         $(document).ready( function() {
    33                 var select = $('#display_name');
     36        $( document ).ready( function() {
     37                var select = $( '#display_name'), $colorpicker, $stylesheet, user_id, current_user_id;
    3438
    35                 $('#pass1').val('').keyup( check_pass_strength );
    36                 $('#pass2').val('').keyup( check_pass_strength );
    37                 $('#pass-strength-result').show();
    38                 $('.color-palette').click( function() {
    39                         $(this).siblings('input[name="admin_color"]').prop('checked', true);
     39                $( '#pass1' ).val( '' ).keyup( check_pass_strength );
     40                $( '#pass2' ).val( '' ).keyup( check_pass_strength );
     41                $( '#pass-strength-result' ).show();
     42                $( '.color-palette' ).click( function() {
     43                        $( this ).siblings('input[name="admin_color"]' ).prop('checked', true);
    4044                });
    4145
    4246                if ( select.length ) {
    43                         $('#first_name, #last_name, #nickname').bind( 'blur.user_profile', function() {
     47                        $( '#first_name, #last_name, #nickname' ).bind( 'blur.user_profile', function() {
    4448                                var dub = [],
    4549                                        inputs = {
    46                                                 display_nickname  : $('#nickname').val() || '',
    47                                                 display_username  : $('#user_login').val() || '',
    48                                                 display_firstname : $('#first_name').val() || '',
    49                                                 display_lastname  : $('#last_name').val() || ''
     50                                                display_nickname : $( '#nickname' ).val() || '',
     51                                                display_username : $( '#user_login' ).val() || '',
     52                                                display_firstname : $( '#first_name' ).val() || '',
     53                                                display_lastname : $( '#last_name' ).val() || ''
    5054                                        };
    5155
    5256                                if ( inputs.display_firstname && inputs.display_lastname ) {
    53                                         inputs['display_firstlast'] = inputs.display_firstname + ' ' + inputs.display_lastname;
    54                                         inputs['display_lastfirst'] = inputs.display_lastname + ' ' + inputs.display_firstname;
     57                                        inputs.display_firstlast = inputs.display_firstname + ' ' + inputs.display_lastname;
     58                                        inputs.display_lastfirst = inputs.display_lastname + ' ' + inputs.display_firstname;
    5559                                }
    5660
    57                                 $.each( $('option', select), function( i, el ){
     61                                $.each( $( 'option', select ), function( i, el ) {
    5862                                        dub.push( el.value );
    5963                                });
    6064
    61                                 $.each(inputs, function( id, value ) {
    62                                         if ( ! value )
     65                                $.each( inputs, function( id, value ) {
     66                                        if ( ! value ) {
    6367                                                return;
     68                                        }
    6469
    65                                         var val = value.replace(/<\/?[a-z][^>]*>/gi, '');
     70                                        var val = value.replace( /<\/?[a-z][^>]*>/gi, '' );
    6671
    67                                         if ( inputs[id].length && $.inArray( val, dub ) == -1 ) {
    68                                                 dub.push(val);
    69                                                 $('<option />', {
    70                                                         'text': val
     72                                        if ( inputs[ id ].length && $.inArray( val, dub ) === -1 ) {
     73                                                dub.push( val );
     74                                                $( '<option />', {
     75                                                        text: val
    7176                                                }).appendTo( select );
    7277                                        }
    7378                                });
    7479                        });
    7580                }
    7681
    77                 var $colorpicker = $( '#color-picker' ),
    78                         $stylesheet = $( '#colors-css' ),
    79                         user_id = $( 'input#user_id' ).val(),
    80                         current_user_id = $( 'input[name="checkuser_id"]' ).val();
     82                $colorpicker = $( '#color-picker' );
     83                $stylesheet = $( '#colors-css' );
     84                user_id = $( 'input#user_id' ).val();
     85                current_user_id = $( 'input[name="checkuser_id"]' ).val();
    8186
    8287                // dropdown toggle
    8388                $colorpicker.on( 'click', '.dropdown-current', function() {
    … …  
    8994                        var color_scheme = $( this ).children( 'input[name="admin_color"]' ).val();
    9095
    9196                        // update selected
    92                         $( this ).siblings( '.selected' ).removeClass( 'selected' )
     97                        $( this ).siblings( '.selected' ).removeClass( 'selected' );
    9398                        $( this ).addClass( 'selected' );
    9499                        $( this ).find( 'input' ).prop( 'checked', true );
    95100
    … …  
    99104                        $colorpicker.toggleClass( 'picker-expanded' );
    100105
    101106                        // preview/save color scheme
    102                         if ( user_id == current_user_id ) {
     107                        if ( user_id === current_user_id ) {
    103108
    104109                                // repaint icons
    105110                                $stylesheet.attr( 'href', $( this ).children( '.css_url' ).val() );
    … …  
    108113
    109114                                // update user option
    110115                                $.post( ajaxurl, {
    111                                         action: 'save-user-color-scheme',
     116                                        action:       'save-user-color-scheme',
    112117                                        color_scheme: color_scheme,
    113                                         user_id: user_id
     118                                        user_id:      user_id
    114119                                });
    115120
    116121                        }