From 78abdd7f704cf405d825e7b845da88e0a9304dbd Mon Sep 17 00:00:00 2001 From: Alex Holliday Date: Mon, 27 May 2024 12:03:32 -0700 Subject: [PATCH] Fixed bug in error handler, finished implementing update user endpoint --- Server/controllers/authController.js | 1 - Server/db/MongoDB.js | 2 +- Server/middleware/handleErrors.js | 2 +- 3 files changed, 2 insertions(+), 3 deletions(-) diff --git a/Server/controllers/authController.js b/Server/controllers/authController.js index 3d3dde796..cc476ab54 100644 --- a/Server/controllers/authController.js +++ b/Server/controllers/authController.js @@ -132,7 +132,6 @@ const userEditController = async (req, res, next) => { return; } - //Find and update user try { const updatedUser = await req.db.updateUser(req, res); res diff --git a/Server/db/MongoDB.js b/Server/db/MongoDB.js index a2e45f5d6..b3b269600 100644 --- a/Server/db/MongoDB.js +++ b/Server/db/MongoDB.js @@ -69,7 +69,7 @@ const updateUser = async (req, res) => { const candidateUser = req.body; try { - const updatedUser = await user.findByIdAndUpdate( + const updatedUser = await UserModel.findByIdAndUpdate( candidateUserId, candidateUser, { new: true } // Returns updated user instead of pre-update user diff --git a/Server/middleware/handleErrors.js b/Server/middleware/handleErrors.js index 78764c1e1..a20db8571 100644 --- a/Server/middleware/handleErrors.js +++ b/Server/middleware/handleErrors.js @@ -3,7 +3,7 @@ const logger = require("../utils/logger"); const handleErrors = (error, req, res, next) => { const status = error.status || 500; const message = error.message || "Something went wrong"; - const service = error.errorService || "Unknown service"; + const service = error.service || "Unknown service"; logger.error(error.message, { service: service }); res.status(status).json({ success: false, msg: message });