fix: make getCurrentUser and getCurrentUserGroups instance methods again

Author: Tom Kirkpatrick

.eslintrc

@@ -1,7 +1,8 @@
 {
-  'extends': 'fullcube',
-  'root': true,
-  'rules': {
-    no-console: 0
+  extends: fullcube,
+  root: true,
+  rules: {
+    no-console: 0,
+    class-methods-use-this: 0
   }
 }

lib/utils.js

@@ -59,7 +59,7 @@ module.exports = class AccessUtils {
       if (typeof Model.observe === 'function') {
         debug('Attaching access observer to %s', modelName)
         Model.observe('access', (ctx, next) => {
-          const currentUser = AccessUtils.getCurrentUser()
+          const currentUser = this.getCurrentUser()
 
           if (currentUser) {
             // Do not filter if options.skipAccess has been set.
@@ -198,8 +198,8 @@ module.exports = class AccessUtils {
   getUserGroups(userId, force, cb) {
     force = force || false
     cb = cb || createPromiseCallback()
-    const currentUser = AccessUtils.getCurrentUser()
-    const currentUserGroups = AccessUtils.getCurrentUserGroups()
+    const currentUser = this.getCurrentUser()
+    const currentUserGroups = this.getCurrentUserGroups()
 
     // Return from the context cache if exists.
     if (!force && currentUser && currentUser.getId() === userId) {
@@ -228,7 +228,7 @@ module.exports = class AccessUtils {
    *
    * @returns {Object} Returns the currently logged in user.
    */
-  static getCurrentUser() {
+  getCurrentUser() {
     const ctx = LoopBackContext.getCurrentContext()
     const currentUser = (ctx && ctx.get('currentUser')) || null
 
@@ -240,7 +240,7 @@ module.exports = class AccessUtils {
    *
    * @returns {Array} Returnds a list of access groups the user is a member of.
    */
-  static getCurrentUserGroups() {
+  getCurrentUserGroups() {
     const ctx = LoopBackContext.getCurrentContext()
     const currentUserGroups = (ctx && ctx.get('currentUserGroups')) || []