Properly log distiction between refined/reversed

Author: TheKodeToad

backend/migrations/12-moderation-reversed-column.sql

@@ -0,0 +1 @@
+ALTER TABLE "moderation_cases" ADD "reversed" BOOLEAN NOT NULL DEFAULT FALSE;

backend/src/plugin/moderation/helper/format.ts

@@ -47,9 +47,12 @@ function caseSummaryBase(type: ModEventType, target: string): string {
 export async function formatCaseDescription(info: CaseInfo, bigTitle: boolean): Promise<string> {
 	let title = "Case #" + info.number;
 
-	if (info.shadowedBy !== null)
-		title = `~~${title}~~ (refined or reversed by #${info.shadowedBy})`;
-	else if (caseExpired(info))
+	if (info.shadowedBy !== null) {
+		if (info.reversed)
+			title = `~~${title}~~ (reversed by #${info.shadowedBy})`;
+		else
+			title = `~~${title}~~ (refined by #${info.shadowedBy})`;
+	} else if (caseExpired(info))
 		title = `~~${title}~~ (expired)`;
 
 	if (bigTitle)

backend/src/plugin/moderation/storage/cases.ts

@@ -10,6 +10,7 @@ export const CaseInfo = z.strictObject({
 	createdAt: z.date(),
 	expiresAt: z.date().nullable(),
 	shadowedBy: z.number().nullable(),
+	reversed: z.boolean(),
 
 	actorID: z.string(),
 	targetID: z.string(),
@@ -155,7 +156,7 @@ export async function createCase(guildID: string, event: ModEvent): Promise<numb
 
 		const reverseType = reverseModEventType(event.type);
 
-		if (!(reverseType === null || reverseType === ModEventType.Warn || reverseType === ModEventType.Unwarn)) {
+		if (reverseType !== null) {
 			await client.query(
 				`
 					WITH "shadowed" AS (
@@ -170,7 +171,9 @@ export async function createCase(guildID: string, event: ModEvent): Promise<numb
 						LIMIT 1
 					)
 					UPDATE "moderation_cases"
-					SET "shadowedBy" = $1
+					SET
+						"shadowedBy" = $1,
+						"reversed" = ("type" = $4)
 					FROM "shadowed"
 					WHERE
 						"moderation_cases"."guildID" = "shadowed"."guildID"