Tetardtek/Sakuin
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: prevent double XP on completion — guard already-completed status

Browse Source
This commit is contained in:
Tetardtek
2026-04-05 07:19:44 +02:00
parent 108f021bd8
commit 1059d4ae17
1 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
backend/src/list/list.service.ts
View File

@@ -97,9 +97,10 @@ export class ListService {
}); });
if (!uw) throw new NotFoundException('Entry not found'); if (!uw) throw new NotFoundException('Entry not found');
const wasAlreadyCompleted = uw.status === ListStatus.COMPLETED;
uw.status = status; uw.status = status;
if (status === ListStatus.COMPLETED) { if (status === ListStatus.COMPLETED && !wasAlreadyCompleted) {
uw.completedAt = new Date(); uw.completedAt = uw.completedAt || new Date();
await this.userService.addXp(userId, XP_COMPLETE); await this.userService.addXp(userId, XP_COMPLETE);
} }