chore:code improvement

informatici · Nov 4, 2024 · 1eb095d · 1eb095d
1 parent 56e0afa
commit 1eb095d
Show file tree

Hide file tree

Showing 4 changed files with 17 additions and 13 deletions.
diff --git a/src/components/accessories/laboratory/examForm/ExamForm.tsx b/src/components/accessories/laboratory/examForm/ExamForm.tsx
@@ -415,7 +415,7 @@ const ExamForm: FC<ExamProps> = ({
                 <ExamRowTable
                   title={t("lab.resultstitle")}
                   headerData={rowTableHeaders}
-                  onBlur={onBlurCallbackForTableRow}
+                  onChange={onBlurCallbackForTableRow}
                   rows={examRows}
                   disabled={isLoading}
                 />

diff --git a/src/components/accessories/patientExams/ExamForm/ExamForm.tsx b/src/components/accessories/patientExams/ExamForm/ExamForm.tsx
@@ -288,7 +288,7 @@ const ExamForm: FC<ExamProps> = ({
                 <ExamRowTable
                   title={t("lab.resultstitle")}
                   headerData={rowTableHeaders}
-                  onBlur={onBlurCallbackForTableRow()}
+                  onChange={onBlurCallbackForTableRow()}
                   rows={examRows}
                   disabled={isLoading}
                 />

diff --git a/src/components/accessories/patientExams/examRowTable/ExamRowTable.tsx b/src/components/accessories/patientExams/examRowTable/ExamRowTable.tsx
@@ -12,14 +12,14 @@ import TableHead from "@mui/material/TableHead";
 import TableRow from "@mui/material/TableRow";
 import { useAppSelector } from "libraries/hooks/redux";
 import { debounce, isEmpty } from "lodash";
-import React, { FC, useCallback } from "react";
+import React, { ChangeEvent, FC, useCallback } from "react";
 import { IState } from "../../../../types";
 import "./styles.scss";
 import { IEditableTableProps } from "./types";
 
 const ExamRowTable: FC<IEditableTableProps> = ({
   rows,
-  onBlur,
+  onChange,
   headerData,
   title,
   disabled = false,
@@ -29,16 +29,22 @@ const ExamRowTable: FC<IEditableTableProps> = ({
       state.laboratories.getLabWithRowsByCode.data?.laboratoryRowList
   );
 
-  const handleOnBlur = (value: string, checked: boolean) => {
-    debounceUpdate(value, checked);
-  };
-
   // eslint-disable-next-line react-hooks/exhaustive-deps
   const debounceUpdate = useCallback(
-    debounce((value: string, checked: boolean) => onBlur(value, checked), 100),
+    debounce(
+      (value: string, checked: boolean) => onChange(value, checked),
+      100
+    ),
     []
   );
 
+  const handleChange = useCallback(
+    (value: string) => (_: ChangeEvent, checked: boolean) => {
+      debounceUpdate(value, checked);
+    },
+    [debounceUpdate]
+  );
+
   return (
     <Accordion disabled={disabled}>
       <AccordionSummary
@@ -75,9 +81,7 @@ const ExamRowTable: FC<IEditableTableProps> = ({
                   </TableCell>
                   <TableCell align="right" component="td" scope="row">
                     <Checkbox
-                      onChange={(e, value) => {
-                        handleOnBlur(row.label, value);
-                      }}
+                      onChange={handleChange(row.label)}
                       defaultChecked={
                         !isEmpty(
                           labToEditRows?.filter((e) => e === row.label) ?? []

diff --git a/src/components/accessories/patientExams/examRowTable/types.ts b/src/components/accessories/patientExams/examRowTable/types.ts
@@ -1,6 +1,6 @@
 export interface IEditableTableProps {
   rows: Array<{ label: string; value: string }>;
-  onBlur: (value: string, checked: boolean) => void;
+  onChange: (value: string, checked: boolean) => void;
   fieldValues?: string[];
   headerData: Array<{
     label: string;