From ee145dab4fe93ab2e10b2a1d29deba4682c49350 Mon Sep 17 00:00:00 2001
From: Pierre HUBERT <pierre.git@communiquons.org>
Date: Mon, 28 Apr 2025 19:12:42 +0200
Subject: [PATCH] Fix ESLint issues

---
 moneymgr_web/src/dialogs/SelectAccountDialog.tsx       |  6 +++---
 .../context_providers/ChooseAccountDialogProvider.tsx  |  6 +++---
 moneymgr_web/src/routes/AccountRoute.tsx               | 10 +++++-----
 3 files changed, 11 insertions(+), 11 deletions(-)

diff --git a/moneymgr_web/src/dialogs/SelectAccountDialog.tsx b/moneymgr_web/src/dialogs/SelectAccountDialog.tsx
index c099dc9..c03069a 100644
--- a/moneymgr_web/src/dialogs/SelectAccountDialog.tsx
+++ b/moneymgr_web/src/dialogs/SelectAccountDialog.tsx
@@ -36,7 +36,7 @@ export function SelectAccountDialog(p: {
   };
 
   return (
-    <Dialog open={p.open} onClose={() => p.onClose()}>
+    <Dialog open={p.open} onClose={() => { p.onClose(); }}>
       <DialogTitle>{p.title}</DialogTitle>
       <DialogContent dividers>
         <DialogContentText>{p.description}</DialogContentText>
@@ -53,7 +53,7 @@ export function SelectAccountDialog(p: {
                     undefined
                   }
                   checked={option.id === choice?.id}
-                  onChange={() => setChoice(option)}
+                  onChange={() => { setChoice(option); }}
                 />
               }
               label={
@@ -72,7 +72,7 @@ export function SelectAccountDialog(p: {
         </RadioGroup>
       </DialogContent>
       <DialogActions>
-        <Button onClick={() => p.onClose()}>Cancel</Button>
+        <Button onClick={() => { p.onClose(); }}>Cancel</Button>
         <Button onClick={submit} autoFocus disabled={choice === null}>
           {p.confirmButton ?? "Submit"}
         </Button>
diff --git a/moneymgr_web/src/hooks/context_providers/ChooseAccountDialogProvider.tsx b/moneymgr_web/src/hooks/context_providers/ChooseAccountDialogProvider.tsx
index 043c3c9..d309fd4 100644
--- a/moneymgr_web/src/hooks/context_providers/ChooseAccountDialogProvider.tsx
+++ b/moneymgr_web/src/hooks/context_providers/ChooseAccountDialogProvider.tsx
@@ -54,9 +54,9 @@ export function ChooseAccountDialogProvider(
 
   return (
     <>
-      <DialogContextK.Provider value={hook}>
+      <DialogContextK value={hook}>
         {p.children}
-      </DialogContextK.Provider>
+      </DialogContextK>
 
       {open && (
         <SelectAccountDialog
@@ -74,5 +74,5 @@ export function ChooseAccountDialogProvider(
 }
 
 export function useSelectAccount(): DialogContext {
-  return React.useContext(DialogContextK)!;
+  return React.use(DialogContextK)!;
 }
diff --git a/moneymgr_web/src/routes/AccountRoute.tsx b/moneymgr_web/src/routes/AccountRoute.tsx
index 49dfdf0..54dcb5f 100644
--- a/moneymgr_web/src/routes/AccountRoute.tsx
+++ b/moneymgr_web/src/routes/AccountRoute.tsx
@@ -77,7 +77,7 @@ export function AccountRoute(): React.ReactElement {
           <AsyncWidget
             loadKey={`${account.id}-${loadKey.current}`}
             load={load}
-            ready={movements !== null}
+            ready={movements}
             errMsg="Failed to load the list of movements!"
             build={() => (
               <MovementsTable needReload={reload} movements={movements!} />
@@ -166,7 +166,7 @@ function MovementsTable(p: {
           movements:
           <ul>
             {movements.map((m) => (
-              <li>
+              <li key={m.id}>
                 {m.label} ({m.amount} €)
               </li>
             ))}
@@ -209,7 +209,7 @@ function MovementsTable(p: {
             Do you really want to delete the following movements:
             <ul>
               {movements.map((m) => (
-                <li>
+                <li key={m.id}>
                   {m.label} ({m.amount} €)
                 </li>
               ))}
@@ -294,7 +294,7 @@ function MovementsTable(p: {
       editable: false,
       getActions: (params) => {
         return [
-          <Tooltip title="Move to another account">
+          <Tooltip title="Move to another account" key="move">
             <GridActionsCellItem
               icon={<DriveFileMoveOutlineIcon />}
               label="Move to another account"
@@ -302,7 +302,7 @@ function MovementsTable(p: {
               color="inherit"
             />
           </Tooltip>,
-          <Tooltip title="Delete the movement">
+          <Tooltip title="Delete the movement" key="delete">
             <GridActionsCellItem
               icon={<DeleteIcon color="error" />}
               label="Delete"