Unify gate guardian check

KSET · Oct 17, 2020 · 91eebd7 · 91eebd7
1 parent 649e2c6
commit 91eebd7
Show file tree

Hide file tree

Showing 4 changed files with 31 additions and 5 deletions.
diff --git a/api/routes/events.js b/api/routes/events.js
@@ -1,3 +1,6 @@
+import {
+ isGateGuardian,
+} from "../../helpers/auth";
 import {
   eventListFromStatus,
   EventStatus,
@@ -73,9 +76,8 @@ const requireCv =
 const requireGateGuardian =
   (req, res, next) => {
     const { authUser } = req;
-    const { companies } = authUser;
 
-    if (!companies || !companies.find(({ id }) => 428 === Number(id))) {
+    if (!isGateGuardian(authUser)) {
       const status = HttpStatus.Error.Client.NotAcceptable;
 
       res.status(status);

diff --git a/helpers/auth.js b/helpers/auth.js
@@ -0,0 +1,16 @@
+import {
+  dotGet,
+} from "./data";
+
+export const isGateGuardian =
+  (user) =>
+    dotGet(
+      user,
+      "companies",
+      [],
+    )
+      .find(
+        ({ id }) =>
+          428 === Number(id),
+      )
+;
diff --git a/middleware/is-gate-guardian.js b/middleware/is-gate-guardian.js
@@ -1,6 +1,9 @@
 import {
   HttpStatus,
 } from "~/api/helpers/http";
+import {
+ isGateGuardian,
+} from "~/helpers/auth";
 
 export default function({ store, error }) {
   const { user } = store.state.user;
@@ -11,9 +14,7 @@ export default function({ store, error }) {
     });
   }
 
-  const { companies } = user;
-
-  if (!companies.find(({ id }) => 428 === Number(id))) {
+  if (!isGateGuardian(user)) {
     return error({
       statusCode: HttpStatus.Error.Client.NotFound,
     });

diff --git a/store/user.js b/store/user.js
@@ -4,6 +4,9 @@ import {
   isModerator,
   isStudent,
 } from "~/api/helpers/permissions";
+import {
+ isGateGuardian,
+} from "~/helpers/auth";
 
 export const state = () => (
   {
@@ -49,6 +52,10 @@ export const getters = {
     return isStudent(user.role);
   },
 
+  isGateGuardian({ user }) {
+    return isGateGuardian(user);
+  },
+
   isLoggedIn({ user }) {
     return Boolean(user);
   },