style(jshint): some style fixes and jshint compatibility improved

bin/lamassu-server

@@ -1,8 +1,10 @@
 #!/usr/bin/env node
+
+'use strict';
+
 var fs = require('fs');
 var createServer = require('../lib/app.js');
 var argv = require('minimist')(process.argv.slice(2));
-var logger = require('../lib/logger');
 
 var options = {
   postgres: process.env.DATABASE_URL

lib/logger.js

@@ -1,3 +1,5 @@
+'use strict';
+
 var bunyan  = require('bunyan');
 var async   = require('async');
 
@@ -10,25 +12,26 @@ var bunyan = bunyan.createLogger({name: 'lamassu-server', level: logLevel});
 
 // log version
 var version = require('../package.json').version;
-bunyan.info('Version:', version);
+bunyan.info('Version: %s', version);
 
 
-// log twitter stuff (optional)
+// log git stuff (optional)
-function wrapper(fn) {
+// `git-rev` omits `err` param in callback, without this wrapper
-  return function(cb) {
+// `async` interprets returned values as errors.
+function wrapper(fn, cb) {
   fn(function(value) {
     cb(null, value);
   });
-  }
 }
 try {
   var git = require('git-rev');
+
   async.parallel([
-    wrapper(git.branch),
+    async.apply(wrapper, git.branch),
-    wrapper(git.short)
+    async.apply(wrapper, git.short)
   ],
   function(err, values) {
-    bunyan.info('Git:', '#' + values[0], '@' + values[1]);
+    bunyan.info('Git: #%s @%s', values[0], values[1]);
   });
 } catch(_) {}
 

lib/plugins.js

@@ -17,12 +17,7 @@ var traderPlugin = null;
 var walletPlugin = null;
 var idVerifierPlugin = null;
 
-var currentlyUsedPlugins = {
+var currentlyUsedPlugins = {};
-  ticker: null,
-  transfer: null,
-  trader: null,
-  idVerifier: null
-};
 
 
 var cachedConfig = null;
@@ -135,7 +130,7 @@ exports.configure = function configure(config) {
     tickerPlugin,
     'ticker',
     deviceCurrency, // device currency
-    function onTickerChange(newTicker, currency) {
+    function onTickerChange(newTicker) {
       tickerPlugin = newTicker;
       pollRate();
     }
@@ -335,7 +330,7 @@ function pollBalance(callback) {
 }
 
 function pollRate(callback) {
-  logger.debug('polling for rates (' + tickerPlugin.NAME + ')');
+  logger.debug('polling for rates (%s)', tickerPlugin.NAME);
 
   tickerPlugin.ticker(deviceCurrency, function(err, resRates) {
     if (err) {

lib/routes.js

@@ -6,7 +6,6 @@ var mock = false;
 
 var plugins;
 var lamassuConfig;
-var config;
 
 module.exports = {
   init: init,
@@ -58,7 +57,7 @@ function poll(req, res) {
   };
 
   if (response.idVerificationEnabled)
-    response.idVerificationLimit = complianceSettings.idVerificationLimit
+    response.idVerificationLimit = complianceSettings.idVerificationLimit;
 
   res.json(response);
 }