From fc51ac17b19ca241da2659fca20945ff32a43ec3 Mon Sep 17 00:00:00 2001 From: Johann150 Date: Mon, 16 May 2022 09:14:42 +0200 Subject: [PATCH] fix: remove unnecessary null check Because `findOneByOrFail` is used above, the null check is unnecessary. --- .../server/api/endpoints/admin/resolve-abuse-user-report.ts | 4 ---- 1 file changed, 4 deletions(-) diff --git a/packages/backend/src/server/api/endpoints/admin/resolve-abuse-user-report.ts b/packages/backend/src/server/api/endpoints/admin/resolve-abuse-user-report.ts index 23940ba05..04ef3454e 100644 --- a/packages/backend/src/server/api/endpoints/admin/resolve-abuse-user-report.ts +++ b/packages/backend/src/server/api/endpoints/admin/resolve-abuse-user-report.ts @@ -25,10 +25,6 @@ export const paramDef = { export default define(meta, paramDef, async (ps, me) => { const report = await AbuseUserReports.findOneByOrFail({ id: ps.reportId }); - if (report == null) { - throw new Error('report not found'); - } - if (ps.forward && report.targetUserHost != null) { const actor = await getInstanceActor(); const targetUser = await Users.findOneByOrFail({ id: report.targetUserId });