Feat: 複数ノートの通報機能

This commit is contained in:
mattyatea 2023-12-10 09:17:24 +09:00
parent 4c135a5ca1
commit 6849d510ac
10 changed files with 197 additions and 69 deletions

1
locales/index.d.ts vendored
View File

@ -682,6 +682,7 @@ export interface Locale {
"forwardReport": string; "forwardReport": string;
"forwardReportIsAnonymous": string; "forwardReportIsAnonymous": string;
"send": string; "send": string;
"reportedNote": string;
"abuseMarkAsResolved": string; "abuseMarkAsResolved": string;
"openInNewTab": string; "openInNewTab": string;
"openInSideView": string; "openInSideView": string;

View File

@ -679,6 +679,7 @@ reporterOrigin: "通報元"
forwardReport: "リモートサーバーに通報を転送する" forwardReport: "リモートサーバーに通報を転送する"
forwardReportIsAnonymous: "リモートサーバーからはあなたの情報は見れず、匿名のシステムアカウントとして表示されます。" forwardReportIsAnonymous: "リモートサーバーからはあなたの情報は見れず、匿名のシステムアカウントとして表示されます。"
send: "送信" send: "送信"
reportedNote: "通報されたノート"
abuseMarkAsResolved: "対応済みにする" abuseMarkAsResolved: "対応済みにする"
openInNewTab: "新しいタブで開く" openInNewTab: "新しいタブで開く"
openInSideView: "サイドビューで開く" openInSideView: "サイドビューで開く"

View File

@ -0,0 +1,11 @@
export class Abusenoteselect1702149469508 {
name = 'Abusenoteselect1702149469508'
async up(queryRunner) {
await queryRunner.query(`ALTER TABLE "abuse_user_report" ADD "notes" jsonb NOT NULL DEFAULT '[]'`);
}
async down(queryRunner) {
await queryRunner.query(`ALTER TABLE "abuse_user_report" DROP COLUMN "notes"`);
}
}

View File

@ -157,6 +157,7 @@ export interface AdminEventTypes {
targetUserId: MiUser['id'], targetUserId: MiUser['id'],
reporterId: MiUser['id'], reporterId: MiUser['id'],
comment: string; comment: string;
notes: any[];
}; };
} }
//#endregion //#endregion

View File

@ -33,6 +33,7 @@ export class AbuseUserReportEntityService {
id: report.id, id: report.id,
createdAt: this.idService.parse(report.id).date.toISOString(), createdAt: this.idService.parse(report.id).date.toISOString(),
comment: report.comment, comment: report.comment,
notes: report.notes,
resolved: report.resolved, resolved: report.resolved,
reporterId: report.reporterId, reporterId: report.reporterId,
targetUserId: report.targetUserId, targetUserId: report.targetUserId,

View File

@ -60,6 +60,11 @@ export class MiAbuseUserReport {
}) })
public comment: string; public comment: string;
@Column('jsonb', {
default: [],
})
public notes: any[];
//#region Denormalized fields //#region Denormalized fields
@Index() @Index()
@Column('varchar', { @Column('varchar', {

View File

@ -3,14 +3,17 @@
* SPDX-License-Identifier: AGPL-3.0-only * SPDX-License-Identifier: AGPL-3.0-only
*/ */
import { setImmediate } from 'node:timers/promises';
import sanitizeHtml from 'sanitize-html'; import sanitizeHtml from 'sanitize-html';
import { Inject, Injectable } from '@nestjs/common'; import { Inject, Injectable } from '@nestjs/common';
import type { AbuseUserReportsRepository } from '@/models/_.js'; import { In } from 'typeorm';
import type { AbuseUserReportsRepository, NotesRepository } from '@/models/_.js';
import { IdService } from '@/core/IdService.js'; import { IdService } from '@/core/IdService.js';
import { Endpoint } from '@/server/api/endpoint-base.js'; import { Endpoint } from '@/server/api/endpoint-base.js';
import { GlobalEventService } from '@/core/GlobalEventService.js'; import { GlobalEventService } from '@/core/GlobalEventService.js';
import { MetaService } from '@/core/MetaService.js'; import { MetaService } from '@/core/MetaService.js';
import { EmailService } from '@/core/EmailService.js'; import { EmailService } from '@/core/EmailService.js';
import { NoteEntityService } from '@/core/entities/NoteEntityService.js';
import { DI } from '@/di-symbols.js'; import { DI } from '@/di-symbols.js';
import { GetterService } from '@/server/api/GetterService.js'; import { GetterService } from '@/server/api/GetterService.js';
import { RoleService } from '@/core/RoleService.js'; import { RoleService } from '@/core/RoleService.js';
@ -21,7 +24,7 @@ export const meta = {
requireCredential: true, requireCredential: true,
description: 'File a report.', description: 'User a report.',
errors: { errors: {
noSuchUser: { noSuchUser: {
@ -49,6 +52,7 @@ export const paramDef = {
properties: { properties: {
userId: { type: 'string', format: 'misskey:id' }, userId: { type: 'string', format: 'misskey:id' },
comment: { type: 'string', minLength: 1, maxLength: 2048 }, comment: { type: 'string', minLength: 1, maxLength: 2048 },
noteIds: { type: 'array', items: { type: 'string', format: 'misskey:id' } },
}, },
required: ['userId', 'comment'], required: ['userId', 'comment'],
} as const; } as const;
@ -59,11 +63,15 @@ export default class extends Endpoint<typeof meta, typeof paramDef> { // eslint-
@Inject(DI.abuseUserReportsRepository) @Inject(DI.abuseUserReportsRepository)
private abuseUserReportsRepository: AbuseUserReportsRepository, private abuseUserReportsRepository: AbuseUserReportsRepository,
@Inject(DI.notesRepository)
private notesRepository: NotesRepository,
private idService: IdService, private idService: IdService,
private metaService: MetaService, private metaService: MetaService,
private emailService: EmailService, private emailService: EmailService,
private getterService: GetterService, private getterService: GetterService,
private roleService: RoleService, private roleService: RoleService,
private noteEntityService: NoteEntityService,
private globalEventService: GlobalEventService, private globalEventService: GlobalEventService,
) { ) {
super(meta, paramDef, async (ps, me) => { super(meta, paramDef, async (ps, me) => {
@ -81,6 +89,10 @@ export default class extends Endpoint<typeof meta, typeof paramDef> { // eslint-
throw new ApiError(meta.errors.cannotReportAdmin); throw new ApiError(meta.errors.cannotReportAdmin);
} }
const notes = ps.noteIds ? await this.notesRepository.find({
where: { id: In(ps.noteIds) },
}) : [];
const filteredNotes = notes.filter(note => note.userId === user.id);
const report = await this.abuseUserReportsRepository.insert({ const report = await this.abuseUserReportsRepository.insert({
id: this.idService.gen(), id: this.idService.gen(),
targetUserId: user.id, targetUserId: user.id,
@ -88,6 +100,7 @@ export default class extends Endpoint<typeof meta, typeof paramDef> { // eslint-
reporterId: me.id, reporterId: me.id,
reporterHost: null, reporterHost: null,
comment: ps.comment, comment: ps.comment,
notes: ps.noteIds ? await this.noteEntityService.packMany(filteredNotes) : [],
}).then(x => this.abuseUserReportsRepository.findOneByOrFail(x.identifiers[0])); }).then(x => this.abuseUserReportsRepository.findOneByOrFail(x.identifiers[0]));
// Publish event to moderators // Publish event to moderators
@ -100,9 +113,9 @@ export default class extends Endpoint<typeof meta, typeof paramDef> { // eslint-
targetUserId: report.targetUserId, targetUserId: report.targetUserId,
reporterId: report.reporterId, reporterId: report.reporterId,
comment: report.comment, comment: report.comment,
notes: report.notes,
}); });
} }
const meta = await this.metaService.fetch(); const meta = await this.metaService.fetch();
if (meta.email) { if (meta.email) {
this.emailService.sendEmail(meta.email, 'New abuse report', this.emailService.sendEmail(meta.email, 'New abuse report',

View File

@ -4,13 +4,13 @@ SPDX-License-Identifier: AGPL-3.0-only
--> -->
<template> <template>
<div class="bcekxzvu _margin _panel"> <div :class="$style.root">
<div class="target"> <div :class="$style.target">
<MkA v-user-preview="report.targetUserId" class="info" :to="`/admin/user/${report.targetUserId}`"> <MkA v-user-preview="report.targetUserId" :class="$style.info" :to="`/admin/user/${report.targetUserId}`">
<MkAvatar class="avatar" :user="report.targetUser" indicator/> <MkAvatar :class="$style.avatar" :user="report.targetUser" indicator/>
<div class="names"> <div :class="$style.name">
<MkUserName class="name" :user="report.targetUser"/> <MkUserName :class="$style.names" :user="report.targetUser"/>
<MkAcct class="acct" :user="report.targetUser" style="display: block;"/> <MkAcct :class="$style.names" :user="report.targetUser" style="display: block;"/>
</div> </div>
</MkA> </MkA>
<MkKeyValue> <MkKeyValue>
@ -18,9 +18,15 @@ SPDX-License-Identifier: AGPL-3.0-only
<template #value>{{ dateString(report.targetUser.createdAt) }} (<MkTime :time="report.targetUser.createdAt"/>)</template> <template #value>{{ dateString(report.targetUser.createdAt) }} (<MkTime :time="report.targetUser.createdAt"/>)</template>
</MkKeyValue> </MkKeyValue>
</div> </div>
<div class="detail"> <div :class="$style.detail">
<div> <div>
<Mfm :text="report.comment"/> <Mfm :text="report.comment"/>
<MkFolder v-if="report.notes.length !== 0" :class="$style.notes">
<template #label>{{ i18n.ts.reportedNote }}</template>
<div v-for="note in report.notes" :class="$style.notes">
<MkNoteSimple :note="note"/>
</div>
</MkFolder>
</div> </div>
<hr/> <hr/>
<div>{{ i18n.ts.reporter }}: <MkA :to="`/admin/user/${report.reporter.id}`" class="_link">@{{ report.reporter.username }}</MkA></div> <div>{{ i18n.ts.reporter }}: <MkA :to="`/admin/user/${report.reporter.id}`" class="_link">@{{ report.reporter.username }}</MkA></div>
@ -42,15 +48,28 @@ SPDX-License-Identifier: AGPL-3.0-only
<script lang="ts" setup> <script lang="ts" setup>
import { ref } from 'vue'; import { ref } from 'vue';
import * as Misskey from 'misskey-js';
import MkButton from '@/components/MkButton.vue'; import MkButton from '@/components/MkButton.vue';
import MkSwitch from '@/components/MkSwitch.vue'; import MkSwitch from '@/components/MkSwitch.vue';
import MkKeyValue from '@/components/MkKeyValue.vue'; import MkKeyValue from '@/components/MkKeyValue.vue';
import * as os from '@/os.js'; import * as os from '@/os.js';
import { i18n } from '@/i18n.js'; import { i18n } from '@/i18n.js';
import { dateString } from '@/filters/date.js'; import { dateString } from '@/filters/date.js';
import MkFolder from '@/components/MkFolder.vue';
import MkNoteSimple from '@/components/MkNoteSimple.vue';
const props = defineProps<{ const props = defineProps<{
report: any; report: {
id: string;
createdAt:string;
targetUserId:Misskey.entities.User['id'];
targetUser:Misskey.entities.User & {createdAt:string;};
reporter:Misskey.entities.User;
assignee:Misskey.entities.User['id'];
comment:string;
notes:Misskey.entities.Note['id'][];
forwarded:boolean;
resolved:boolean;
};
}>(); }>();
const emit = defineEmits<{ const emit = defineEmits<{
@ -69,18 +88,28 @@ function resolve() {
} }
</script> </script>
<style lang="scss" scoped> <style lang="scss" module>
.bcekxzvu { .root {
display: flex; display: flex;
margin: var(--margin) 0;
background: var(--panel);
border-radius: var(--radius);
overflow: clip;
}
> .target { .notes {
margin: var(--margin) 0;
padding: 0;
}
.target {
width: 35%; width: 35%;
box-sizing: border-box; box-sizing: border-box;
text-align: left; text-align: left;
padding: 24px; padding: 24px;
border-right: solid 1px var(--divider); border-right: solid 1px var(--divider);
}
> .info { .info {
display: flex; display: flex;
box-sizing: border-box; box-sizing: border-box;
align-items: center; align-items: center;
@ -89,27 +118,26 @@ function resolve() {
--c: rgb(255 196 0 / 15%); --c: rgb(255 196 0 / 15%);
background-image: linear-gradient(45deg, var(--c) 16.67%, transparent 16.67%, transparent 50%, var(--c) 50%, var(--c) 66.67%, transparent 66.67%, transparent 100%); background-image: linear-gradient(45deg, var(--c) 16.67%, transparent 16.67%, transparent 50%, var(--c) 50%, var(--c) 66.67%, transparent 66.67%, transparent 100%);
background-size: 16px 16px; background-size: 16px 16px;
}
> .avatar { .avatar {
width: 42px; width: 42px;
height: 42px; height: 42px;
} }
> .names { .names {
margin-left: 0.3em; margin-left: 0.3em;
padding: 0 8px; padding: 0 8px;
flex: 1; flex: 1;
}
> .name { .name {
font-weight: bold; font-weight: bold;
} }
}
}
}
> .detail { .detail {
flex: 1; flex: 1;
padding: 24px; padding: 24px;
}
} }
</style> </style>

View File

@ -4,7 +4,7 @@ SPDX-License-Identifier: AGPL-3.0-only
--> -->
<template> <template>
<MkWindow ref="uiWindow" :initialWidth="400" :initialHeight="500" :canResize="true" @closed="emit('closed')"> <MkWindow ref="uiWindow" :initialWidth="400" :initialHeight="500" :canResize="true" style="overflow-x: clip;" @closed="emit('closed')">
<template #header> <template #header>
<i class="ti ti-exclamation-circle" style="margin-right: 0.5em;"></i> <i class="ti ti-exclamation-circle" style="margin-right: 0.5em;"></i>
<I18n :src="i18n.ts.reportAbuseOf" tag="span"> <I18n :src="i18n.ts.reportAbuseOf" tag="span">
@ -13,19 +13,45 @@ SPDX-License-Identifier: AGPL-3.0-only
</template> </template>
</I18n> </I18n>
</template> </template>
<Transition
mode="out-in"
:enterActiveClass="$style.transition_x_enterActive"
:leaveActiveClass="$style.transition_x_leaveActive"
:enterFromClass="$style.transition_x_enterFrom"
:leaveToClass="$style.transition_x_leaveTo"
>
<template v-if="page === 0">
<MkSpacer :marginMin="20" :marginMax="28">
<div class="_gaps_m" :class="$style.root">
<MkPagination v-slot="{items}" :key="user.id" :pagination="Pagination">
<div v-for="item in items" :key="item.id" :class="$style.note">
<MkSwitch v-model="item.isAbuseReport" @update:modelValue="pushAbuseReportNote($event,item.id)"></MkSwitch>
<MkAvatar :user="item.user" preview/>
<MkNoteSimple :note="item"/>
</div>
</MkPagination>
<div class="_buttonsCenter">
<MkButton primary rounded gradate @click="page++">{{ i18n.ts.continue }} <i class="ti ti-arrow-right"></i></MkButton>
</div>
</div>
</MkSpacer>
</template>
<template v-else-if="page === 1">
<MkSpacer :marginMin="20" :marginMax="28"> <MkSpacer :marginMin="20" :marginMax="28">
<div class="_gaps_m" :class="$style.root"> <div class="_gaps_m" :class="$style.root">
<div class="">
<MkTextarea v-model="comment"> <MkTextarea v-model="comment">
<template #label>{{ i18n.ts.details }}</template> <template #label>{{ i18n.ts.details }}</template>
<template #caption>{{ i18n.ts.fillAbuseReportDescription }}</template> <template #caption>{{ i18n.ts.fillAbuseReportDescription }}</template>
</MkTextarea> </MkTextarea>
</div> <div class="_buttonsCenter">
<div class=""> <MkButton @click="page--"><i class="ti ti-arrow-left"></i> {{ i18n.ts.goBack }}</MkButton>
<MkButton primary full :disabled="comment.length === 0" @click="send">{{ i18n.ts.send }}</MkButton> <MkButton primary :disabled="comment.length === 0" @click="send">{{ i18n.ts.send }}</MkButton>
</div> </div>
</div> </div>
</MkSpacer> </MkSpacer>
</template>
</Transition>
</MkWindow> </MkWindow>
</template> </template>
@ -37,23 +63,46 @@ import MkTextarea from '@/components/MkTextarea.vue';
import MkButton from '@/components/MkButton.vue'; import MkButton from '@/components/MkButton.vue';
import * as os from '@/os.js'; import * as os from '@/os.js';
import { i18n } from '@/i18n.js'; import { i18n } from '@/i18n.js';
import MkPagination from '@/components/MkPagination.vue';
import MkNoteSimple from '@/components/MkNoteSimple.vue';
import MkSwitch from '@/components/MkSwitch.vue';
const props = defineProps<{ const props = defineProps<{
user: Misskey.entities.User; user: Misskey.entities.User;
initialComment?: string; initialComment?: string;
initialNoteId?: Misskey.entities.Note['id'];
}>(); }>();
const Pagination = {
endpoint: 'users/notes' as const,
limit: 10,
params: {
userId: props.user.id,
},
};
const emit = defineEmits<{ const emit = defineEmits<{
(ev: 'closed'): void; (ev: 'closed'): void;
}>(); }>();
const abuseNotesId = ref(props.initialNoteId ? [props.initialNoteId] : []);
const page = ref(0);
const uiWindow = shallowRef<InstanceType<typeof MkWindow>>(); const uiWindow = shallowRef<InstanceType<typeof MkWindow>>();
const comment = ref(props.initialComment ?? ''); const comment = ref(props.initialComment ?? '');
function pushAbuseReportNote(v, id) {
if (v) {
abuseNotesId.value.push(id);
} else {
abuseNotesId.value = abuseNotesId.value.filter(noteId => noteId !== id);
}
}
function send() { function send() {
os.apiWithDialog('users/report-abuse', { os.apiWithDialog('users/report-abuse', {
userId: props.user.id, userId: props.user.id,
comment: comment.value, comment: comment.value,
noteIds: abuseNotesId.value,
}, undefined).then(res => { }, undefined).then(res => {
os.alert({ os.alert({
type: 'success', type: 'success',
@ -69,4 +118,22 @@ function send() {
.root { .root {
--root-margin: 16px; --root-margin: 16px;
} }
.transition_x_enterActive,
.transition_x_leaveActive {
transition: opacity 0.3s cubic-bezier(0,0,.35,1), transform 0.3s cubic-bezier(0,0,.35,1);
}
.transition_x_enterFrom {
opacity: 0;
transform: translateX(50px);
}
.transition_x_leaveTo {
opacity: 0;
transform: translateX(-50px);
}
.note{
display: flex;
margin: var(--margin) 0;
align-items: center;
}
</style> </style>

View File

@ -102,7 +102,7 @@ export function getAbuseNoteMenu(note: misskey.entities.Note, text: string): Men
const u = note.url ?? note.uri ?? `${url}/notes/${note.id}`; const u = note.url ?? note.uri ?? `${url}/notes/${note.id}`;
os.popup(defineAsyncComponent(() => import('@/components/MkAbuseReportWindow.vue')), { os.popup(defineAsyncComponent(() => import('@/components/MkAbuseReportWindow.vue')), {
user: note.user, user: note.user,
initialComment: `Note: ${u}\n-----\n`, initialNoteId: note.id,
}, {}, 'closed'); }, {}, 'closed');
}, },
}; };