Fixes JS semicolon to following code standard [#1399496]

(I thought those are wrongly reported. It isn't buggies but code improves.)

Comment	File	Size	Author
	js_semicolon.patch	1.54 KB	droplet

Comments

patrickd CreditAttribution: patrickd commented 28 January 2012 at 16:16

Status:

Needs review

» Reviewed & tested by the community

Looks good, could be applied successfully, I think there is not much to discuss about.

sun

German

Karlsruhe

CreditAttribution: sun commented 29 January 2012 at 00:02

Issue tags:

+++ b/core/modules/menu/menu.admin.js
@@ -42,6 +42,6 @@
-  }
+  };
 
 })(jQuery);

This entire file shouldn't be indented in the first place. Let's open a follow-up issue for that.

+++ b/core/modules/shortcut/shortcut.admin.js
@@ -79,7 +79,7 @@ Drupal.behaviors.shortcutDrag = {
-      };
+      }

This shouldn't be a local function in the first place. Let's open a follow-up issue for that.

droplet CreditAttribution: droplet commented 29 January 2012 at 00:32

quick patches,

(im going out, leave me message. I will check when back. Thanks all reviews.)

Dries CreditAttribution: Dries commented 29 January 2012 at 13:25

Status:

Reviewed & tested by the community

» Fixed

Committed to 8.x and backported to 7.x. Thanks.

12 February 2012 at 13:30

Automatically closed -- issue fixed for 2 weeks with no activity.