give some of the logs more context

PenguinMod · Jul 21, 2024 · 9c89377 · 9c89377
1 parent 3b7536a
commit 9c89377
Show file tree

Hide file tree

Showing 6 changed files with 47 additions and 7 deletions.
diff --git a/api/v1/routes/users/meta/customization/setCustomization.js b/api/v1/routes/users/meta/customization/setCustomization.js
@@ -16,8 +16,6 @@ module.exports = (app, utils) => {
             return;
         }
 
-        const badges = await utils.UserManager.getBadges(username);
-
         if (!await utils.UserManager)
 
         await utils.UserManager.setUserCustomization(username, customization);

diff --git a/api/v1/routes/users/mod/assignPossition.js b/api/v1/routes/users/mod/assignPossition.js
@@ -23,10 +23,28 @@ module.exports = (app, utils) => {
             return;
         }
 
+        const isAdmin = await utils.UserManager.isAdmin(target);
+        const isModerator = await utils.UserManager.isModerator(target);
+
         await utils.UserManager.setAdmin(target, Boolean(admin));
         await utils.UserManager.setModerator(target, Boolean(approver));
 
-        utils.logs.sendAdminUserLog(username, target, "Admin or mod has updated user's permissions.", 0x7f3ddc);
+        let fields = [];
+
+        if (isAdmin !== admin) {
+            fields.push({
+                name: "Admin",
+                value: `${isAdmin} -> ${admin}`
+            });
+        }
+        if (isModerator !== approver) {
+            fields.push({
+                name: "Approver",
+                value: `${isModerator} -> ${approver}`
+            });
+        }
+
+        utils.logs.sendAdminUserLog(username, target, "Admin or mod has updated user's permissions.", 0x7f3ddc, fields);
 
         res.status(200);
         res.header("Content-Type", 'application/json');

diff --git a/api/v1/routes/users/mod/setBadges.js b/api/v1/routes/users/mod/setBadges.js
@@ -30,7 +30,12 @@ module.exports = (app, utils) => {
 
         await utils.UserManager.setBadges(target, badges);
 
-        utils.logs.sendAdminUserLog(username, target, "Admin has updated user's badges.", 0x3d4ddc);
+        utils.logs.sendAdminUserLog(username, target, "Admin has updated user's badges.", 0x3d4ddc, [
+            {
+                name: "Badges",
+                value: badges.join(", ")
+            }
+        ]);
 
         res.status(200);
         res.header('Content-type', "application/json");

diff --git a/api/v1/routes/users/mod/setFeaturedProjectAdmin.js b/api/v1/routes/users/mod/setFeaturedProjectAdmin.js
@@ -36,7 +36,16 @@ module.exports = (app, utils) => {
         await utils.UserManager.setFeaturedProject(target, project);
         await utils.UserManager.setFeaturedProjectTitle(target, title);
 
-        utils.logs.sendAdminUserLog(username, target, "Admin or mod has updated user's featured project.", 0xc1e518);
+        utils.logs.sendAdminUserLog(username, target, "Admin or mod has updated user's featured project.", 0xc1e518, [
+            {
+                name: "Project",
+                value: project
+            },
+            {
+                name: "Title",
+                value: title
+            }
+        ]);
 
         res.status(200);
         res.header("Content-Type", 'application/json');

diff --git a/api/v1/routes/users/mod/setUsername.js b/api/v1/routes/users/mod/setUsername.js
@@ -25,7 +25,16 @@ module.exports = (app, utils) => {
 
         await utils.UserManager.changeUsername(target, newUsername);
 
-        utils.logs.sendAdminUserLog(username, newUsername, `Admin or mod has updated user's username.\nOld username: ${target}\nNew username: ${newUsername}`, 0xf47420);
+        utils.logs.sendAdminUserLog(username, newUsername, `Admin or mod has updated user's username.`, 0xf47420, [
+            {
+                name: "Old Username",
+                value: target
+            },
+            {
+                name: "New Username",
+                value: newUsername
+            }
+        ]);
 
         res.status(200);
         res.header("Content-Type", 'application/json');

diff --git a/utils/Logs.js b/utils/Logs.js
@@ -167,7 +167,7 @@ function sendMultiReportLog(username, id, target, targetID, reason) {
     }
 }
 
-function sendAdminUserLog(username, target, action, color=0xff0000) {
+function sendAdminUserLog(username, target, action, color=0xff0000, extraFields=[]) {
     sendAdminLog(
         {
             action,
@@ -181,6 +181,7 @@ function sendAdminUserLog(username, target, action, color=0xff0000) {
                     name: "Target",
                     value: target
                 },
+                ...extraFields,
                 {
                     name: "URL",
                     value: `https://penguinmod.com/profile?user=${target}`
-Original file line number
+Diff line change
@@ Expand Up / @@ -16,8 +16,6 @@ module.exports = (app, utils) => { @@
                 return;
             }
-            const badges = await utils.UserManager.getBadges(username);
             if (!await utils.UserManager)
             await utils.UserManager.setUserCustomization(username, customization);
@@ Expand Down @@