From 734f8c26c780132a40540db73a79f682cc36d836 Mon Sep 17 00:00:00 2001
From: michaelschal <35957947-michaelschal@users.noreply.replit.com>
Date: Mon, 8 Sep 2025 20:25:36 +0000
Subject: [PATCH] Update invoice editing to correctly save financial amounts

Refactor client-side logic for editing invoice amounts to use local state, ensuring accurate data persistence and preventing conflicts with form handling.

Replit-Commit-Author: Agent
Replit-Commit-Session-Id: 9fcf4b21-eb0c-4e53-a567-e2ce4a6ad869
Replit-Commit-Checkpoint-Type: full_checkpoint
Replit-Commit-Screenshot-Url: https://storage.googleapis.com/screenshot-production-us-central1/1957c339-2757-4d1f-8e92-e9f71a1ce58e/9fcf4b21-eb0c-4e53-a567-e2ce4a6ad869/Ii96ETr
---
 client/src/pages/Avoirs.tsx | 39 +++++++++++++++++++++----------------
 1 file changed, 22 insertions(+), 17 deletions(-)

diff --git a/client/src/pages/Avoirs.tsx b/client/src/pages/Avoirs.tsx
index 8f36832..05b99a6 100644
--- a/client/src/pages/Avoirs.tsx
+++ b/client/src/pages/Avoirs.tsx
@@ -128,6 +128,9 @@ export default function Avoirs() {
   // États pour le système de vérification de facture
   const [avoirVerificationResults, setAvoirVerificationResults] = useState<Record<number, any>>({});
   const [verifyingAvoirs, setVerifyingAvoirs] = useState<Set<number>>(new Set());
+
+  // ✅ État local pour le champ montant d'édition pour éviter les conflits avec React Hook Form
+  const [editAmountValue, setEditAmountValue] = useState<string>("");
   // Utiliser le contexte global du magasin
   const { selectedStoreId } = useStore();
   const { toast } = useToast();
@@ -383,12 +386,21 @@ export default function Avoirs() {
   // Handle edit form submission
   const onEditSubmit = (data: AvoirFormData) => {
     if (selectedAvoir) {
+      // ✅ Utiliser la valeur de l'état local pour le montant
+      const finalAmount = editAmountValue === "" ? undefined : parseFloat(editAmountValue);
+      const finalData = {
+        ...data,
+        amount: finalAmount
+      };
+      
       console.log('💰 Editing avoir - Form data sent:', {
         originalCommercial: selectedAvoir.commercialProcessed,
-        formCommercial: data.commercialProcessed,
-        fullData: data
+        formCommercial: finalData.commercialProcessed,
+        editAmountValue,
+        finalAmount,
+        fullData: finalData
       });
-      editAvoirMutation.mutate({ id: selectedAvoir.id, data });
+      editAvoirMutation.mutate({ id: selectedAvoir.id, data: finalData });
     }
   };
 
@@ -397,11 +409,14 @@ export default function Avoirs() {
     if (!avoir || !avoir.id) return;
     
     setSelectedAvoir(avoir);
+    // ✅ Initialiser l'état local du montant
+    setEditAmountValue(avoir.amount !== null && avoir.amount !== undefined ? avoir.amount.toString() : "");
+    
     editForm.reset({
       supplierId: avoir.supplierId || 0,
       groupId: avoir.groupId || 0,
       invoiceReference: avoir.invoiceReference || "",
-      amount: avoir.amount ?? undefined, // ✅ Convertir null en undefined pour permettre la suppression
+      amount: avoir.amount ?? undefined,
       comment: avoir.comment || "",
       commercialProcessed: avoir.commercialProcessed || false,
       status: avoir.status || "En attente de demande",
@@ -1361,21 +1376,11 @@ export default function Avoirs() {
                         type="number" 
                         step="0.01" 
                         placeholder="Montant en euros (optionnel)" 
-                        value={field.value ?? ""}
+                        value={editAmountValue}
                         onChange={(e) => {
-                          const value = e.target.value;
-                          if (value === "") {
-                            field.onChange(undefined);
-                          } else {
-                            const num = parseFloat(value);
-                            if (!isNaN(num)) {
-                              field.onChange(num);
-                            }
-                          }
+                          // ✅ Utiliser seulement l'état local, complètement séparé de React Hook Form
+                          setEditAmountValue(e.target.value);
                         }}
-                        onBlur={field.onBlur}
-                        name={field.name}
-                        ref={field.ref}
                       />
                     </FormControl>
                     <FormMessage />