facttic/mv-issues#58 Add exec() and details

Author: pehuen-rodriguez

deny_list/dao.js

@@ -65,6 +65,7 @@ DenyListSchema.statics.getByUserIdStrByManifestationId = async function getByUse
 
 DenyListSchema.statics.removeById = async function removeById(_id, userId = null) {
   const { nModified } = await DenyListDAO.deleteById(_id, userId).exec();
+
   // nModified will be 1 when _id exists
   if (nModified) {
     return _id;

manifestation/dao.js

@@ -14,18 +14,20 @@ ManifestationSchema.statics.createNew = async function createNew(manifestation)
 
 // TODO: googlear y revisar como mergear las propiedades del objeto existente
 ManifestationSchema.statics.udpate = async function udpate(_id, manifestation) {
-  return await ManifestationDAO.findByIdAndUpdate({ _id }, manifestation, {
+  return await ManifestationDAO.findByIdAndUpdate(_id, manifestation, {
     new: true,
     runValidators: true,
   }).exec();
 };
 
 ManifestationSchema.statics.getAll = async function getAll({ skip, limit, sort, query }) {
-  const manifestationsTotal = await ManifestationDAO.countDocuments({ deleted: false });
+  const manifestationsTotal = await ManifestationDAO.countDocuments({ deleted: false }).exec();
   const manifestations = await ManifestationDAO.find({ ...query })
     .skip(skip)
     .limit(limit)
-    .sort(sort);
+    .sort(sort)
+    .exec();
+
   return {
     count: manifestations.length,
     list: manifestations,
@@ -39,6 +41,7 @@ ManifestationSchema.statics.getById = async function getById(_id) {
 
 ManifestationSchema.statics.removeById = async function removeById(_id, userId = null) {
   const { nModified } = await ManifestationDAO.deleteById(_id, userId).exec();
+
   // nModified will be 1 when _id exists
   if (nModified) {
     return _id;

post/dao.js

@@ -59,7 +59,7 @@ PostSchema.statics.getById = async function getById(_id) {
 };
 
 PostSchema.statics.removeByUserIdStr = async function removeByUserIdStr(userIdStr, userId = null) {
-  const { nModified } = await PostDAO.delete({ "user.id_str": userIdStr }, userId);
+  const { nModified } = await PostDAO.delete({ "user.id_str": userIdStr }, userId).exec();
 
   if (nModified) {
     return userIdStr;
@@ -68,7 +68,7 @@ PostSchema.statics.removeByUserIdStr = async function removeByUserIdStr(userIdSt
 };
 
 PostSchema.statics.removeById = async function removeById(_id, userId = null) {
-  const { nModified } = await PostDAO.deleteById(_id, userId);
+  const { nModified } = await PostDAO.deleteById(_id, userId).exec();
 
   if (nModified) {
     return _id;
@@ -81,7 +81,10 @@ PostSchema.statics.removeByIdByManifestationId = async function removeByIdByMani
   _id,
   userId = null,
 ) {
-  const { nModified } = await PostDAO.delete({ _id, manifestation_id: manifestationId }, userId);
+  const { nModified } = await PostDAO.delete(
+    { _id, manifestation_id: manifestationId },
+    userId,
+  ).exec();
 
   if (nModified) {
     return _id;