Changeset 26195

Timestamp:

11/15/2013 04:41:51 AM (11 years ago)

Author:

nacin

Message:

Fix JSHint errors in six files.

props mdbitz.
fixes ##26011, #26012, #26013, #26014, #26038, #26039.

Location:

trunk/src/wp-includes/js

Files:

• comment-reply.js (modified) (3 diffs)
• customize-base.js (modified) (3 diffs)
• tinymce/mark_loaded_src.js (modified) (2 diffs)
• wp-auth-check.js (modified) (4 diffs)
• wp-lists.js (modified) (1 diff)
• wp-pointer.js (modified) (7 diffs)

trunk/src/wp-includes/js/comment-reply.js

@@ -1 @@
-
-addComment = {
+var addComment = {
     moveForm : function(commId, parentId, respondId, postId) {
         var t = this, div, comm = t.I(commId), respond = t.I(respondId), cancel = t.I('cancel-comment-reply-link'), parent = t.I('comment_parent'), post = t.I('comment_post_ID');
@@ -35 +34 @@
             this.onclick = null;
             return false;
-        }
+        };
 
         try { t.I('comment').focus(); }
@@ -46 +45 @@
         return document.getElementById(e);
     }
-}
+};

trunk/src/wp-includes/js/customize-base.js

@@ -130 +130 @@
         },
 
-        bind: function( id, callback ) {
+        bind: function( id ) {
             this.topics = this.topics || {};
             this.topics[ id ] = this.topics[ id ] || $.Callbacks();
@@ -137 +137 @@
         },
 
-        unbind: function( id, callback ) {
+        unbind: function( id ) {
             if ( this.topics && this.topics[ id ] )
                 this.topics[ id ].remove.apply( this.topics[ id ], slice.call( arguments, 1 ) );
@@ -210 +210 @@
         },
 
-        bind: function( callback ) {
+        bind: function() {
             this.callbacks.add.apply( this.callbacks, arguments );
             return this;
         },
 
-        unbind: function( callback ) {
+        unbind: function() {
             this.callbacks.remove.apply( this.callbacks, arguments );
             return this;

trunk/src/wp-includes/js/tinymce/mark_loaded_src.js

@@ +1 @@
+/* global tinyMCEPreInit, tinymce */
 (function(){
     if ( typeof tinyMCEPreInit === 'undefined' )
@@ -15 +16 @@
             markDone( baseurl+'/plugins/'+plugin+'/editor_plugin'+suffix+'.js' );
             markDone( baseurl+'/plugins/'+plugin+'/langs/'+lang+'.js' );
-            markDone( baseurl+'/plugins/'+plugin+'/langs/'+lang+'_dlg.js' )
+            markDone( baseurl+'/plugins/'+plugin+'/langs/'+lang+'_dlg.js' );
         }
     });

trunk/src/wp-includes/js/wp-auth-check.js

@@ +1 @@
+/* global adminpage */
 // Interim login dialog
 (function($){
-    var wrap, check, next;
+    var wrap, next;
 
     function show() {
@@ -13 +14 @@
 
             frame = $('<iframe id="wp-auth-check-frame" frameborder="0">').attr( 'title', noframe.text() );
-            frame.load( function(e) {
+            frame.load( function() {
                 var height, body;
 
@@ -68 +69 @@
 
         // When on the Edit Post screen, speed up heartbeat after the user logs in to quickly refresh nonces
-        if ( typeof adminpage != 'undefined' && ( adminpage == 'post-php' || adminpage == 'post-new-php' )
-             && typeof wp != 'undefined' && wp.heartbeat ) {
+        if ( typeof adminpage != 'undefined' && ( adminpage == 'post-php' || adminpage == 'post-new-php' ) &&
+            typeof wp != 'undefined' && wp.heartbeat ) {
 
             wp.heartbeat.interval( 'fast', 1 );
@@ -99 +100 @@
         schedule();
         wrap = $('#wp-auth-check-wrap');
-        wrap.find('.wp-auth-check-close').on( 'click', function(e) {
+        wrap.find('.wp-auth-check-close').on( 'click', function() {
             hide();
         });

trunk/src/wp-includes/js/wp-lists.js

@@ +1 @@
+/* global ajaxurl, wpAjax */
 (function($) {
 var fs = {add:'ajaxAdd',del:'ajaxDel',dim:'ajaxDim',process:'process',recolor:'recolor'}, wpList;

trunk/src/wp-includes/js/wp-pointer.js

@@ +1 @@
+/* global wpPointerL10n */
 /**
  * Pointer jQuery widget.
@@ -6 +7 @@
         zindex = 9999;
 
-    $.widget("wp.pointer", {
+    $.widget('wp.pointer', {
         options: {
             pointerClass: 'wp-pointer',
             pointerWidth: 320,
-            content: function( respond, event, t ) {
+            content: function() {
                 return $(this).text();
             },
@@ -61 +62 @@
 
             // Handle document transfer
-            if ( key === "document" && value !== o.document ) {
+            if ( key === 'document' && value !== o.document ) {
                 tip.detach().appendTo( value.body );
 
             // Handle class change
-            } else if ( key === "pointerClass" ) {
+            } else if ( key === 'pointerClass' ) {
                 tip.removeClass( o.pointerClass ).addClass( value );
             }
@@ -73 +74 @@
 
             // Reposition automatically
-            if ( key === "position" ) {
+            if ( key === 'position' ) {
                 this.reposition();
 
             // Update content automatically if pointer is open
-            } else if ( key === "content" && this.active ) {
+            } else if ( key === 'content' && this.active ) {
                 this.update();
             }
@@ -102 +103 @@
             dfd.done( function( content ) {
                 self._update( event, content );
-            })
+            });
 
             // Either o.content is a string...
@@ -236 +237 @@
             this.active = true;
 
-            this._trigger( "open", event, this._handoff() );
-
-            this._trigger( "show", event, this._handoff({
+            this._trigger( 'open', event, this._handoff() );
+
+            this._trigger( 'show', event, this._handoff({
                 opened: function() {
-                    self._trigger( "opened", event, self._handoff() );
+                    self._trigger( 'opened', event, self._handoff() );
                 }
             }));
@@ -252 +253 @@
             this.active = false;
 
-            this._trigger( "close", event, this._handoff() );
-            this._trigger( "hide", event, this._handoff({
+            this._trigger( 'close', event, this._handoff() );
+            this._trigger( 'hide', event, this._handoff({
                 closed: function() {
-                    self._trigger( "closed", event, self._handoff() );
+                    self._trigger( 'closed', event, self._handoff() );
                 }
             }));
         },
 
-        sendToTop: function( event ) {
+        sendToTop: function() {
             if ( this.active )
                 this.pointer.css( 'z-index', zindex++ );