Changeset 26266

Timestamp:

11/19/2013 04:39:05 AM (11 years ago)

Author:

nacin

Message:

Fix JShint errors in user-profile.js.

props dougwollison.
fixes #26016.

File:

• trunk/src/wp-admin/js/user-profile.js (modified) (8 diffs)

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

@@ +1 @@
+/* global ajaxurl, pwsL10n, svgPainter */
 (function($){
 
     function check_pass_strength() {
-        var pass1 = $('#pass1').val(), user = $('#user_login').val(), pass2 = $('#pass2').val(), strength;
+        var pass1 = $('#pass1').val(), pass2 = $('#pass2').val(), strength;
 
         $('#pass-strength-result').removeClass('short bad good strong');
@@ -14 +15 @@
         switch ( strength ) {
             case 2:
-                $('#pass-strength-result').addClass('bad').html( pwsL10n['bad'] );
+                $('#pass-strength-result').addClass('bad').html( pwsL10n.bad );
                 break;
             case 3:
-                $('#pass-strength-result').addClass('good').html( pwsL10n['good'] );
+                $('#pass-strength-result').addClass('good').html( pwsL10n.good );
                 break;
             case 4:
-                $('#pass-strength-result').addClass('strong').html( pwsL10n['strong'] );
+                $('#pass-strength-result').addClass('strong').html( pwsL10n.strong );
                 break;
             case 5:
-                $('#pass-strength-result').addClass('short').html( pwsL10n['mismatch'] );
+                $('#pass-strength-result').addClass('short').html( pwsL10n.mismatch );
                 break;
             default:
@@ -31 +32 @@
 
     $(document).ready( function() {
-        var select = $('#display_name');
+        var $colorpicker, $stylesheet, user_id, current_user_id,
+            select = $( '#display_name' );
 
         $('#pass1').val('').keyup( check_pass_strength );
@@ -51 +53 @@
 
                 if ( inputs.display_firstname && inputs.display_lastname ) {
-                    inputs['display_firstlast'] = inputs.display_firstname + ' ' + inputs.display_lastname;
-                    inputs['display_lastfirst'] = inputs.display_lastname + ' ' + inputs.display_firstname;
+                    inputs.display_firstlast = inputs.display_firstname + ' ' + inputs.display_lastname;
+                    inputs.display_lastfirst = inputs.display_lastname + ' ' + inputs.display_firstname;
                 }
 
@@ -60 +62 @@
 
                 $.each(inputs, function( id, value ) {
-                    if ( ! value )
+                    if ( ! value ) {
                         return;
+                    }
 
                     var val = value.replace(/<\/?[a-z][^>]*>/gi, '');
 
-                    if ( inputs[id].length && $.inArray( val, dub ) == -1 ) {
+                    if ( inputs[id].length && $.inArray( val, dub ) === -1 ) {
                         dub.push(val);
                         $('<option />', {
@@ -75 +78 @@
         }
 
-        var $colorpicker = $( '#color-picker' ),
-            $stylesheet = $( '#colors-css' ),
-            user_id = $( 'input#user_id' ).val(),
-            current_user_id = $( 'input[name="checkuser_id"]' ).val();
+        $colorpicker = $( '#color-picker' );
+        $stylesheet = $( '#colors-css' );
+        user_id = $( 'input#user_id' ).val();
+        current_user_id = $( 'input[name="checkuser_id"]' ).val();
 
         // dropdown toggle
@@ -90 +93 @@
 
             // update selected
-            $( this ).siblings( '.selected' ).removeClass( 'selected' )
+            $( this ).siblings( '.selected' ).removeClass( 'selected' );
             $( this ).addClass( 'selected' );
             $( this ).find( 'input' ).prop( 'checked', true );
@@ -100 +103 @@
 
             // preview/save color scheme
-            if ( user_id == current_user_id ) {
+            if ( user_id === current_user_id ) {
 
                 // repaint icons