Refactor code

controllers/users.controller.js

@@ -52,7 +52,6 @@ exports.createUser = async function (request, response) {
 };
 
 exports.login = async function (request, response) {
-    //TO-DO: Check role admin or user?
     const account = await Account.findOne({email: request.body.email});
     if (!account) return response.status(422).send('Email is not correct');
     const checkPassword = await bcrypt.compare(request.body.password, account.password);
@@ -88,6 +87,28 @@ exports.getAllUsers = function (req, res) {
     });
 };
 
+exports.getUserById = function (req, res) {
+    const userId = req.params.id
+    User.findById(userId).exec(function (err, user) {
+        if (err) {
+            res.send(400, err);
+        } else {
+            res.send(user);
+        }
+    });
+};
+
+exports.getUserIdByUsername = function (req, res) {
+    const username = req.params.username
+    User.find({username: username}).exec(function (err, user) {
+        if (err) {
+            res.send(400, err);
+        } else {
+            res.send(user);
+        }
+    });
+};
+
 exports.getAvatar = async function (req, res) {
     const key = req.params.key;
     try {

controllers/video-info.controller.js

@@ -51,7 +51,6 @@ exports.getAllPublicVideoInfos = function (req,res) {
 
 exports.getVideoInfoById = function (req, res) {
     const id = req.params.id
-    console.log("id: ", id)
     Video.findById(id)
         .exec(function(err, result) {
             if (!err)

routes/users.route.js

@@ -3,6 +3,10 @@ const router = express.Router();
 const user = require('../controllers/users.controller');
 const verifyToken = require('./../middlewares/verifyToken.middleware');
 
+// router.get('/:id', verifyToken, user.getUserById);
+
+router.get('/:username', verifyToken, user.getUserIdByUsername);
+
 router.get('/', verifyToken, user.getAllUsers);
 
 router.post('/', user.createUser);