if (notes.length === 0) return;

misskey-dev · Feb 21, 2024 · 22e2324 · 22e2324
1 parent 6be1d98
commit 22e2324
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 6 deletions.
diff --git a/packages/backend/src/core/NoteReadService.ts b/packages/backend/src/core/NoteReadService.ts
@@ -88,6 +88,8 @@ export class NoteReadService implements OnApplicationShutdown {
 		userId: MiUser['id'],
 		notes: (MiNote | Packed<'Note'>)[],
 	): Promise<void> {
+		if (notes.length === 0) return;
+
 		const readMentions: (MiNote | Packed<'Note'>)[] = [];
 		const readSpecifiedNotes: (MiNote | Packed<'Note'>)[] = [];
 

diff --git a/packages/backend/src/core/entities/NotificationEntityService.ts b/packages/backend/src/core/entities/NotificationEntityService.ts
@@ -237,9 +237,7 @@ export class NotificationEntityService implements OnModuleInit {
 					}
 					return acc;
 				}, new Set<Packed<'Note'>>());
-				if (notesToRead.size > 0) {
-					trackPromise(this.noteReadService.read(meId, Array.from(notesToRead)));
-				}
+				trackPromise(this.noteReadService.read(meId, Array.from(notesToRead)));
 			} catch (e) {
 				// console.error('error thrown by NoteReadService.read', e);
 			}

diff --git a/packages/backend/src/server/api/endpoints/antennas/notes.ts b/packages/backend/src/server/api/endpoints/antennas/notes.ts
@@ -124,9 +124,7 @@ export default class extends Endpoint<typeof meta, typeof paramDef> { // eslint-
 				notes.sort((a, b) => a.id > b.id ? -1 : 1);
 			}
 
-			if (notes.length > 0) {
-				this.noteReadService.read(me.id, notes);
-			}
+			this.noteReadService.read(me.id, notes);
 
 			return await this.noteEntityService.packMany(notes, me);
 		});