added standard linter

replace jshint task with standard task in gulp dev formatted all files wrt standard linter fixed lint warnings regarding path and split vars fixed lint warnings around error handling fixed other misc lint warnings
dexter0201 · Aug 27, 2016 · 36e50c1 · 36e50c1
1 parent dbb39c5
commit 36e50c1
Show file tree

Hide file tree

Showing 29 changed files with 608 additions and 597 deletions.
diff --git a/app.js b/app.js
@@ -1,50 +1,50 @@
-'use strict';
+'use strict'
 
-import 'angular/angular-csp.css';
-import 'angular-ui-select/select.min.css';
-import 'bootstrap/dist/css/bootstrap.min.css';
+import 'angular/angular-csp.css'
+import 'angular-ui-select/select.min.css'
+import 'bootstrap/dist/css/bootstrap.min.css'
 
-import jQuery from 'jquery';
-import 'angular';
-import 'angular-ui-select/select';
-import 'angular-mocks';
-import 'angular-cookies';
-import 'angular-resource';
-import 'angular-sanitize';
-import 'angular-ui-router';
-import 'angular-jwt';
-import 'angular-bootstrap/ui-bootstrap-tpls';
+import jQuery from 'jquery'
+import 'angular'
+import 'angular-ui-select/select'
+import 'angular-mocks'
+import 'angular-cookies'
+import 'angular-resource'
+import 'angular-sanitize'
+import 'angular-ui-router'
+import 'angular-jwt'
+import 'angular-bootstrap/ui-bootstrap-tpls'
 
-window.$ = jQuery;
+window.$ = jQuery
 
 angular.element(document).ready(function () {
-    //Fixing facebook bug with redirect
-    if (window.location.hash === '#_=_') window.location.hash = '#!';
-
-    //Then init the app
-    angular.bootstrap(document, ['mean']);
-
-});
-
-function processModules(modules) {
-
-    var packageModules = ['ngCookies', 'ngResource', 'ui.bootstrap', 'ui.router', 'ui.select', 'ngSanitize'], m, mn;
-    for (var index in modules) {
-        m = modules[index];
-        mn = 'mean.' + m.name;
-        angular.module(mn, m.angularDependencies || []);
-        packageModules.push(mn);
-    }
-
-    var req = require.context('./packages', true, /\/public\/(?!tests|assets|views)(.*)\.js$/);
-    req.keys().map(req);
-    req = require.context('./node_modules', true, /\/meanio-(.*)\/public\/(?!tests|assets|views)(.*)\.js$/);
-    req.keys().map(req);
-    angular.module('mean', packageModules);
+  // Fixing facebook bug with redirect
+  if (window.location.hash === '#_=_') window.location.hash = '#!'
+
+  // Then init the app
+  angular.bootstrap(document, ['mean'])
+})
+
+function processModules (modules) {
+  var packageModules = ['ngCookies', 'ngResource', 'ui.bootstrap', 'ui.router', 'ui.select', 'ngSanitize']
+  var m
+  var mn
+  for (var index in modules) {
+    m = modules[index]
+    mn = 'mean.' + m.name
+    angular.module(mn, m.angularDependencies || [])
+    packageModules.push(mn)
+  }
+
+  var req = require.context('./packages', true, /\/public\/(?!tests|assets|views)(.*)\.js$/)
+  req.keys().map(req)
+  req = require.context('./node_modules', true, /\/meanio-(.*)\/public\/(?!tests|assets|views)(.*)\.js$/)
+  req.keys().map(req)
+  angular.module('mean', packageModules)
 }
 
 jQuery.ajax('/_getModules', {
-    dataType: 'json',
-    async: false,
-    success: processModules
-});
+  dataType: 'json',
+  async: false,
+  success: processModules
+})
diff --git a/config/env/default.js b/config/env/default.js
@@ -1,7 +1,7 @@
-'use strict';
+'use strict'
 
-var path = require('path'),
-  rootPath = path.normalize(__dirname + '/../..');
+var path = require('path')
+var rootPath = path.join(__dirname, '/../..')
 
 module.exports = {
   root: rootPath,
@@ -45,20 +45,20 @@ module.exports = {
   public: {
     languages: [{
       locale: 'en',
-      direction: 'ltr',
+      direction: 'ltr'
     }, {
       locale: 'he',
-      direction: 'rtl',
+      direction: 'rtl'
     }],
     currentLanguage: 'en',
     loginPage: '/auth/login',
     cssFramework: 'bootstrap'
   },
   clusterSticky: false,
   stickyOptions: {
-    proxy: false, //activate layer 4 patching
-    header: 'x-forwarded-for', //provide here your header containing the users ip
-    num: (process.env.CPU_COUNT || require('os').cpus().length) - 1,
+    proxy: false, // activate layer 4 patching
+    header: 'x-forwarded-for', // provide here your header containing the users ip
+    num: (process.env.CPU_COUNT || require('os').cpus().length) - 1
   },
   // The session cookie name
   sessionName: 'connect.sid',
@@ -67,4 +67,4 @@ module.exports = {
     json: {limit: '100kb'},
     urlencoded: {limit: '100kb', extended: true}
   }
-};
+}
diff --git a/config/env/development.js b/config/env/development.js
@@ -1,4 +1,4 @@
-'use strict';
+'use strict'
 
 module.exports = {
   db: 'mongodb://' + (process.env.DB_PORT_27017_TCP_ADDR || 'localhost') + '/mean-dev',
@@ -58,6 +58,6 @@ module.exports = {
       user: 'EMAIL_ID',
       pass: 'PASSWORD'
     }
-  }, 
+  },
   secret: 'SOME_TOKEN_SECRET'
-};
+}
diff --git a/config/env/production.js b/config/env/production.js
@@ -1,4 +1,4 @@
-'use strict';
+'use strict'
 
 module.exports = {
   db: 'mongodb://' + (process.env.DB_PORT_27017_TCP_ADDR || 'localhost') + '/mean-prod',
@@ -78,4 +78,4 @@ module.exports = {
     }
   },
   secret: 'SOME_TOKEN_SECRET'
-};
+}
diff --git a/config/env/test.js b/config/env/test.js
@@ -1,4 +1,4 @@
-'use strict';
+'use strict'
 
 module.exports = {
   db: 'mongodb://' + (process.env.DB_PORT_27017_TCP_ADDR || 'localhost') + '/mean-test',
@@ -60,4 +60,4 @@ module.exports = {
     }
   },
   secret: 'SOME_TOKEN_SECRET'
-};
+}
diff --git a/config/express.js b/config/express.js
@@ -1,71 +1,68 @@
 /* globals require */
-'use strict';
+'use strict'
 
 /**
  * Module dependencies.
  */
-var mean = require('meanio'),
-  compression = require('compression'),
-  consolidate = require('consolidate'),
-  express = require('express'),
-  helpers = require('view-helpers'),
-  flash = require('connect-flash'),
-  modRewrite = require('connect-modrewrite'),
-  // seo = require('mean-seo'),
-  config = mean.getConfig(),
-  bodyParser = require('body-parser'),
-  helmet = require('helmet');
-
-
-module.exports = function(app, db) {
-
-  app.use(bodyParser.json(config.bodyParser.json));
-  app.use(bodyParser.urlencoded(config.bodyParser.urlencoded));
-
-  app.use(helmet());
-
-  app.set('showStackError', true);
+var mean = require('meanio')
+var compression = require('compression')
+var consolidate = require('consolidate')
+var express = require('express')
+var helpers = require('view-helpers')
+var flash = require('connect-flash')
+var modRewrite = require('connect-modrewrite')
+// seo = require('mean-seo'),
+var config = mean.getConfig()
+var bodyParser = require('body-parser')
+var helmet = require('helmet')
+
+module.exports = function (app, db) {
+  app.use(bodyParser.json(config.bodyParser.json))
+  app.use(bodyParser.urlencoded(config.bodyParser.urlencoded))
+
+  app.use(helmet())
+
+  app.set('showStackError', true)
 
   // Prettify HTML
-  app.locals.pretty = true;
+  app.locals.pretty = true
 
   // cache=memory or swig dies in NODE_ENV=production
-  app.locals.cache = 'memory';
+  app.locals.cache = 'memory'
 
   // Should be placed before express.static
   // To ensure that all assets and data are compressed (utilize bandwidth)
   app.use(compression({
     // Levels are specified in a range of 0 to 9, where-as 0 is
     // no compression and 9 is best compression, but slowest
     level: 9
-  }));
+  }))
 
   // Enable compression on bower_components
-  app.use('/bower_components', express.static(config.root + '/bower_components'));
+  app.use('/bower_components', express.static(config.root + '/bower_components'))
 
-  app.use('/bundle', express.static(config.root + '/bundle'));
+  app.use('/bundle', express.static(config.root + '/bundle'))
 
   // Adds logging based on logging config in config/env/ entry
-  require('./middlewares/logging')(app, config.logging);
+  require('./middlewares/logging')(app, config.logging)
 
   // assign the template engine to .html files
-  app.engine('html', consolidate[config.templateEngine]);
+  app.engine('html', consolidate[config.templateEngine])
 
   // set .html as the default extension
-  app.set('view engine', 'html');
-
+  app.set('view engine', 'html')
 
   // Dynamic helpers
-  app.use(helpers(config.app.name));
+  app.use(helpers(config.app.name))
 
   // Connect flash for flash messages
-  app.use(flash());
+  app.use(flash())
 
   app.use(modRewrite([
-
-    '!^/api/.*|\\_getModules|\\.html|\\.js|\\.css|\\.swf|\\.jp(e?)g|\\.JP(E?)G|\\.PNG|\\.png|\\.ico|\\.gif|\\.svg|\\.eot|\\.ttf|\\.woff|\\.txt|\\.pdf$ / [L]'    
 
-  ]));
+    '!^/api/.*|\\_getModules|\\.html|\\.js|\\.css|\\.swf|\\.jp(e?)g|\\.JP(E?)G|\\.PNG|\\.png|\\.ico|\\.gif|\\.svg|\\.eot|\\.ttf|\\.woff|\\.txt|\\.pdf$ / [L]'
+
+  ]))
 
-  // app.use(seo());
-};
+// app.use(seo())
+}
diff --git a/config/middlewares/logging.js b/config/middlewares/logging.js
@@ -1,18 +1,17 @@
 /* globals require */
-(function() {
-  'use strict';
+;(function () {
+  'use strict'
 
-  module.exports = function(app, config) {
-    var format, options;
+  module.exports = function (app, config) {
+    var format, options
 
     if (config !== false) {
-      config = config || {};
+      config = config || {}
 
-      format  = config.format || 'dev';
-      options = config.options || {};
+      format = config.format || 'dev'
+      options = config.options || {}
 
-      app.use(require('morgan')(format, options));
+      app.use(require('morgan')(format, options))
     }
-  };
-
-})();
+  }
+})()