sup39/firefish
1
0
Fork 0
Code Issues Pull requests Activity

fix (backend): apply word mutes in notifications

Browse source 
Co-authored-by: naskya <m@naskya.net>
This commit is contained in:
sup39 2024-02-23 20:26:03 +09:00
parent 478afc6558
commit f54a44d019
Signed by: sup39
GPG key ID: 111C00916C1641E5
2 changed files with 10 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
packages/backend/src/server/api/endpoints/notes/mentions.ts
View file

@ -7,6 +7,7 @@ import { generateMutedUserQuery } from "@/server/api/common/generate-muted-user-
import { makePaginationQuery } from "@/server/api/common/make-pagination-query.js"; import { makePaginationQuery } from "@/server/api/common/make-pagination-query.js";
import { generateBlockedUserQuery } from "@/server/api/common/generate-block-query.js"; import { generateBlockedUserQuery } from "@/server/api/common/generate-block-query.js";
import { generateMutedNoteThreadQuery } from "@/server/api/common/generate-muted-note-thread-query.js"; import { generateMutedNoteThreadQuery } from "@/server/api/common/generate-muted-note-thread-query.js";
import { generateMutedNoteQuery } from "@/server/api/common/generate-muted-note-query.js";
export const meta = { export const meta = {
tags: ["notes"], tags: ["notes"],
@ -69,6 +70,7 @@ export default define(meta, paramDef, async (ps, user) => {
generateVisibilityQuery(query, user); generateVisibilityQuery(query, user);
generateMutedUserQuery(query, user); generateMutedUserQuery(query, user);
generateMutedNoteQuery(query, user);
generateMutedNoteThreadQuery(query, user); generateMutedNoteThreadQuery(query, user);
generateBlockedUserQuery(query, user); generateBlockedUserQuery(query, user);

13
packages/backend/src/services/note/create.ts
View file

@ -887,14 +887,17 @@ async function createMentionedEvents(
nm: NotificationManager, nm: NotificationManager,
) { ) {
for (const u of mentionedUsers.filter((u) => Users.isLocalUser(u))) { for (const u of mentionedUsers.filter((u) => Users.isLocalUser(u))) {
const threadMuted = await NoteThreadMutings.findOneBy({ const isWordMuted = await MutedNotes.existsBy({
userId: u.id, userId: u.id,
threadId: note.threadId || note.id, noteId: note.id,
}); });
if (isWordMuted) continue;
if (threadMuted) { const isThreadMuted = await NoteThreadMutings.existsBy({
continue; userId: u.id,
} threadId: note.threadId ?? note.id,
});
if (isThreadMuted) continue;
// note with "specified" visibility might not be visible to mentioned users // note with "specified" visibility might not be visible to mentioned users
try { try {