Changeset 26265

Timestamp:

11/19/2013 04:34:29 AM (11 years ago)

Author:

nacin

Message:

JSHint fixes for inline-edit-tax.js.

props dougwollison.
fixes #26002.

File:

• trunk/src/wp-admin/js/inline-edit-tax.js (modified) (7 diffs)

trunk/src/wp-admin/js/inline-edit-tax.js

@@ +1 @@
+/* global inlineEditL10n, ajaxurl */
 
+var inlineEditTax;
 (function($) {
 inlineEditTax = {
@@ -15 +17 @@
 
         // prepare the edit row
-        row.keyup(function(e) { if(e.which == 27) return inlineEditTax.revert(); });
+        row.keyup( function( e ) {
+            if ( e.which === 27 ) {
+                return inlineEditTax.revert();
+            }
+        });
 
-        $('a.cancel', row).click(function() { return inlineEditTax.revert(); });
-        $('a.save', row).click(function() { return inlineEditTax.save(this); });
-        $('input, select', row).keydown(function(e) { if(e.which == 13) return inlineEditTax.save(this); });
+        $( 'a.cancel', row ).click( function() {
+            return inlineEditTax.revert();
+        });
+        $( 'a.save', row ).click( function() {
+            return inlineEditTax.save(this);
+        });
+        $( 'input, select', row ).keydown( function( e ) {
+            if ( e.which === 13 ) {
+                return inlineEditTax.save( this );
+            }
+        });
 
-        $('#posts-filter input[type="submit"]').mousedown(function(e){
+        $( '#posts-filter input[type="submit"]' ).mousedown( function() {
             t.revert();
         });
@@ -28 +42 @@
     toggle : function(el) {
         var t = this;
-        $(t.what+t.getId(el)).css('display') == 'none' ? t.revert() : t.edit(el);
+        $(t.what+t.getId(el)).css('display') === 'none' ? t.revert() : t.edit(el);
     },
 
     edit : function(id) {
-        var t = this, editRow;
+        var editRow, rowData,
+            t = this;
         t.revert();
 
-        if ( typeof(id) == 'object' )
+        if ( typeof(id) === 'object' ) {
             id = t.getId(id);
+        }
 
         editRow = $('#inline-edit').clone(true), rowData = $('#inline_'+id);
         $('td', editRow).attr('colspan', $('.widefat:first thead th:visible').length);
 
-        if ( $(t.what+id).hasClass('alternate') )
+        if ( $( t.what + id ).hasClass( 'alternate' ) ) {
             $(editRow).addClass('alternate');
+        }
 
         $(t.what+id).hide().after(editRow);
@@ -58 +75 @@
         var params, fields, tax = $('input[name="taxonomy"]').val() || '';
 
-        if( typeof(id) == 'object' )
+        if( typeof(id) === 'object' ) {
             id = this.getId(id);
+        }
 
         $('table.widefat .spinner').show();
@@ -80 +98 @@
 
                 if (r) {
-                    if ( -1 != r.indexOf('<tr') ) {
+                    if ( -1 !== r.indexOf( '<tr' ) ) {
                         $(inlineEditTax.what+id).remove();
                         new_id = $(r).attr('id');
@@ -87 +105 @@
                         row = new_id ? $('#'+new_id) : $(inlineEditTax.what+id);
                         row.hide().fadeIn();
-                    } else
+                    } else {
                         $('#edit-'+id+' .inline-edit-save .error').html(r).show();
-                } else
+                    }
+                } else {
                     $('#edit-'+id+' .inline-edit-save .error').html(inlineEditL10n.error).show();
+                }
 
-                if ($(row).prev('tr').hasClass('alternate'))
+                if ( $( row ).prev( 'tr' ).hasClass( 'alternate' ) ) {
                     $(row).removeClass('alternate');
+                }
             }
         );
@@ -113 +134 @@
 
     getId : function(o) {
-        var id = o.tagName == 'TR' ? o.id : $(o).parents('tr').attr('id'), parts = id.split('-');
+        var id = o.tagName === 'TR' ? o.id : $(o).parents('tr').attr('id'), parts = id.split('-');
         return parts[parts.length - 1];
     }