style: fix eslint issues

Author: Tom Kirkpatrick

lib/index.js

@@ -1,36 +1,35 @@
-'use strict';
+'use strict'
 
-const debug = require('debug')('loopback:component:access');
-const AccessUtils = require('./utils');
-const accessLogger = require('./middleware/access-logger');
-const userContext = require('./middleware/user-context');
+const debug = require('debug')('loopback:component:access')
+const AccessUtils = require('./utils')
+const accessLogger = require('./middleware/access-logger')
+const userContext = require('./middleware/user-context')
 
 module.exports = function loopbackComponentAccess(app, options) {
-  debug('initializing component');
-  const loopback = app.loopback;
-  const loopbackMajor = loopback && loopback.version &&
-    loopback.version.split('.')[0] || 1;
+  debug('initializing component')
+  const { loopback } = app
+  const loopbackMajor = (loopback && loopback.version && loopback.version.split('.')[0]) || 1
 
   if (loopbackMajor < 2) {
-    throw new Error('loopback-component-access-groups requires loopback 2.0 or newer');
+    throw new Error('loopback-component-access-groups requires loopback 2.0 or newer')
   }
 
   // Initialize middleware
-  app.middleware('auth:after', userContext());
-  app.middleware('routes:before', accessLogger());
+  app.middleware('auth:after', userContext())
+  app.middleware('routes:before', accessLogger())
 
   // Initialise helper class.
-  const accessUtils = new AccessUtils(app, options);
+  const accessUtils = new AccessUtils(app, options)
 
-  app.accessUtils = accessUtils;
+  app.accessUtils = accessUtils
 
   // Set up role resolvers.
-  accessUtils.setupRoleResolvers();
+  accessUtils.setupRoleResolvers()
 
   // Set up model opertion hooks.
   if (options.applyToStatic) {
-    accessUtils.setupFilters();
+    accessUtils.setupFilters()
   }
 
 // TODO: Create Group Access model automatically if one hasn't been specified
-};
+}

lib/middleware/access-logger.js

@@ -1,39 +1,39 @@
-'use strict';
+'use strict'
 
-const debug = require('debug')('loopback:component:access:logger');
+const debug = require('debug')('loopback:component:access:logger')
 
 module.exports = function accessLoggerMiddleware() {
-  debug('initializing access logger middleware');
+  debug('initializing access logger middleware')
   return function accessLogger(req, res, next) {
     if (req.accessToken) {
-      debug('req: %s %s, token: %o', req.method, req.originalUrl, req.accessToken);
+      debug('req: %s %s, token: %o', req.method, req.originalUrl, req.accessToken)
     }
     else {
-      debug('req', req.method, req.originalUrl);
+      debug('req', req.method, req.originalUrl)
     }
 
-    const start = new Date();
+    const start = new Date()
 
     if (res._responseTime) {
-      return next();
+      return next()
     }
-    res._responseTime = true;
+    res._responseTime = true
 
     // install a listener for when the response is finished
     res.on('finish', () => {
       // the request was handled, print the log entry
-      const duration = new Date() - start;
+      const duration = new Date() - start
 
       debug('res %s %s: %o', req.method, req.originalUrl, {
         lbHttpMethod: req.method,
         lbUrl: req.originalUrl,
         lbStatusCode: res.statusCode,
         lbResponseTime: duration,
-        lbResponseTimeUnit: 'ms'
+        lbResponseTimeUnit: 'ms',
       }
-      );
-    });
+      )
+    })
 
-    return next();
-  };
-};
+    return next()
+  }
+}

lib/middleware/user-context.js

@@ -1,42 +1,42 @@
-'use strict';
+'use strict'
 
-const debug = require('debug')('loopback:component:access:context');
-const loopback = require('loopback');
-const Promise = require('bluebird');
+const debug = require('debug')('loopback:component:access:context')
+const loopback = require('loopback')
+const Promise = require('bluebird')
 
 module.exports = function userContextMiddleware() {
-  debug('initializing user context middleware');
+  debug('initializing user context middleware')
   // set current user to enable user access for remote methods
   return function userContext(req, res, next) {
-    const loopbackContext = loopback.getCurrentContext();
+    const loopbackContext = loopback.getCurrentContext()
 
     if (!loopbackContext) {
-      debug('No user context (loopback current context not found)');
-      return next();
+      debug('No user context (loopback current context not found)')
+      return next()
     }
 
     if (!req.accessToken) {
-      debug('No user context (access token not found)');
-      return next();
+      debug('No user context (access token not found)')
+      return next()
     }
 
-    loopbackContext.set('accessToken', req.accessToken.id);
-    const app = req.app;
-    const UserModel = app.accessUtils.options.userModel || 'User';
+    loopbackContext.set('accessToken', req.accessToken.id)
+    const { app } = req
+    const UserModel = app.accessUtils.options.userModel || 'User'
 
     return Promise.join(
       app.models[UserModel].findById(req.accessToken.userId),
       app.accessUtils.getUserGroups(req.accessToken.userId),
       (user, groups) => {
         if (!user) {
-          return next(new Error('No user with this access token was found.'));
+          return next(new Error('No user with this access token was found.'))
         }
-        loopbackContext.set('currentUser', user);
-        loopbackContext.set('currentUserGroups', groups);
-        debug('currentUser', user);
-        debug('currentUserGroups', groups);
-        return next();
+        loopbackContext.set('currentUser', user)
+        loopbackContext.set('currentUserGroups', groups)
+        debug('currentUser', user)
+        debug('currentUserGroups', groups)
+        return next()
       })
-      .catch(next);
-  };
-};
+      .catch(next)
+  }
+}

lib/mixins/get-current-user.js

@@ -1,23 +1,23 @@
-'use strict';
+'use strict'
 
-const loopback = require('loopback');
-const debug = require('debug')('loopback:component:access:utils');
+const loopback = require('loopback')
+const debug = require('debug')('loopback:component:access:utils')
 
 module.exports = function getCurrentUserMixin(Model) {
-  debug('initializing GetCurrentUser Mixin for model %s', Model.modelName);
+  debug('initializing GetCurrentUser Mixin for model %s', Model.modelName)
 
   Model.getCurrentUser = function getCurrentUser() {
-    const ctx = loopback.getCurrentContext();
-    const currentUser = ctx && ctx.get('currentUser') || null;
+    const ctx = loopback.getCurrentContext()
+    const currentUser = (ctx && ctx.get('currentUser')) || null
 
     if (ctx) {
-      debug(`${Model.definition.name}.getCurrentUser() - currentUser: %o`, currentUser);
+      debug(`${Model.definition.name}.getCurrentUser() - currentUser: %o`, currentUser)
     }
     else {
       // this means its a server-side logic call w/o any HTTP req/resp aspect to it.
-      debug(`${Model.definition.name}.getCurrentUser() - no loopback context`);
+      debug(`${Model.definition.name}.getCurrentUser() - no loopback context`)
     }
 
-    return currentUser;
-  };
-};
+    return currentUser
+  }
+}