fix:eslint warings - unused imports, exhaustive deps, ...

src/components/Private.js

@@ -1,12 +1,11 @@
 import React from "react";
-import { Navigate, Outlet, useLocation } from "react-router-dom";
+import { Navigate, Outlet } from "react-router-dom";
 import { PATHS } from "../consts";
 import { isAuthenticated } from "../libraries/authUtils/isAuthenticated";
 import { useAuthentication } from "../libraries/authUtils/useAuthentication";
 
 export const Private = () => {
   useAuthentication();
-  const { pathname } = useLocation();
 
   return isAuthenticated() ? <Outlet /> : <Navigate to={PATHS.login} replace />;
 };

src/components/accessories/admin/diseases/Diseases.tsx

@@ -37,14 +37,11 @@ export const Diseases = () => {
     });
   };
 
-  const handleViewChange = useCallback(
-    (event: any, value: any) => {
-      if (!isEmpty(value)) {
-        setView(value);
-      }
-    },
-    [view]
-  );
+  const handleViewChange = useCallback((event: any, value: any) => {
+    if (!isEmpty(value)) {
+      setView(value);
+    }
+  }, []);
 
   const predicate = useCallback(
     (disease: DiseaseDTO) => {

src/components/accessories/admin/suppliers/editSupplier/EditSupplier.tsx

@@ -24,7 +24,7 @@ export const EditSupplier = () => {
     if (state?.supId !== Number(id)) {
       navigate(PATHS.admin_suppliers);
     }
-  }, [id, state]);
+  }, [id, navigate, state]);
 
   return (
     <SupplierForm

src/components/accessories/admin/suppliers/supplierForm/SupplierForm.tsx

@@ -30,8 +30,6 @@ import TextField from "../../../textField/TextField";
 import "./styles.scss";
 import { ISupplierFormProps } from "./types";
 
-const FORMAT = /^([0-9]{3})?[0-9]{2,10}$/;
-
 const SupplierForm: FC<ISupplierFormProps> = ({
   fields,
   onSubmit,

src/components/accessories/admin/types/TypesAdmin.tsx

@@ -1,6 +1,6 @@
 import { useAppSelector } from "libraries/hooks/redux";
 import { sortBy } from "lodash";
-import React, { useEffect, useState } from "react";
+import React, { useEffect, useMemo, useState } from "react";
 import { useTranslation } from "react-i18next";
 import { Outlet, useLocation, useNavigate } from "react-router";
 import { PATHS } from "../../../../consts";
@@ -16,7 +16,11 @@ const TypesAdmin = () => {
   const { t } = useTranslation();
   const location = useLocation();
   const navigate = useNavigate();
-  const defaultTypeOption: TypeOption = { label: "", value: "" };
+  const defaultTypeOption: TypeOption = useMemo(
+    () => ({ label: "", value: "" }),
+    []
+  );
+
   const [selectedOption, setSelectedOption] =
     useState<TypeOption>(defaultTypeOption);
   const mode = useAppSelector((state) => state.types.config.mode);
@@ -57,7 +61,7 @@ const TypesAdmin = () => {
         setSelectedOption(defaultTypeOption);
       }
     }
-  }, [location]);
+  }, [defaultTypeOption, location, typeOptions]);
 
   const handleTypeChange = (value: string) => {
     if (value?.length > 0) {

src/components/accessories/admission/PatientAdmission.tsx

@@ -83,7 +83,7 @@ const PatientAdmission: FC = () => {
     if (patientCode && creationMode) {
       dispatch(getLastOpd(parseInt(id!!)));
     }
-  }, [dispatch, patientCode, creationMode]);
+  }, [dispatch, patientCode, creationMode, id]);
 
   const fields = useFields(admissionToEdit, lastOpd?.disease);
 
@@ -157,7 +157,7 @@ const PatientAdmission: FC = () => {
       dispatch(getPatient(id!!));
       dispatch(getCurrentAdmission(parseInt(id!!)));
     }
-  }, [createStatus, updateStatus]);
+  }, [createStatus, dispatch, id, updateStatus]);
 
   const resetFormCallback = () => {
     setCreationMode(true);
@@ -173,7 +173,7 @@ const PatientAdmission: FC = () => {
     return () => {
       dispatch(getCurrentAdmissionReset());
     };
-  }, [dispatch]);
+  }, [dispatch, id]);
 
   const onEdit = (row: AdmissionDTO) => {
     setAdmissionToEdit(row);

src/components/accessories/appHeader/AppHeader.tsx

@@ -35,7 +35,7 @@ const AppHeader: FunctionComponent<IOwnProps> = ({ breadcrumbMap }) => {
   );
   useEffect(() => {
     dispatch(getHospital());
-  }, [dispatch, getHospital]);
+  }, [dispatch]);
 
   const hospital = useAppSelector(
     (state) => state.hospital.getHospital.data

src/components/accessories/appHeader/types.ts

@@ -1,5 +1,4 @@
 import { TUserCredentials } from "../../../state/main/types";
-import { TAPIResponseStatus } from "../../../state/types";
 
 export type TBreadcrumbMap = Record<string, string>;
 

src/components/accessories/autocompleteField/AutocompleteField.tsx

@@ -11,7 +11,6 @@ import _ from "lodash";
 import React, {
   ChangeEvent,
   FC,
-  Fragment,
   useCallback,
   useEffect,
   useState,
@@ -76,6 +75,7 @@ const AutocompleteField: FC<IProps> = ({
     }
   };
 
+  // eslint-disable-next-line react-hooks/exhaustive-deps
   const debounceUpdate = useCallback(
     debounce((value: any) => {
       setValue(value);
@@ -111,16 +111,6 @@ const AutocompleteField: FC<IProps> = ({
     return option.label;
   };
 
-  const isSelected = (option: DefaultOptionType, v: DefaultOptionType) => {
-    return option.value === v.value;
-  };
-
-  const rendOption = (props: any, option: DefaultOptionType | string) => {
-    return (
-      <Fragment>{typeof option === "string" ? option : option.label}</Fragment>
-    );
-  };
-
   const filter = createFilterOptions<DefaultOptionType>({
     limit: options_limit,
   });

src/components/accessories/billTable/BillTable.tsx

@@ -91,7 +91,7 @@ export const BillTable: FC<IBillTableProps> = ({ fields }) => {
         dispatch(searchBills(filter));
         break;
     }
-  }, [filter]);
+  }, [dispatch, filter]);
 
   const { setFieldValue, handleBlur } = formik;
 

src/components/accessories/billrecords/BillRecords.tsx

@@ -67,7 +67,7 @@ const BillRecords = () => {
     dispatch(deleteBillReset());
     dispatch(payBillReset());
     dispatch(closeBillReset());
-  }, []);
+  }, [dispatch]);
 
   useEffect(() => {
     if (patient && patient.code) {
@@ -275,7 +275,7 @@ const BillRecords = () => {
         }}
         handleSecondaryButtonClick={() => ({})}
       />
-      <iframe id="ifmcontentstoprint"></iframe>
+      <iframe id="ifmcontentstoprint" title="ifmcontentstoprint"></iframe>
     </div>
   );
 };

src/components/accessories/billsRecap/BillsRecap.tsx

@@ -12,14 +12,7 @@ import {
 } from "chart.js";
 import { useAppDispatch, useAppSelector } from "libraries/hooks/redux";
 import moment from "moment";
-import React, {
-  FC,
-  useCallback,
-  useEffect,
-  useMemo,
-  useRef,
-  useState,
-} from "react";
+import React, { FC, useCallback, useEffect, useMemo, useState } from "react";
 import { Bar, Doughnut, Line, Pie } from "react-chartjs-2";
 import { useTranslation } from "react-i18next";
 import { getBillsByYear, searchBills } from "../../../state/bills";
@@ -98,6 +91,7 @@ export const BillsRecap: FC = () => {
 
   useEffect(() => {
     dispatch(searchBills(filter as TFilterValues));
+    // eslint-disable-next-line react-hooks/exhaustive-deps
   }, [dispatch]);
 
   useEffect(() => {
@@ -188,7 +182,7 @@ export const BillsRecap: FC = () => {
         ],
       };
     },
-    [summaryCurrentYear]
+    [summaryCurrentYear, t]
   );
 
   const paymentsVariationsData = useMemo(() => {
@@ -206,7 +200,7 @@ export const BillsRecap: FC = () => {
         {
           label: t("bill.payments"),
           data:
-            labels.length != 0
+            labels.length !== 0
               ? labels.map(
                   (item) => summaryCurrentYear.paymentsByMonthsOfYear[item] ?? 0
                 )
@@ -217,7 +211,7 @@ export const BillsRecap: FC = () => {
         {
           label: t("bill.debts"),
           data:
-            labels.length != 0
+            labels.length !== 0
               ? labels.map(
                   (item) => summaryCurrentYear.debtsByMonthsOfYear[item] ?? 0
                 )
@@ -227,6 +221,7 @@ export const BillsRecap: FC = () => {
         },
       ],
     };
+    // eslint-disable-next-line react-hooks/exhaustive-deps
   }, [summaryCurrentYear]);
 
   const getOptionsFromYears = (years: number[]) => {
@@ -237,7 +232,6 @@ export const BillsRecap: FC = () => {
       };
     });
   };
-  const infoBoxRef = useRef<HTMLDivElement>(null);
 
   return (
     <div className="bills__recap">

src/components/accessories/currentAdmission/currentAdmissionForm/CurrentAdmissionForm.tsx

@@ -13,17 +13,15 @@ import {
   AdmissionTypeDTO,
   DiseaseDTO,
   DiseaseTypeDTO,
-  WardDTO
+  WardDTO,
 } from "../../../../generated";
 import {
   differenceInDays,
   formatAllFieldValues,
-  getFromFields
+  getFromFields,
 } from "../../../../libraries/formDataHandling/functions";
 import { updateAdmissionReset } from "../../../../state/admissions";
-import {
-  getPatient
-} from "../../../../state/patients";
+import { getPatient } from "../../../../state/patients";
 import { IState } from "../../../../types";
 import AutocompleteField from "../../autocompleteField/AutocompleteField";
 import Button from "../../button/Button";
@@ -130,9 +128,9 @@ export const CurrentAdmissionForm: FunctionComponent<IOwnProps> = ({
       }
       onDiscard();
     }
-  }, [dispatch, activityTransitionState]);
+  }, [dispatch, activityTransitionState, patient, onDiscard]);
 
-  const { setFieldValue, resetForm, handleBlur } = formik;
+  const { setFieldValue, handleBlur } = formik;
 
   const isValid = (fieldName: string): boolean => {
     return has(formik.touched, fieldName) && has(formik.errors, fieldName);
@@ -154,7 +152,7 @@ export const CurrentAdmissionForm: FunctionComponent<IOwnProps> = ({
       ).toString();
       setFieldValue("bedDays", days);
     },
-    [setFieldValue]
+    [formik, setFieldValue]
   );
 
   const onBlurCallback = useCallback(

src/components/accessories/currentAdmission/currentAdmissionForm/consts.ts

@@ -1,4 +1,4 @@
-import { AdmissionDTO, PatientDTO } from "../../../../generated";
+import { AdmissionDTO } from "../../../../generated";
 import { parseDate } from "../../../../libraries/formDataHandling/functions";
 import { TFields } from "../../../../libraries/formDataHandling/types";
 import { TCurrentAdmissionFieldName } from "./types";

src/components/accessories/dashboard/admissions/useData.ts

@@ -1,15 +1,6 @@
-import { useMemo, useState } from "react";
-import { useTranslation } from "react-i18next";
 import { useAppSelector } from "libraries/hooks/redux";
-import {
-  AdmissionDTO,
-  WardDTO,
-  AdmissionTypeDTO,
-  AgeTypeDTO,
-} from "../../../../generated";
+import { useTranslation } from "react-i18next";
 import { colorGen } from "../../../../libraries/uiUtils/colorGenerator";
-import { TAPIResponseStatus } from "../../../../state/types";
-import { IState } from "../../../../types";
 
 export const useData = () => {
   const { t } = useTranslation();

src/components/accessories/dashboard/card/DashboardCard.tsx

@@ -38,16 +38,20 @@ export const DashboardCard: React.FC<TDashboardCardProps> = ({
     if (sizeChangeHandler && cardBodyRef.current) {
       resizeObserver.observe(cardBodyRef.current);
     }
+    // eslint-disable-next-line react-hooks/exhaustive-deps
   }, [cardBodyRef]);
 
   // Remove the observer on component unmount
   useEffect(() => {
+    const currentRef = cardBodyRef.current;
+
     return () => {
-      if (sizeChangeHandler && cardBodyRef.current) {
-        resizeObserver.unobserve(cardBodyRef.current);
+      if (sizeChangeHandler && currentRef) {
+        resizeObserver.unobserve(currentRef);
         resizeObserver.disconnect();
       }
     };
+    // eslint-disable-next-line react-hooks/exhaustive-deps
   }, []);
 
   const getTitle = () => {

src/components/accessories/dashboard/dashboardContent/filter/DashboardFilter.tsx

@@ -27,6 +27,7 @@ export const DashboardFilter: FC<IOwnProps> = ({ onPeriodChange }) => {
 
   useEffect(() => {
     onPeriodChange([range[0], range[1]]);
+    // eslint-disable-next-line react-hooks/exhaustive-deps
   }, [range]);
 
   const handleViewChange = useCallback(
@@ -36,6 +37,7 @@ export const DashboardFilter: FC<IOwnProps> = ({ onPeriodChange }) => {
         setSelection("current");
       }
     },
+    // eslint-disable-next-line react-hooks/exhaustive-deps
     [view]
   );
 
@@ -45,7 +47,7 @@ export const DashboardFilter: FC<IOwnProps> = ({ onPeriodChange }) => {
         setSelection(value as TPeriodType);
       }
     },
-    [selection]
+    [setSelection]
   );
 
   const onIconClickHandler = useCallback(
@@ -66,6 +68,7 @@ export const DashboardFilter: FC<IOwnProps> = ({ onPeriodChange }) => {
       setSelection("custom");
       setOpen(false);
     },
+    // eslint-disable-next-line react-hooks/exhaustive-deps
     [dateRange]
   );
 
@@ -78,7 +81,7 @@ export const DashboardFilter: FC<IOwnProps> = ({ onPeriodChange }) => {
       setSelection("custom");
       setOpen(false);
     },
-    [dateRange]
+    [setDateRange, setSelection, setView]
   );
 
   return (