From 6c2213b48876603c2ba5b800ca132bea661d93e1 Mon Sep 17 00:00:00 2001
From: Mattias <mattias.rasmusson.1032@student.lu.se>
Date: Tue, 2 Apr 2024 15:43:12 +0200
Subject: [PATCH] Update handleUpdateWeeklyReport function and fix input
 validation

---
 frontend/src/Components/EditWeeklyReport.tsx | 54 ++++++++++++++++----
 1 file changed, 43 insertions(+), 11 deletions(-)

diff --git a/frontend/src/Components/EditWeeklyReport.tsx b/frontend/src/Components/EditWeeklyReport.tsx
index 384359e..7f907af 100644
--- a/frontend/src/Components/EditWeeklyReport.tsx
+++ b/frontend/src/Components/EditWeeklyReport.tsx
@@ -1,5 +1,5 @@
 import { useState, useEffect } from "react";
-import { WeeklyReport, NewWeeklyReport } from "../Types/goTypes";
+import { WeeklyReport, UpdateWeeklyReport } from "../Types/goTypes";
 import { api } from "../API/API";
 import { useNavigate, useParams } from "react-router-dom";
 import Button from "./Button";
@@ -60,8 +60,9 @@ export default function GetWeeklyReport(): JSX.Element {
     void fetchWeeklyReport();
   }, [projectName, fetchedWeek, token]);
 
-  const handleNewWeeklyReport = async (): Promise<void> => {
-    const newWeeklyReport: NewWeeklyReport = {
+  const handleUpdateWeeklyReport = async (): Promise<void> => {
+    const updateWeeklyReport: UpdateWeeklyReport = {
+      userName: "user2",
       projectName: projectName ?? "",
       week,
       developmentTime,
@@ -72,7 +73,7 @@ export default function GetWeeklyReport(): JSX.Element {
       testingTime,
     };
 
-    await api.submitWeeklyReport(newWeeklyReport, token);
+    await api.updateWeeklyReport(updateWeeklyReport, token);
   };
 
   const navigate = useNavigate();
@@ -89,7 +90,8 @@ export default function GetWeeklyReport(): JSX.Element {
               return;
             }
             e.preventDefault();
-            void handleNewWeeklyReport();
+            void handleUpdateWeeklyReport();
+            alert("Changes submitted");
             navigate(-1);
           }}
         >
@@ -128,7 +130,12 @@ export default function GetWeeklyReport(): JSX.Element {
                       }}
                       onKeyDown={(event) => {
                         const keyValue = event.key;
-                        if (!/\d/.test(keyValue) && keyValue !== "Backspace")
+                        if (
+                          !/\d/.test(keyValue) &&
+                          keyValue !== "Backspace" &&
+                          keyValue !== "ArrowLeft" &&
+                          keyValue !== "ArrowRight"
+                        )
                           event.preventDefault();
                       }}
                     />
@@ -152,7 +159,12 @@ export default function GetWeeklyReport(): JSX.Element {
                       }}
                       onKeyDown={(event) => {
                         const keyValue = event.key;
-                        if (!/\d/.test(keyValue) && keyValue !== "Backspace")
+                        if (
+                          !/\d/.test(keyValue) &&
+                          keyValue !== "Backspace" &&
+                          keyValue !== "ArrowLeft" &&
+                          keyValue !== "ArrowRight"
+                        )
                           event.preventDefault();
                       }}
                     />
@@ -176,7 +188,12 @@ export default function GetWeeklyReport(): JSX.Element {
                       }}
                       onKeyDown={(event) => {
                         const keyValue = event.key;
-                        if (!/\d/.test(keyValue) && keyValue !== "Backspace")
+                        if (
+                          !/\d/.test(keyValue) &&
+                          keyValue !== "Backspace" &&
+                          keyValue !== "ArrowLeft" &&
+                          keyValue !== "ArrowRight"
+                        )
                           event.preventDefault();
                       }}
                     />
@@ -200,7 +217,12 @@ export default function GetWeeklyReport(): JSX.Element {
                       }}
                       onKeyDown={(event) => {
                         const keyValue = event.key;
-                        if (!/\d/.test(keyValue) && keyValue !== "Backspace")
+                        if (
+                          !/\d/.test(keyValue) &&
+                          keyValue !== "Backspace" &&
+                          keyValue !== "ArrowLeft" &&
+                          keyValue !== "ArrowRight"
+                        )
                           event.preventDefault();
                       }}
                     />
@@ -224,7 +246,12 @@ export default function GetWeeklyReport(): JSX.Element {
                       }}
                       onKeyDown={(event) => {
                         const keyValue = event.key;
-                        if (!/\d/.test(keyValue) && keyValue !== "Backspace")
+                        if (
+                          !/\d/.test(keyValue) &&
+                          keyValue !== "Backspace" &&
+                          keyValue !== "ArrowLeft" &&
+                          keyValue !== "ArrowRight"
+                        )
                           event.preventDefault();
                       }}
                     />
@@ -248,7 +275,12 @@ export default function GetWeeklyReport(): JSX.Element {
                       }}
                       onKeyDown={(event) => {
                         const keyValue = event.key;
-                        if (!/\d/.test(keyValue) && keyValue !== "Backspace")
+                        if (
+                          !/\d/.test(keyValue) &&
+                          keyValue !== "Backspace" &&
+                          keyValue !== "ArrowLeft" &&
+                          keyValue !== "ArrowRight"
+                        )
                           event.preventDefault();
                       }}
                     />