use CacheService.getUsers instead of promiseMap+CacheService.getOptionalUserById

This commit is contained in:
Hazelnoot 2025-09-16 11:58:11 -04:00
parent 1d38814450
commit 7fe1f8b39f

View file

@ -34,7 +34,6 @@ import { CacheService } from '@/core/CacheService.js';
import { NoteVisibilityService } from '@/core/NoteVisibilityService.js';
import { TimeService } from '@/global/TimeService.js';
import { CollapsedQueueService } from '@/core/CollapsedQueueService.js';
import { promiseMap } from '@/misc/promise-map.js';
import type { DataSource } from 'typeorm';
const FALLBACK = '\u2764';
@ -299,9 +298,11 @@ export class ReactionService implements OnModuleInit {
if (['public', 'home', 'followers'].includes(note.visibility)) {
dm.addFollowersRecipe();
} else if (note.visibility === 'specified') {
const visibleUsers = await promiseMap(note.visibleUserIds, async id => await this.cacheService.findOptionalUserById(id), { limit: 2 });
for (const u of visibleUsers.filter(u => u && isRemoteUser(u))) {
dm.addDirectRecipe(u as MiRemoteUser);
const visibleUsers = await this.cacheService.getUsers(note.visibleUserIds);
for (const u of visibleUsers.values()) {
if (isRemoteUser(u)) {
dm.addDirectRecipe(u as MiRemoteUser);
}
}
}