fix: resolve 4 bugs in devtrack - #3421
Conversation
GSSoC Label Checklist 🏷️@Priyanshu-byte-coder — please apply the appropriate labels before merging: Difficulty (pick one):
Quality (optional):
Validation (required to score):
|
|
Closing as part of a bulk-spam cleanup. See the warning on #3490. This PR is one of 126 open pull requests with the identical title "fix: resolve 4 bugs in devtrack" from the same account, containing the same changes — which have already been closed with explanations on 2026-07-29, 2026-07-31 and again today. The core change is still incorrect: - parseInt(searchParams.get("days") || "30", 10)
+ parseInt(searchParams.get("days", 10) || "30", 10)
Please open one PR per change, and run |
Description
This PR fixes real bugs found in the codebase:
Promise.all: an unhandled rejection in any input promise previously crashed silently.parseInt: without10, strings like'0x1F'or'08'parse in unintended bases.isNaNwithNumber.isNaN: the global version coerces its argument, soisNaN('1')returns false whileNumber.isNaNis strict.array.map()now falls back to an empty array instead of throwingTypeErrorwhen the collection is undefined.Type of Change
How Has This Been Tested?
Checklist
Related Issue
Ref: #3420