From b9caca8794a2631ed5d26d9ca05b9786daa94e7f Mon Sep 17 00:00:00 2001 From: Marty Sloan Date: Thu, 21 Jun 2018 20:40:53 -0400 Subject: [PATCH] update routes/admin.js to fix bug deleting product options --- routes/admin.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/routes/admin.js b/routes/admin.js index 9ef5673..db0aeb9 100644 --- a/routes/admin.js +++ b/routes/admin.js @@ -187,11 +187,11 @@ router.post('/admin/settings/option/remove', common.restrict, common.checkAccess let optJson = JSON.parse(product.productOptions); delete optJson[req.body.optName]; - db.products.update({_id: common.getImages(req.body.productId)}, {$set: {productOptions: JSON.stringify(optJson)}}, (err, numReplaced) => { + db.products.update({_id: common.getId(req.body.productId)}, {$set: {productOptions: JSON.stringify(optJson)}}, (err, numReplaced) => { if(err){ console.info(err.stack); } - if(numReplaced === 1){ + if(numReplaced.result.nModified === 1){ res.status(200).json({message: 'Option successfully removed'}); }else{ res.status(400).json({message: 'Failed to remove option. Please try again.'});