improve emoji rendering

Signed-off-by: eternal-flame-AD <yume@yumechi.jp>

packages/backend/src/core/entities/EmojiEntityService.ts

@@ -10,6 +10,7 @@ import type { Packed } from '@/misc/json-schema.js';
 import type { } from '@/models/Blocking.js';
 import type { MiEmoji } from '@/models/Emoji.js';
 import { bindThis } from '@/decorators.js';
+import { In } from 'typeorm';
 
 @Injectable()
 export class EmojiEntityService {
@@ -20,11 +21,9 @@ export class EmojiEntityService {
 	}
 
 	@bindThis
-	public async packSimple(
+	public packSimpleNoQuery(
-		src: MiEmoji['id'] | MiEmoji,
+		emoji: MiEmoji,
-	): Promise<Packed<'EmojiSimple'>> {
+	): Packed<'EmojiSimple'> {
-		const emoji = typeof src === 'object' ? src : await this.emojisRepository.findOneByOrFail({ id: src });
-
 		return {
 			aliases: emoji.aliases,
 			name: emoji.name,
@@ -38,18 +37,34 @@ export class EmojiEntityService {
 	}
 
 	@bindThis
-	public packSimpleMany(
+	public async packSimple(
-		emojis: any[],
+		src: MiEmoji['id'] | MiEmoji,
-	) {
+	): Promise<Packed<'EmojiSimple'>> {
-		return Promise.all(emojis.map(x => this.packSimple(x)));
+		const emoji = typeof src === 'object' ? src : await this.emojisRepository.findOneByOrFail({ id: src });
+
+		return this.packSimpleNoQuery(emoji);
 	}
 
 	@bindThis
-	public async packDetailed(
+	public async packSimpleMany(
-		src: MiEmoji['id'] | MiEmoji,
+		emojis: MiEmoji['id'][] | MiEmoji[],
-	): Promise<Packed<'EmojiDetailed'>> {
+	): Promise<Packed<'EmojiSimple'>[]> {
-		const emoji = typeof src === 'object' ? src : await this.emojisRepository.findOneByOrFail({ id: src });
+		if (emojis.length === 0) {
+			return [];
+		}
 		
+		if (typeof emojis[0] === 'string') {
+			const res = await this.emojisRepository.findBy({ id: In(emojis as MiEmoji['id'][]) });
+			return res.map(this.packSimpleNoQuery);
+		}
+
+		return (emojis as MiEmoji[]).map(this.packSimpleNoQuery);
+	}
+
+	@bindThis
+	public packDetailedNoQuery(
+		emoji: MiEmoji,
+	): Packed<'EmojiDetailed'> {
 		return {
 			id: emoji.id,
 			aliases: emoji.aliases,
@@ -65,11 +80,28 @@ export class EmojiEntityService {
 		};
 	}
 
+	@bindThis
+	public async packDetailed(
+		src: MiEmoji['id'] | MiEmoji,
+	): Promise<Packed<'EmojiDetailed'>> {
+		const emoji = typeof src === 'object' ? src : await this.emojisRepository.findOneByOrFail({ id: src });
+
+		return this.packDetailedNoQuery(emoji);
+	}
+
 	@bindThis
 	public packDetailedMany(
-		emojis: any[],
+		emojis: MiEmoji['id'][] | MiEmoji[],
 	) {
-		return Promise.all(emojis.map(x => this.packDetailed(x)));
+		if (emojis.length === 0) {
+			return [];
+		}
+		
+		if (typeof emojis[0] === 'string') {
+			return this.emojisRepository.findBy({ id: In(emojis as MiEmoji['id'][]) }).then(res => res.map(this.packDetailedNoQuery));
+		}
+
+		return (emojis as MiEmoji[]).map(this.packDetailedNoQuery);
 	}
 }
 

packages/backend/src/server/api/endpoints/emojis.ts

@@ -50,18 +50,15 @@ export default class extends Endpoint<typeof meta, typeof paramDef> { // eslint-
 		private emojiEntityService: EmojiEntityService,
 	) {
 		super(meta, paramDef, async (ps, me) => {
-			const emojis = await this.emojisRepository.find({
+			const emojis = await this.emojisRepository
-				where: {
+			.createQueryBuilder()
-					host: IsNull(),
+			.where({ host: IsNull() })
-				},
+			.orderBy('LOWER(category)', 'ASC')
-				order: {
+			.addOrderBy('LOWER(name)', 'ASC')
-					category: 'ASC',
+			.getMany();
-					name: 'ASC',
-				},
-			});
 
 			return {
-				emojis: await this.emojiEntityService.packSimpleMany(emojis),
+				emojis: emojis.map(this.emojiEntityService.packSimpleNoQuery),			
 			};
 		});
 	}