Changeset 26195

Timestamp:

11/15/2013 04:41:51 AM (13 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++ );