restrict usersettings to owner plus some minor eslin fixes

Author: appinteractive

server/hooks/map-create-to-upsert.js

@@ -3,7 +3,7 @@
 
 module.exports = function (upsertQuery) {
   if (typeof upsertQuery !== 'function') {
-    console.warn('No `upsertQuery` function was passed to the mapCreateToUpsert hook. Please set params.upsertQuery in the hook context to dynamically declare the function.');
+    throw new Error('No `upsertQuery` function was passed to the mapCreateToUpsert hook. Please set params.upsertQuery in the hook context to dynamically declare the function.');
   }
 
   return function mapCreateToUpsert (context) {

server/seeder/development/usersettings.js

@@ -1,15 +1,14 @@
-const seedHelpers = require('../../helper/seed-helpers');
-const _ = require('lodash');
+const { size, keys } = require('lodash');
 
 let userKeys = [];
 
 // eslint-disable-next-line no-unused-vars
 module.exports = (seederstore) => {
-  userKeys = _.keys(seederstore.users);
+  userKeys = keys(seederstore.users);
   return {
     services: [{
       path: 'usersettings',
-      count: _.size(seederstore.users),
+      count: size(seederstore.users),
       template: {
         userId: () => {
           return userKeys.pop();

server/services/usersettings/usersettings.hooks.js

@@ -12,9 +12,15 @@ module.exports = {
         return { userId: data.userId };
       })
     ],
-    update: [],
-    patch: [],
-    remove: []
+    update: [
+      restrictToOwner()
+    ],
+    patch: [
+      restrictToOwner()
+    ],
+    remove: [
+      restrictToOwner()
+    ]
   },
   after: {
     all: [],