summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDan McGee <dan@archlinux.org>2011-09-29 21:50:58 -0500
committerDan McGee <dan@archlinux.org>2011-09-29 21:50:58 -0500
commit447c4c4bd56b6364a68068ccf691d048e6c2d65b (patch)
tree910e74603f44d197390abff7bcab76cc2d940748
parent797185faed0555efb88a1e6a18e447548a9935fd (diff)
JSLint suggested cleanups
Signed-off-by: Dan McGee <dan@archlinux.org>
-rw-r--r--media/archweb.js49
1 files changed, 35 insertions, 14 deletions
diff --git a/media/archweb.js b/media/archweb.js
index 49f2a319..52f3cba2 100644
--- a/media/archweb.js
+++ b/media/archweb.js
@@ -6,7 +6,7 @@ if (typeof $.tablesorter !== 'undefined') {
is: function(s) { return false; },
format: function(s) {
var m = s.match(/\d+/);
- return m ? parseInt(m[0]) : 0;
+ return m ? parseInt(m[0], 10) : 0;
},
type: 'numeric'
});
@@ -27,7 +27,9 @@ if (typeof $.tablesorter !== 'undefined') {
return ($.inArray(s, this.special) > -1) || $.tablesorter.isDigit(s, c);
},
format: function(s) {
- if ($.inArray(s, this.special) > -1) return Number.MAX_VALUE;
+ if ($.inArray(s, this.special) > -1) {
+ return Number.MAX_VALUE;
+ }
return $.tablesorter.formatFloat(s);
},
type: 'numeric'
@@ -41,9 +43,13 @@ if (typeof $.tablesorter !== 'undefined') {
return ($.inArray(s, this.special) > -1) || this.re.test(s);
},
format: function(s) {
- if ($.inArray(s, this.special) > -1) return Number.MAX_VALUE;
+ if ($.inArray(s, this.special) > -1) {
+ return Number.MAX_VALUE;
+ }
var matches = this.re.exec(s);
- if (!matches) return Number.MAX_VALUE;
+ if (!matches) {
+ return Number.MAX_VALUE;
+ }
return matches[1] * 60 + matches[2];
},
type: 'numeric'
@@ -56,9 +62,13 @@ if (typeof $.tablesorter !== 'undefined') {
},
format: function(s) {
var matches = this.re.exec(s);
- if (!matches) return 0;
+ if (!matches) {
+ return 0;
+ }
/* skip group 6, group 7 is optional seconds */
- if (matches[7] == undefined) matches[7] = 0;
+ if (matches[7] === undefined) {
+ matches[7] = 0;
+ }
/* The awesomeness of the JS date constructor. Month needs to be
* between 0-11, because things have to be difficult. */
var date = new Date(matches[1], matches[2] - 1, matches[3],
@@ -75,7 +85,9 @@ if (typeof $.tablesorter !== 'undefined') {
},
format: function(s) {
var matches = this.re.exec(s);
- if (!matches) return 0;
+ if (!matches) {
+ return 0;
+ }
var size = parseFloat(matches[1]);
var suffix = matches[2];
@@ -146,16 +158,24 @@ function filter_packages() {
// all this just to get the split version out of the table cell
var ver_a = cells.eq(2).find('span').text().match(pat);
- if (!ver_a) return true;
+ if (!ver_a) {
+ return true;
+ }
var ver_b = cells.eq(3).find('span').text().match(pat);
- if (!ver_b) return true;
+ if (!ver_b) {
+ return true;
+ }
// first check pkgver
- if (ver_a[1] !== ver_b[1]) return true;
+ if (ver_a[1] !== ver_b[1]) {
+ return true;
+ }
// pkgver matched, so see if rounded pkgrel matches
- if (Math.floor(parseFloat(ver_a[2])) ==
- Math.floor(parseFloat(ver_b[2]))) return false;
+ if (Math.floor(parseFloat(ver_a[2])) ===
+ Math.floor(parseFloat(ver_b[2]))) {
+ return false;
+ }
// pkgrel didn't match, so keep the row
return true;
});
@@ -199,12 +219,13 @@ function signoff_package() {
$(link).append(signoff);
}
/* update the approved column to reflect reality */
+ var approved;
if (data.approved) {
- var approved = $(link).closest('tr').children('.signoff-no');
+ approved = $(link).closest('tr').children('.signoff-no');
approved.text('Yes').addClass(
'signoff-yes').removeClass('signoff-no');
} else {
- var approved = $(link).closest('tr').children('.signoff-yes');
+ approved = $(link).closest('tr').children('.signoff-yes');
approved.text('No').addClass(
'signoff-no').removeClass('signoff-yes');
}