feat: refactor user retrieval logic to use allUsers function in login and user management routes
This commit is contained in:
@@ -60,6 +60,7 @@ function allUsers() {
|
||||
|
||||
// Route to handle user login
|
||||
app.post("/login", (req, res) => {
|
||||
|
||||
allUsers();
|
||||
|
||||
// Attempt to log in the user with provided credentials
|
||||
@@ -100,13 +101,8 @@ app.post(["/createUser", "/updateUser", "/deleteUser"], (req, res) => {
|
||||
} else {
|
||||
let response;
|
||||
|
||||
getAllUsers().then((resultFromFunc) => {
|
||||
if (resultFromFunc.success) {
|
||||
response = resultFromFunc.result;
|
||||
} else {
|
||||
response = resultFromFunc.result;
|
||||
}
|
||||
});
|
||||
allUsers();
|
||||
|
||||
// Render dashboard with alert if trying to delete logged-in user
|
||||
res.status(400).render("dashboard.ejs", {
|
||||
sqlResult: latestUser,
|
||||
@@ -133,13 +129,8 @@ app.post(["/createUser", "/updateUser", "/deleteUser"], (req, res) => {
|
||||
if (result.success === true) {
|
||||
let response;
|
||||
|
||||
getAllUsers().then((resultFromFunc) => {
|
||||
if (resultFromFunc.success) {
|
||||
response = resultFromFunc.result;
|
||||
} else {
|
||||
response = resultFromFunc.result;
|
||||
}
|
||||
});
|
||||
allUsers();
|
||||
|
||||
// On success, render dashboard with success message
|
||||
res.status(201).render("dashboard.ejs", {
|
||||
sqlResult: latestUser,
|
||||
@@ -151,13 +142,7 @@ app.post(["/createUser", "/updateUser", "/deleteUser"], (req, res) => {
|
||||
} else {
|
||||
let response;
|
||||
|
||||
getAllUsers().then((resultFromFunc) => {
|
||||
if (resultFromFunc.success) {
|
||||
response = resultFromFunc.result;
|
||||
} else {
|
||||
response = resultFromFunc.result;
|
||||
}
|
||||
});
|
||||
allUsers();
|
||||
|
||||
// On failure, render dashboard with alert
|
||||
res.status(400).render("dashboard.ejs", {
|
||||
|
Reference in New Issue
Block a user