Changeset 5334

Timestamp:

04/03/2014 09:32:45 PM (11 years ago)

Author:

johnjamesjacoby

Message:

Improve output of grunt jshint task. See #2578.

Location:

trunk/src/templates/default/js

Files:

• editor.js (modified) (2 diffs)
• forum.js (modified) (1 diff)
• reply.js (modified) (5 diffs)
• topic.js (modified) (1 diff)

trunk/src/templates/default/js/editor.js

@@ -9 +9 @@
     /* Tab from topic title */
     jQuery( '#bbp_topic_title' ).bind( 'keydown.editor-focus', function(e) {
-        if ( e.which !== 9 )
+        if ( e.which !== 9 ) {
             return;
+        }
 
         if ( !e.ctrlKey && !e.altKey && !e.shiftKey ) {
@@ -30 +31 @@
     /* Shift + tab from topic tags */
     jQuery( '#bbp_topic_tags' ).bind( 'keydown.editor-focus', function(e) {
-        if ( e.which !== 9 )
+        if ( e.which !== 9 ) {
             return;
+        }
 
         if ( e.shiftKey && !e.ctrlKey && !e.altKey ) {

trunk/src/templates/default/js/forum.js

@@ -15 +15 @@
                     response.content = bbpForumJS.generic_ajax_error;
                 }
-                alert( response.content );
+                window.alert( response.content );
             }
         } );

trunk/src/templates/default/js/reply.js

@@ -3 +3 @@
         var t = this, div, reply = t.I(replyId), respond = t.I(respondId), cancel = t.I('bbp-cancel-reply-to-link'), parent = t.I('bbp_reply_to'), post = t.I('bbp_topic_id');
 
-        if ( ! reply || ! respond || ! cancel || ! parent )
+        if ( ! reply || ! respond || ! cancel || ! parent ) {
             return;
+        }
 
         t.respondId = respondId;
@@ -17 +18 @@
 
         reply.parentNode.insertBefore(respond);
-        if ( post && postId )
+        if ( post && postId ) {
             post.value = postId;
+        }
         parent.value = parentId;
         cancel.style.display = '';
@@ -25 +27 @@
             var t = addReply, temp = t.I('bbp-temp-form-div'), respond = t.I(t.respondId);
 
-            if ( ! temp || ! respond )
+            if ( ! temp || ! respond ) {
                 return;
+            }
 
             t.I('bbp_reply_to').value = '0';
@@ -34 +37 @@
             this.onclick = null;
             return false;
-        }
+        };
 
         try { t.I('bbp_reply_content').focus(); }
@@ -45 +48 @@
         return document.getElementById(e);
     }
-}
+};

trunk/src/templates/default/js/topic.js

@@ -15 +15 @@
                     response.content = bbpTopicJS.generic_ajax_error;
                 }
-                alert( response.content );
+                window.alert( response.content );
             }
         } );