From caa793d036d24c29820ff23a345a6cd4114d57bd Mon Sep 17 00:00:00 2001
From: Davenludd <david.ludde01@gmail.com>
Date: Wed, 20 Mar 2024 15:32:14 +0100
Subject: [PATCH] Fix week validation in NewWeeklyReport component for safari
and firefox
---
frontend/src/Components/NewWeeklyReport.tsx | 11 +++--------
1 file changed, 3 insertions(+), 8 deletions(-)
diff --git a/frontend/src/Components/NewWeeklyReport.tsx b/frontend/src/Components/NewWeeklyReport.tsx
index 2593782..f1ce3f8 100644
--- a/frontend/src/Components/NewWeeklyReport.tsx
+++ b/frontend/src/Components/NewWeeklyReport.tsx
@@ -45,8 +45,8 @@ export default function NewWeeklyReport(): JSX.Element {
<div className="border-4 border-black bg-white flex flex-col justify-start min-h-[65vh] h-fit w-[50vw] rounded-3xl overflow-scroll space-y-[2vh] p-[30px] items-center">
<form
onSubmit={(e) => {
- if (week === 0) {
- alert("Please enter a week number");
+ if (week === 0 || week > 53 || week < 1) {
+ alert("Please enter a valid week number");
e.preventDefault();
return;
}
@@ -81,12 +81,7 @@ export default function NewWeeklyReport(): JSX.Element {
placeholder="Week (Numbers Only)"
onChange={(e) => {
const weekNumber = parseInt(e.target.value);
- if (isNaN(weekNumber) || weekNumber < 1 || weekNumber > 53) {
- setWeek(0);
- return;
- } else {
- setWeek(weekNumber);
- }
+ setWeek(weekNumber);
}}
onKeyDown={(event) => {
const keyValue = event.key;