Bladeren bron

Move mentions logic before test, use for loop

sbkwgh 8 jaren geleden
bovenliggende
commit
20a981a516
1 gewijzigde bestanden met toevoegingen van 6 en 6 verwijderingen
  1. 6 6
      routes/post.js

+ 6 - 6
routes/post.js

@@ -154,6 +154,12 @@ router.post('/', async (req, res) => {
 
 		await thread.increment('postsCount')
 
+		if(req.body.mentions) {
+			for(var i = 0; i < req.body.mentions.length; i++) {
+				await Notification.createMention({ mention: req.body.mentions[i], user, post })
+			}
+		}
+
 		res.json(await post.reload({
 			include: Post.includeOptions()
 		}))
@@ -162,12 +168,6 @@ router.post('/', async (req, res) => {
 			postNumber: thread.postsCount
 		})
 
-		if(req.body.mentions) {
-			req.body.mentions.forEach(mention => {
-				Notification.createMention({ mention, user, post })
-			})
-		}
-
 	} catch (e) {
 		if(e === Errors.VALIDATION_ERROR) {
 			res.status(400)