From 4ddb5640cf983a03078452771191a21c5ad02d40 Mon Sep 17 00:00:00 2001 From: Blodhgarm Date: Thu, 30 Apr 2026 19:00:14 -0500 Subject: [PATCH] fix: reports not reporting all possible reports (#5933) * Fix Reports page only giving 695 reports due to Labrinth Issue Basically, for some reason, Labrinth returns 5 less depending on the amount requested and the offset position, leading to the end of all reports, even if it's not correct. Signed-off-by: Blodhgarm * remove constant condition --------- Signed-off-by: Blodhgarm Co-authored-by: Prospector <6166773+Prospector@users.noreply.github.com> --- apps/frontend/src/pages/moderation/reports/index.vue | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/apps/frontend/src/pages/moderation/reports/index.vue b/apps/frontend/src/pages/moderation/reports/index.vue index 4ced93ca4..d4c09c21b 100644 --- a/apps/frontend/src/pages/moderation/reports/index.vue +++ b/apps/frontend/src/pages/moderation/reports/index.vue @@ -103,12 +103,16 @@ const { data: allReports } = await useLazyAsyncData('new-moderation-reports', as const enrichmentPromises: Promise[] = [] let reports: Report[] - do { + let hasMoreReports = true + while (hasMoreReports) { reports = (await useBaseFetch(`report?count=${REPORT_ENDPOINT_COUNT}&offset=${currentOffset}`, { apiVersion: 3, })) as Report[] - if (reports.length === 0) break + hasMoreReports = reports.length > 0 + if (!hasMoreReports) { + break + } const enrichmentPromise = enrichReportBatch(reports) enrichmentPromises.push(enrichmentPromise) @@ -119,7 +123,7 @@ const { data: allReports } = await useLazyAsyncData('new-moderation-reports', as const completed = await Promise.all(enrichmentPromises.splice(0, 2)) allReports.push(...completed.flat()) } - } while (reports.length === REPORT_ENDPOINT_COUNT) + } const remainingBatches = await Promise.all(enrichmentPromises) allReports.push(...remainingBatches.flat())