Edited Permission Change Logs controller + router

Edited controller and router related to PermissionChangeLog to handle getting permissionChangeLogs from the database.
OneCommunityGlobal · Sep 21, 2023 · d516ccc · d516ccc
1 parent c2cd1e5
commit d516ccc
Show file tree

Hide file tree

Showing 3 changed files with 20 additions and 13 deletions.
diff --git a/src/controllers/permissionChangeLogsController.js b/src/controllers/permissionChangeLogsController.js
@@ -1,10 +1,25 @@
+const UserProfile = require('../models/userProfile');
+
 const permissionChangeLogController = function (PermissionChangeLog) {
+
   const getPermissionChangeLogs = async function (req, res) {
-    // Check if user is Owner here. Skipped for now.
 
-    const changeLogs = await PermissionChangeLog.find({})
-    console.log("🚀 ~ file: permissionChangeLogsController.js:8 ~ getPermissionChangeLogs ~ changeLogs:", changeLogs)
-    res.status(200).send(changeLogs)
+    try {
+      const userProfile = await UserProfile.findOne({ _id: req.params.userId }).exec()
+
+      if (userProfile) {
+        if (userProfile.role !== 'Owner') {
+          res.status(204).send([])
+        } else {
+          const changeLogs = await PermissionChangeLog.find({})
+          res.status(200).send(changeLogs)
+        }
+      } else {
+        res.status(403).send(`User (${req.params.userId}) not found.`)
+      }
+    } catch (err) {
+      console.error(err)
+    }
   }
 
   return {

diff --git a/src/routes/permissionChangeLogsRouter.js b/src/routes/permissionChangeLogsRouter.js
@@ -5,7 +5,7 @@ const routes = function (permissionChangeLog) {
 
   const permissionChangeLogRouter = express.Router()
 
-  permissionChangeLogRouter.route("/permissionChangeLogs")
+  permissionChangeLogRouter.route("/permissionChangeLogs/:userId")
   .get(controller.getPermissionChangeLogs)
 
   return permissionChangeLogRouter

diff --git a/src/utilities/logPermissionChangeByAccount.js b/src/utilities/logPermissionChangeByAccount.js
@@ -19,7 +19,6 @@ const findLatestRelatedLog = (roleId) => {
           reject(err);
           return;
         }
-        console.log("🚀 ~ file: logPermissionChangeByAccount.js:20 ~ .exec ~ document:", document);
         resolve(document);
       });
   })
@@ -36,15 +35,10 @@ const logPermissionChangeByAccount = async (requestBody) => {
 
     // Find the latest log related to permission
     const document = await findLatestRelatedLog(roleId)
-    console.log("🚀 ~ file: logPermissionChangeByAccount.js:36 ~ logPermissionChangeByAccount ~ document:", document)
 
     if (document) {
-      // console.log(document);
-      console.log("🚀 ~ file: logPermissionChangeByAccount.js:29 ~ .exec ~ document:", document)
       permissionsRemoved = document.permissions.filter(item => !(permissions.includes(item)))
-      console.log("🚀 ~ file: logPermissionChangeByAccount.js:31 ~ .exec ~ permissionsRemoved:", permissionsRemoved)
       permissionsAdded = permissions.filter(item => !(document.permissions.includes(item)))
-      console.log("🚀 ~ file: logPermissionChangeByAccount.js:33 ~ .exec ~ permissionsAdded:", permissionsAdded)
     } else {
       // else this is the first permissions change log for this particular role
       permissionsAdded = permissions
@@ -62,8 +56,6 @@ const logPermissionChangeByAccount = async (requestBody) => {
       requestorRole: role,
       requestorEmail: email,
     })
-
-    console.log("🚀 ~ file: logPermissionChangeByAccount.js:46 ~ .exec ~ logEntry:", logEntry)
 
     await logEntry.save()