Changeset 26264

Timestamp:

11/19/2013 04:31:42 AM (13 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.