From 99f5aad94238030207bd0a349bb4ea902aefd822 Mon Sep 17 00:00:00 2001
From: "G.Ambatte" <sean@robertson-family.nz>
Date: Thu, 25 Apr 2024 14:32:04 +1200
Subject: [PATCH] Stop review request if it has already been logged

---
 server/admin.api.js | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/server/admin.api.js b/server/admin.api.js
index 846757f23..dfaa6d03f 100644
--- a/server/admin.api.js
+++ b/server/admin.api.js
@@ -193,19 +193,20 @@ router.get('/admin/lock/requestreview/:id', mw.adminOnly, async (req, res)=>{
 			'lock.locked' : true
 		};
 
-		const brew = await HomebrewModel.findOne(filter).exec();
+		const brew = await HomebrewModel.findOne(filter);
 		if(!brew) { return res.status(500).json({ error: `Brew ID ${req.params.id} is not locked!` }); };
 
+		if(brew.lock.reviewRequested){ return res.status(500).json({ error: `Review already requested for brew ${brew.shareId} - ${brew.title}` }); };
+
 		brew.lock.reviewRequested = new Date();
 		brew.markModified('lock');
 
-		console.log(brew);
-
 		await brew.save()
 		.catch((err)=>{
 			return err;
 		});
 
+		console.log(`Review requested on brew ${brew.shareId} - ${brew.title}`);
 		return res.json(brew);
 	} catch (error) {
 		console.error(error);