boha_admin/app
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 26 Wiki Activity

fix: handle plain month number in attendance route, not just YYYY-MM

Browse Source 
AttendanceAdmin sends ?year=YYYY&month=M (separate params), but the
route handler assumed month was always in "YYYY-MM" format. When
query.month was just "4", the split produced NaN for month and 4 for
year, causing the service to skip the month filter entirely.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
This commit is contained in:
BOHA
2026-04-29 16:50:20 +02:00
parent 3bd0d055d9
commit c0c3439abb
2 changed files with 9 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
package.json
View File

@@ -1,6 +1,6 @@
{ {
"name": "app-ts", "name": "app-ts",
"version": "1.6.0", "version": "1.6.1",
"description": "", "description": "",
"main": "dist/server.js", "main": "dist/server.js",
"scripts": { "scripts": {

6
src/routes/admin/attendance.ts
View File

@@ -221,10 +221,16 @@ export default async function attendanceRoutes(
isAdmin, isAdmin,
authUserId: authData.userId, authUserId: authData.userId,
month: query.month month: query.month
? String(query.month).includes("-")
? Number(String(query.month).split("-")[1]) ? Number(String(query.month).split("-")[1])
: Number(query.month)
: undefined, : undefined,
year: query.month year: query.month
? String(query.month).includes("-")
? Number(String(query.month).split("-")[0]) ? Number(String(query.month).split("-")[0])
: query.year
? Number(query.year)
: undefined
: query.year : query.year
? Number(query.year) ? Number(query.year)
: undefined, : undefined,