From 5bc0d623a55b77b25bb1fc975cc82171293cdcfd Mon Sep 17 00:00:00 2001 From: LoneRifle Date: Sat, 20 Oct 2018 16:33:12 +0800 Subject: [PATCH] deleteSubmissions - remove spurious condition `FormSubmission.remove` should not use `admin` as a condition: - the field is not present in the schema or in any created submissions - the user is unlikely to be the admin of a submission Remove this field to facilitate deletes Fixes #315 --- app/controllers/forms.server.controller.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/controllers/forms.server.controller.js b/app/controllers/forms.server.controller.js index c511bcf0..14198564 100644 --- a/app/controllers/forms.server.controller.js +++ b/app/controllers/forms.server.controller.js @@ -19,7 +19,7 @@ exports.deleteSubmissions = function(req, res) { var submission_id_list = req.body.deleted_submissions, form = req.form; - FormSubmission.remove({ form: req.form, admin: req.user, _id: {$in: submission_id_list} }, function(err){ + FormSubmission.remove({ form: req.form, _id: {$in: submission_id_list} }, function(err){ if(err){ res.status(400).send({