2012-10-04 19:48:24 +02:00
|
|
|
"use strict";
|
|
|
|
|
2012-10-04 07:45:44 +02:00
|
|
|
// Global variables, categorized by place of definition.
|
|
|
|
var globals =
|
|
|
|
// Third-party libraries
|
2012-10-05 20:44:29 +02:00
|
|
|
' $ jQuery Spinner Handlebars XDate'
|
2012-10-04 07:45:44 +02:00
|
|
|
|
|
|
|
// index.html
|
2012-10-10 23:26:01 +02:00
|
|
|
+ ' initial_pointer email stream_list people_list have_initial_messages'
|
2012-10-04 07:45:44 +02:00
|
|
|
|
2012-10-18 20:29:16 +02:00
|
|
|
// common.js
|
|
|
|
+ ' status_classes'
|
|
|
|
|
2012-10-04 07:45:44 +02:00
|
|
|
// compose.js
|
2012-10-18 20:29:16 +02:00
|
|
|
+ ' compose'
|
2012-10-04 07:45:44 +02:00
|
|
|
|
2012-10-18 20:46:06 +02:00
|
|
|
// rows.js
|
2012-10-18 20:55:41 +02:00
|
|
|
+ ' rows'
|
2012-10-04 07:45:44 +02:00
|
|
|
|
|
|
|
// hotkey.js
|
2012-10-18 19:58:10 +02:00
|
|
|
+ ' hotkeys'
|
2012-10-04 07:45:44 +02:00
|
|
|
|
|
|
|
// narrow.js
|
2012-10-18 20:12:04 +02:00
|
|
|
+ ' narrow'
|
2012-10-04 07:45:44 +02:00
|
|
|
|
|
|
|
// setup.js
|
|
|
|
+ ' loading_spinner templates'
|
|
|
|
|
2012-10-18 21:42:08 +02:00
|
|
|
// subs.js
|
2012-10-18 21:37:07 +02:00
|
|
|
+ ' subs'
|
2012-10-04 07:45:44 +02:00
|
|
|
|
|
|
|
// ui.js
|
2012-10-09 16:59:49 +02:00
|
|
|
+ ' register_onclick hide_email show_email'
|
2012-10-17 20:43:20 +02:00
|
|
|
+ ' report_error report_success report_message clicking mouse_moved'
|
2012-10-04 07:45:44 +02:00
|
|
|
+ ' update_autocomplete autocomplete_needs_update'
|
|
|
|
|
|
|
|
// zephyr.js
|
2012-10-15 19:24:34 +02:00
|
|
|
+ ' message_array message_dict selected_message_class'
|
2012-10-18 20:29:16 +02:00
|
|
|
+ ' clear_table add_to_table subject_dict'
|
2012-10-10 20:20:31 +02:00
|
|
|
+ ' keep_pointer_in_view move_pointer_at_page_top_and_bottom'
|
|
|
|
+ ' respond_to_message'
|
2012-10-09 23:51:45 +02:00
|
|
|
+ ' select_message select_message_by_id'
|
2012-10-22 19:05:06 +02:00
|
|
|
+ ' scroll_to_selected'
|
2012-10-10 16:17:58 +02:00
|
|
|
+ ' selected_message selected_message_id'
|
2012-10-19 20:04:20 +02:00
|
|
|
+ ' start_reload_app reloading_app do_reload_app_preserving_compose'
|
2012-10-05 22:29:59 +02:00
|
|
|
+ ' at_top_of_viewport at_bottom_of_viewport'
|
2012-10-16 04:07:52 +02:00
|
|
|
+ ' viewport'
|
2012-10-04 07:45:44 +02:00
|
|
|
;
|
|
|
|
|
|
|
|
|
|
|
|
var jslint_options = {
|
|
|
|
browser: true, // Assume browser environment
|
|
|
|
vars: true, // Allow multiple 'var' per function
|
|
|
|
sloppy: true, // Don't require "use strict"
|
|
|
|
white: true, // Lenient whitespace rules
|
|
|
|
plusplus: true, // Allow increment/decrement operators
|
|
|
|
regexp: true, // Allow . and [^...] in regular expressions
|
|
|
|
todo: true, // Allow "TODO" comments.
|
|
|
|
|
|
|
|
predef: globals.split(/\s+/)
|
|
|
|
};
|
|
|
|
|
|
|
|
|
|
|
|
// For each error.raw message, we can return 'true' to ignore
|
|
|
|
// the error.
|
|
|
|
var exceptions = {
|
|
|
|
"Expected '{a}' and instead saw '{b}'." : function (error) {
|
|
|
|
// We allow single-statement 'if' with no brace.
|
|
|
|
// This exception might be overly broad but oh well.
|
|
|
|
return (error.a === '{');
|
|
|
|
},
|
|
|
|
|
|
|
|
"Unexpected 'else' after 'return'." : function () {
|
|
|
|
return true;
|
|
|
|
}
|
|
|
|
};
|
|
|
|
|
|
|
|
|
|
|
|
var fs = require('fs');
|
2012-10-04 19:15:37 +02:00
|
|
|
var path = require('path');
|
|
|
|
var JSLINT = require(path.join(__dirname, 'jslint')).JSLINT;
|
2012-10-04 07:45:44 +02:00
|
|
|
|
2012-10-04 19:40:29 +02:00
|
|
|
var cwd = process.cwd();
|
2012-10-04 19:15:37 +02:00
|
|
|
var js_dir = fs.realpathSync(path.join(__dirname, '../../zephyr/static/js'));
|
2012-10-04 07:45:44 +02:00
|
|
|
|
|
|
|
var exit_code = 0;
|
|
|
|
|
|
|
|
fs.readdirSync(js_dir).forEach(function (filename) {
|
|
|
|
if (filename.slice('-3') !== '.js')
|
|
|
|
return;
|
|
|
|
|
2012-10-04 19:40:29 +02:00
|
|
|
var filepath = path.join(js_dir, filename);
|
|
|
|
var contents = fs.readFileSync(filepath, 'utf8');
|
2012-10-04 07:45:44 +02:00
|
|
|
var messages = [];
|
|
|
|
|
|
|
|
if (!JSLINT(contents, jslint_options)) {
|
|
|
|
JSLINT.errors.forEach(function (error) {
|
|
|
|
if (error === null) {
|
|
|
|
// JSLint stopping error
|
|
|
|
messages.push(' (JSLint giving up)');
|
|
|
|
return;
|
|
|
|
}
|
|
|
|
|
|
|
|
var exn = exceptions[error.raw];
|
|
|
|
if (exn && exn(error)) {
|
|
|
|
// Ignore this error.
|
|
|
|
return;
|
|
|
|
}
|
|
|
|
|
|
|
|
// NB: this will break on a 10,000 line file
|
|
|
|
var line = (' ' + error.line).slice(-4);
|
|
|
|
|
|
|
|
messages.push(' ' + line + ' ' + error.reason);
|
|
|
|
});
|
|
|
|
|
|
|
|
if (messages.length > 0) {
|
|
|
|
exit_code = 1;
|
|
|
|
|
2012-10-04 19:40:29 +02:00
|
|
|
console.log(path.relative(cwd, filepath));
|
2012-10-04 07:45:44 +02:00
|
|
|
|
|
|
|
// Something very wacky happens if we do
|
|
|
|
// .forEach(console.log) directly.
|
|
|
|
messages.forEach(function (msg) {
|
|
|
|
console.log(msg);
|
|
|
|
});
|
|
|
|
|
|
|
|
console.log('');
|
|
|
|
}
|
|
|
|
}
|
|
|
|
});
|
|
|
|
|
|
|
|
process.exit(exit_code);
|