Merge lp:~bjornt/launchpad/fix-jslint-warnings into lp:launchpad

Proposed by Björn Tillenius
Status: Merged
Merged at revision: not available
Proposed branch: lp:~bjornt/launchpad/fix-jslint-warnings
Merge into: lp:launchpad
Diff against target: None lines
To merge this branch: bzr merge lp:~bjornt/launchpad/fix-jslint-warnings
Reviewer Review Type Date Requested Status
Henning Eggers (community) Approve
Review via email: mp+9912@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Björn Tillenius (bjornt) wrote :

Fix jslint warnings in bugtask-index.js.

--
Björn Tillenius | https://launchpad.net/~bjornt

Revision history for this message
Henning Eggers (henninge) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'lib/canonical/launchpad/javascript/bugs/bugtask-index.js'
2--- lib/canonical/launchpad/javascript/bugs/bugtask-index.js 2009-08-07 08:56:22 +0000
3+++ lib/canonical/launchpad/javascript/bugs/bugtask-index.js 2009-08-10 12:12:00 +0000
4@@ -1619,7 +1619,7 @@
5 else {
6 submit_button.set('disabled', false);
7 }
8- };
9+ }
10 /* Hook up hide_or_show_submit_button on both keyup and mouseup to
11 handle both entering text with the keyboard, and pasting with the
12 mouse. */
13@@ -1679,7 +1679,7 @@
14 var attachment_link = Y.get('.menu-link-addcomment');
15 attachment_link.on('click', function(e) {
16 var comment_input = Y.get('[id="field.comment"]');
17- if (comment_input.get('value') != '') {
18+ if (comment_input.get('value') !== '') {
19 var current_url = attachment_link.getAttribute('href');
20 var attachment_url = current_url + '?field.comment=' +
21 encodeURIComponent(comment_input.get('value'));