Changeset 26264

Timestamp:

11/19/2013 04:31:42 AM (12 years ago)

Author:

nacin

Message:

JShint fixes for inline-edit-post.js.

props dougwollison.
fixes #26001.

File:

• trunk/src/wp-admin/js/inline-edit-post.js (modified) (21 diffs)

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

@@ +1 @@
+/* global inlineEditL10n, ajaxurl, typenow */
+
+var inlineEditPost;
 (function($) {
 inlineEditPost = {
@@ -10 +13 @@
         // prepare the edit rows
         qeRow.keyup(function(e){
-            if (e.which == 27)
+            if ( e.which === 27 ) {
                 return inlineEditPost.revert();
+            }
         });
         bulkRow.keyup(function(e){
-            if (e.which == 27)
+            if ( e.which === 27 ) {
                 return inlineEditPost.revert();
+            }
         });
 
@@ -25 +30 @@
         });
         $('td', qeRow).keydown(function(e){
-            if ( e.which == 13 )
+            if ( e.which === 13 ) {
                 return inlineEditPost.save(this);
+            }
         });
 
@@ -58 +64 @@
         $('#doaction, #doaction2').click(function(e){
             var n = $(this).attr('id').substr(2);
-            if ( $('select[name="'+n+'"]').val() == 'edit' ) {
+            if ( 'edit' === $( 'select[name="' + n + '"]' ).val() ) {
                 e.preventDefault();
                 t.setBulk();
@@ -69 +75 @@
     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 );
     },
 
@@ -80 +86 @@
         $('#bulk-edit').addClass('inline-editor').show();
 
-        $('tbody th.check-column input[type="checkbox"]').each(function(i){
+        $( 'tbody th.check-column input[type="checkbox"]' ).each( function() {
             if ( $(this).prop('checked') ) {
                 c = false;
@@ -89 +95 @@
         });
 
-        if ( c )
+        if ( c ) {
             return this.revert();
+        }
 
         $('#bulk-titles').html(te);
@@ -101 +108 @@
 
         // enable autocomplete for tags
-        if ( 'post' == type ) {
+        if ( 'post' === type ) {
             // support multi taxonomies?
             tax = 'post_tag';
@@ -113 +120 @@
         t.revert();
 
-        if ( typeof(id) == 'object' )
+        if ( typeof(id) === 'object' ) {
             id = t.getId(id);
+        }
 
         fields = ['post_title', 'post_name', 'post_author', '_status', 'jj', 'mm', 'aa', 'hh', 'mn', 'ss', 'post_password', 'post_format', 'menu_order'];
-        if ( t.type == 'page' )
+        if ( t.type === 'page' ) {
             fields.push('post_parent', 'page_template');
+        }
 
         // add the new blank row
@@ -124 +133 @@
         $('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);
 
@@ -134 +144 @@
             $(':input[name="post_author"]', editRow).prepend('<option value="' + $('.post_author', rowData).text() + '">' + $('#' + t.type + '-' + id + ' .author').text() + '</option>');
         }
-        if ( $(':input[name="post_author"] option', editRow).length == 1 ) {
+        if ( $( ':input[name="post_author"] option', editRow ).length === 1 ) {
             $('label.inline-edit-author', editRow).hide();
         }
@@ -142 +152 @@
         $('option.unsupported', editRow).each(function() {
             var $this = $(this);
-            if ( $this.val() != cur_format )
+            if ( $this.val() !== cur_format ) {
                 $this.remove();
+            }
         });
 
@@ -150 +161 @@
         }
 
-        if ( $('.comment_status', rowData).text() == 'open' )
-            $('input[name="comment_status"]', editRow).prop("checked", true);
-        if ( $('.ping_status', rowData).text() == 'open' )
-            $('input[name="ping_status"]', editRow).prop("checked", true);
-        if ( $('.sticky', rowData).text() == 'sticky' )
-            $('input[name="sticky"]', editRow).prop("checked", true);
+        if ( $( '.comment_status', rowData ).text() === 'open' ) {
+            $( 'input[name="comment_status"]', editRow ).prop( 'checked', true );
+        }
+        if ( $( '.ping_status', rowData ).text() === 'open' ) {
+            $( 'input[name="ping_status"]', editRow ).prop( 'checked', true );
+        }
+        if ( $( '.sticky', rowData ).text() === 'sticky' ) {
+            $( 'input[name="sticky"]', editRow ).prop( 'checked', true );
+        }
 
         // hierarchical taxonomies
         $('.post_category', rowData).each(function(){
-            var term_ids = $(this).text();
+            var taxname,
+                term_ids = $(this).text();
 
             if ( term_ids ) {
@@ -175 +190 @@
 
             if ( terms ) {
-                if ( ',' !== comma )
+                if ( ',' !== comma ) {
                     terms = terms.replace(/,/g, comma);
+                }
                 textarea.val(terms);
             }
@@ -185 +201 @@
         // handle the post status
         status = $('._status', rowData).text();
-        if ( 'future' != status )
+        if ( 'future' !== status ) {
             $('select[name="_status"] option[value="future"]', editRow).remove();
-
-        if ( 'private' == status ) {
-            $('input[name="keep_private"]', editRow).prop("checked", true);
+        }
+
+        if ( 'private' === status ) {
+            $('input[name="keep_private"]', editRow).prop('checked', true);
             $('input.inline-edit-password-input').val('').prop('disabled', true);
         }
@@ -200 +217 @@
             while ( pageLoop ) {
                 nextPage = nextPage.next('option');
-                if (nextPage.length == 0) break;
+                if ( nextPage.length === 0 ) {
+                    break;
+                }
+
                 nextLevel = nextPage[0].className.split('-')[1];
+
                 if ( nextLevel <= pageLevel ) {
                     pageLoop = false;
@@ -221 +242 @@
         var params, fields, page = $('.post_status_page').val() || '';
 
-        if ( typeof(id) == 'object' )
+        if ( typeof(id) === 'object' ) {
             id = this.getId(id);
+        }
 
         $('table.widefat .spinner').show();
@@ -243 +265 @@
 
                 if (r) {
-                    if ( -1 != r.indexOf('<tr') ) {
+                    if ( -1 !== r.indexOf( '<tr' ) ) {
                         $(inlineEditPost.what+id).remove();
                         $('#edit-'+id).before(r).remove();
@@ -258 +280 @@
                     $('#post-'+id).removeClass('alternate');
                 }
-            }
-        , 'html');
+            },
+        'html');
         return false;
     },
@@ -269 +291 @@
             $('table.widefat .spinner').hide();
 
-            if ( 'bulk-edit' == id ) {
+            if ( 'bulk-edit' === id ) {
                 $('table.widefat #bulk-edit').removeClass('inline-editor').hide();
                 $('#bulk-titles').html('');
@@ -320 +342 @@
 
     $('#the-list tr').each( function(i, el) {
-        if ( el.id )
+        if ( el.id ) {
             check.push( el.id );
+        }
     });
 
-    if ( check.length )
+    if ( check.length ) {
         data['wp-check-locked-posts'] = check;
+    }
 }).ready( function() {
     // Set the heartbeat interval to 15 sec.