Changeset 26265

Timestamp:

11/19/2013 04:34:29 AM (13 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];
         }