fix(backend): notes/reactionsのページネーションが機能しない
This commit is contained in:
parent
ad8ddbf12f
commit
cac1c2f1e9
2 changed files with 10 additions and 15 deletions
|
@ -89,6 +89,7 @@
|
||||||
- Enhance: 自分へのメンション一覧を取得する際のパフォーマンスを向上
|
- Enhance: 自分へのメンション一覧を取得する際のパフォーマンスを向上
|
||||||
- Enhance: Docker環境でjemallocを使用することでメモリ使用量を削減
|
- Enhance: Docker環境でjemallocを使用することでメモリ使用量を削減
|
||||||
- Fix: MK_ONLY_SERVERオプションを指定した際にクラッシュする問題を修正
|
- Fix: MK_ONLY_SERVERオプションを指定した際にクラッシュする問題を修正
|
||||||
|
- Fix: notes/reactionsのページネーションが機能しない問題を修正
|
||||||
- Fix: ノート検索 `notes/search` にてhostを指定した際に検索結果に反映されるように
|
- Fix: ノート検索 `notes/search` にてhostを指定した際に検索結果に反映されるように
|
||||||
- Fix: 一部のfeatured noteを照会できない問題を修正
|
- Fix: 一部のfeatured noteを照会できない問題を修正
|
||||||
- Fix: muteがapiからのuser list timeline取得で機能しない問題を修正
|
- Fix: muteがapiからのuser list timeline取得で機能しない問題を修正
|
||||||
|
|
|
@ -4,12 +4,13 @@
|
||||||
*/
|
*/
|
||||||
|
|
||||||
import { Inject, Injectable } from '@nestjs/common';
|
import { Inject, Injectable } from '@nestjs/common';
|
||||||
|
import { Brackets, type FindOptionsWhere } from 'typeorm';
|
||||||
import type { NoteReactionsRepository } from '@/models/_.js';
|
import type { NoteReactionsRepository } from '@/models/_.js';
|
||||||
import type { MiNoteReaction } from '@/models/NoteReaction.js';
|
import type { MiNoteReaction } from '@/models/NoteReaction.js';
|
||||||
import { Endpoint } from '@/server/api/endpoint-base.js';
|
import { Endpoint } from '@/server/api/endpoint-base.js';
|
||||||
import { NoteReactionEntityService } from '@/core/entities/NoteReactionEntityService.js';
|
import { NoteReactionEntityService } from '@/core/entities/NoteReactionEntityService.js';
|
||||||
import { DI } from '@/di-symbols.js';
|
import { DI } from '@/di-symbols.js';
|
||||||
import type { FindOptionsWhere } from 'typeorm';
|
import { QueryService } from '@/core/QueryService.js';
|
||||||
|
|
||||||
export const meta = {
|
export const meta = {
|
||||||
tags: ['notes', 'reactions'],
|
tags: ['notes', 'reactions'],
|
||||||
|
@ -44,7 +45,6 @@ export const paramDef = {
|
||||||
noteId: { type: 'string', format: 'misskey:id' },
|
noteId: { type: 'string', format: 'misskey:id' },
|
||||||
type: { type: 'string', nullable: true },
|
type: { type: 'string', nullable: true },
|
||||||
limit: { type: 'integer', minimum: 1, maximum: 100, default: 10 },
|
limit: { type: 'integer', minimum: 1, maximum: 100, default: 10 },
|
||||||
offset: { type: 'integer', default: 0 },
|
|
||||||
sinceId: { type: 'string', format: 'misskey:id' },
|
sinceId: { type: 'string', format: 'misskey:id' },
|
||||||
untilId: { type: 'string', format: 'misskey:id' },
|
untilId: { type: 'string', format: 'misskey:id' },
|
||||||
},
|
},
|
||||||
|
@ -58,29 +58,23 @@ export default class extends Endpoint<typeof meta, typeof paramDef> { // eslint-
|
||||||
private noteReactionsRepository: NoteReactionsRepository,
|
private noteReactionsRepository: NoteReactionsRepository,
|
||||||
|
|
||||||
private noteReactionEntityService: NoteReactionEntityService,
|
private noteReactionEntityService: NoteReactionEntityService,
|
||||||
|
private queryService: QueryService,
|
||||||
) {
|
) {
|
||||||
super(meta, paramDef, async (ps, me) => {
|
super(meta, paramDef, async (ps, me) => {
|
||||||
const query = {
|
const query = this.queryService.makePaginationQuery(this.noteReactionsRepository.createQueryBuilder('reaction'), ps.sinceId, ps.untilId)
|
||||||
noteId: ps.noteId,
|
.andWhere('reaction.noteId = :noteId', { noteId: ps.noteId })
|
||||||
} as FindOptionsWhere<MiNoteReaction>;
|
.leftJoinAndSelect('reaction.user', 'user')
|
||||||
|
.leftJoinAndSelect('reaction.note', 'note');
|
||||||
|
|
||||||
if (ps.type) {
|
if (ps.type) {
|
||||||
// ローカルリアクションはホスト名が . とされているが
|
// ローカルリアクションはホスト名が . とされているが
|
||||||
// DB 上ではそうではないので、必要に応じて変換
|
// DB 上ではそうではないので、必要に応じて変換
|
||||||
const suffix = '@.:';
|
const suffix = '@.:';
|
||||||
const type = ps.type.endsWith(suffix) ? ps.type.slice(0, ps.type.length - suffix.length) + ':' : ps.type;
|
const type = ps.type.endsWith(suffix) ? ps.type.slice(0, ps.type.length - suffix.length) + ':' : ps.type;
|
||||||
query.reaction = type;
|
query.andWhere('reaction.reaction = :type', { type });
|
||||||
}
|
}
|
||||||
|
|
||||||
const reactions = await this.noteReactionsRepository.find({
|
const reactions = await query.limit(ps.limit).getMany();
|
||||||
where: query,
|
|
||||||
take: ps.limit,
|
|
||||||
skip: ps.offset,
|
|
||||||
order: {
|
|
||||||
id: -1,
|
|
||||||
},
|
|
||||||
relations: ['user', 'note'],
|
|
||||||
});
|
|
||||||
|
|
||||||
return await Promise.all(reactions.map(reaction => this.noteReactionEntityService.pack(reaction, me)));
|
return await Promise.all(reactions.map(reaction => this.noteReactionEntityService.pack(reaction, me)));
|
||||||
});
|
});
|
||||||
|
|
Loading…
Reference in a new issue