Changeset 26210

Timestamp:

11/15/2013 02:15:39 PM (10 years ago)

Author:

nacin

Message:

Fix JSHint errors in theme-install.js.

props kovshenin.
fixes #26045.

File:

• trunk/src/wp-admin/js/theme-install.js (modified) (14 diffs)

trunk/src/wp-admin/js/theme-install.js

@@ +1 @@
+/* global ajaxurl, list_args, theme_list_args */
+
+var theme_viewer;
+
 /**
  * Theme Browsing
@@ -29 +33 @@
         var tbWindow = $('#TB_window'), width = $(window).width(), H = $(window).height(), W = ( 1040 < width ) ? 1040 : width, adminbar_height = 0;
 
-        if ( $('body.admin-bar').length )
+        if ( $('body.admin-bar').length ) {
             adminbar_height = 28;
+        }
 
         if ( tbWindow.size() ) {
@@ -36 +41 @@
             $('#TB_iframeContent').width( W - 50 ).height( H - 75 - adminbar_height );
             tbWindow.css({'margin-left': '-' + parseInt( ( ( W - 50 ) / 2 ), 10 ) + 'px'});
-            if ( typeof document.body.style.maxWidth != 'undefined' )
+            if ( typeof document.body.style.maxWidth !== 'undefined' ) {
                 tbWindow.css({'top': 20 + adminbar_height + 'px','margin-top':'0'});
-        };
+            }
+        }
     };
 
@@ -50 +56 @@
  */
 jQuery( function($) {
-    if( ! window.postMessage )
+    if ( ! window.postMessage ) {
         return;
+    }
 
     var preview = $('#theme-installer'),
@@ -87 +94 @@
 
 (function($){
-    ThemeViewer = function( args ) {
+    ThemeViewer = function() {
 
         function init() {
@@ -101 +108 @@
                     text  = $( '#filter-click' ).text();
 
-                if ( text.indexOf( '(' ) != -1 )
+                if ( text.indexOf( '(' ) !== -1 ) {
                     text = text.substr( 0, text.indexOf( '(' ) );
-
-                if ( count == 0 )
+                }
+
+                if ( count === 0 ) {
                     $( '#filter-click' ).text( text );
-                else
+                } else {
                     $( '#filter-click' ).text( text + ' (' + count + ')' );
+                }
             });
 
@@ -131 +140 @@
         };
 
-    return api;
-    }
+        return api;
+    };
 })(jQuery);
 
-jQuery( document ).ready( function($) {
+jQuery( document ).ready( function() {
     theme_viewer = new ThemeViewer();
     theme_viewer.init();
@@ -170 +179 @@
             // Get out early if we don't have the required arguments.
             if ( typeof ajaxurl === 'undefined' ||
-                 typeof list_args === 'undefined' ||
-                 typeof theme_list_args === 'undefined' ) {
-                    $('.pagination-links').show();
+                typeof list_args === 'undefined' ||
+                typeof theme_list_args === 'undefined' ) {
+
+                $('.pagination-links').show();
                     return;
             }
@@ -190 +200 @@
              * when user hits the bottom of the current page
              */
-            if ( theme_list_args.total_pages >= this.nextPage )
-                this.pollInterval =
-                    setInterval( function() {
-                        return self.poll();
-                    }, this.scrollPollingDelay );
+            if ( theme_list_args.total_pages >= this.nextPage ) {
+                this.pollInterval = setInterval( function() {
+                    return self.poll();
+                }, this.scrollPollingDelay );
+            }
         },
 
@@ -208 +218 @@
 
             if ( this.querying ||
-                ( bottom < this.$outList.height() - this.outListBottomThreshold ) )
+                ( bottom < this.$outList.height() - this.outListBottomThreshold ) ) {
                 return;
+            }
 
             this.ajax();
@@ -228 +239 @@
             }
 
-            if ( this.nextPage > theme_list_args.total_pages )
+            if ( this.nextPage > theme_list_args.total_pages ) {
                 clearInterval( this.pollInterval );
-
-            if ( this.nextPage <= ( theme_list_args.total_pages + 1 ) )
+            }
+
+            if ( this.nextPage <= ( theme_list_args.total_pages + 1 ) ) {
                 this.$outList.append( results.rows );
+            }
         },
 
@@ -242 +255 @@
          */
         ajax: function() {
-            var self = this;
-
-            this.querying = true;
-
-            var query = {
+            var self = this,
+                query = {
                 action: 'fetch-list',
                 paged: this.nextPage,
@@ -256 +266 @@
                 'list_args': list_args
             };
+
+            this.querying = true;
 
             this.$spinner.show();
@@ -271 +283 @@
                 });
         }
-    }
-
-    $(document).ready( function($) {
+    };
+
+    $(document).ready( function() {
         ThemeScroller.init();
     });