Leverage built-in method.

codingthat · codingthat · commit d212c8f5bf8a · 2020-09-09T15:38:38.000+02:00
diff --git a/users/models/users.model.js b/users/models/users.model.js
@@ -59,19 +59,9 @@ exports.list = (perPage, page) => {
 };
 
 exports.patchUser = (id, userData) => {
-    return new Promise((resolve, reject) => {
-        User.findById(id, function (err, user) {
-            if (err) reject(err);
-            for (let i in userData) {
-                user[i] = userData[i];
-            }
-            user.save(function (err, updatedUser) {
-                if (err) return reject(err);
-                resolve(updatedUser);
-            });
-        });
-    })
-
+    return User.findOneAndUpdate({
+        _id: id
+    }, userData);
 };
 
 exports.removeById = (userId) => {
