Update: add returns
This commit is contained in:
parent
76390fcec5
commit
b73da1a288
1 changed files with 8 additions and 8 deletions
|
|
@ -338,19 +338,19 @@ exports.delete = (req, res) => {
|
||||||
User.findByIdAndUpdate(id, {isActive: false}, {useFindAndModify: false})
|
User.findByIdAndUpdate(id, {isActive: false}, {useFindAndModify: false})
|
||||||
.then(data => {
|
.then(data => {
|
||||||
if(data) {
|
if(data) {
|
||||||
sendMessage(res, 8, {message: `User ${id} was successfully deleted.`}, token);
|
return sendMessage(res, 8, {message: `User ${id} was successfully deleted.`}, token);
|
||||||
} else {
|
} else {
|
||||||
sendError(res, 404, 105, `User not found with id=${id}`, token);
|
return sendError(res, 404, 105, `User not found with id=${id}`, token);
|
||||||
}
|
}
|
||||||
})
|
})
|
||||||
.catch(err => {
|
.catch(err => {
|
||||||
sendError(res, 500, 100, err.message || `Some error occurred while deleting the User with id=${id}`, token);
|
return sendError(res, 500, 100, err.message || `Some error occurred while deleting the User with id=${id}`, token);
|
||||||
});
|
});
|
||||||
} else {
|
} else {
|
||||||
sendError(res, 500, -1, `Error id is not valid`, token);
|
return sendError(res, 500, -1, `Error id is not valid`, token);
|
||||||
}
|
}
|
||||||
} else {
|
} else {
|
||||||
sendError(res, 500, -1, `No id given`, token);
|
return sendError(res, 500, -1, `No id given`, token);
|
||||||
}
|
}
|
||||||
};
|
};
|
||||||
|
|
||||||
|
|
@ -360,12 +360,12 @@ exports.deleteAll = (req, res) => {
|
||||||
if(token) {
|
if(token) {
|
||||||
User.deleteMany({login: {$ne: "superAdmin"}})
|
User.deleteMany({login: {$ne: "superAdmin"}})
|
||||||
.then(data => {
|
.then(data => {
|
||||||
sendMessage(res, 1, {
|
return sendMessage(res, 9, {
|
||||||
message: `${data.deletedCount} Users were deleted successfully.`
|
message: `${data.deletedCount} Users were deleted successfully.`
|
||||||
});
|
});
|
||||||
})
|
})
|
||||||
.catch(err => {
|
.catch(err => {
|
||||||
sendError(res, 500, -1, err.message || "Some error occurred while removing all Users.");
|
return sendError(res, 500, 100, err.message || "Some error occurred while removing all Users.");
|
||||||
});
|
});
|
||||||
}
|
}
|
||||||
};
|
};
|
||||||
|
|
@ -374,7 +374,7 @@ exports.deleteAll = (req, res) => {
|
||||||
exports.roles = (req, res) => {
|
exports.roles = (req, res) => {
|
||||||
const token = checkLogin(req, res);
|
const token = checkLogin(req, res);
|
||||||
if(token){
|
if(token){
|
||||||
sendMessage(res, 10, roles, token);
|
return sendMessage(res, 10, roles, token);
|
||||||
}
|
}
|
||||||
};
|
};
|
||||||
|
|
||||||
|
|
|
||||||
Reference in a new issue