Nemcio/NodeBB
1
0
Fork 0
mirror of https://github.com/NodeBB/NodeBB.git synced 2025-11-02 03:55:55 +01:00
Code Issues Packages Projects Releases Wiki Activity

fix: #11841, use unique nid per user who flagged

Browse Source 
if multiple users flag a post/user, the notifications are merged using mergeId.
if they use the same nid they overwrite each other
This commit is contained in:
Barış Soner Uşaklı
2023-07-25 10:24:18 -04:00
parent e43fca1163
commit 17783440e4
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
src/flags.js
View File

@@ -844,7 +844,7 @@ Flags.notify = async function (flagObj, uid, notifySelf = false) {
bodyLong: await plugins.hooks.fire('filter:parse.raw', String(flagObj.description || '')),
pid: flagObj.targetId,
path: `/flags/${flagObj.flagId}`,
nid: `flag:post:${flagObj.targetId}`,
nid: `flag:post:${flagObj.targetId}:${uid}`,
from: uid,
mergeId: `notifications:user_flagged_post_in|${flagObj.targetId}`,
topicTitle: title,
@@ -857,7 +857,7 @@ Flags.notify = async function (flagObj, uid, notifySelf = false) {
bodyShort: `[[notifications:user_flagged_user, ${displayname}, ${targetDisplayname}]]`,
bodyLong: await plugins.hooks.fire('filter:parse.raw', String(flagObj.description || '')),
path: `/flags/${flagObj.flagId}`,
nid: `flag:user:${flagObj.targetId}`,
nid: `flag:user:${flagObj.targetId}:${uid}`,
from: uid,
mergeId: `notifications:user_flagged_user|${flagObj.targetId}`,
});