From c9892936cddfb27036ee7ba95bd52ecfb4283d6c Mon Sep 17 00:00:00 2001 From: Dragory <2606411+Dragory@users.noreply.github.com> Date: Mon, 9 Nov 2020 03:34:01 +0200 Subject: [PATCH] Fix bug in getRecentActionCount where the fn could return a boolean or reset the count during counting --- backend/src/plugins/Spam/util/getRecentActionCount.ts | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/backend/src/plugins/Spam/util/getRecentActionCount.ts b/backend/src/plugins/Spam/util/getRecentActionCount.ts index fc0c898a..c7ae5c55 100644 --- a/backend/src/plugins/Spam/util/getRecentActionCount.ts +++ b/backend/src/plugins/Spam/util/getRecentActionCount.ts @@ -7,12 +7,12 @@ export function getRecentActionCount( userId: string, actionGroupId: string, since: number, -) { +): number { return pluginData.state.recentActions.reduce((count, action) => { if (action.timestamp < since) return count; if (action.type !== type) return count; if (action.actionGroupId !== actionGroupId) return count; - if (action.userId !== userId) return false; + if (action.userId !== userId) return count; return count + action.count; }, 0); }