@topconsultnpm/sdkui-react 6.22.0-dev2.25 → 6.22.0-dev2.27

package/lib/components/features/documents/TMMultiMasterDetailDcmtsForm.js

@@ -122,7 +122,12 @@ const TMMultiMasterDetailDcmtsForm = (props) => {
     // Un nuovo riferimento a ogni render farebbe rifare a DevExtreme il setup di selezione/editing
     const gridSelection = useMemo(() => ({ mode: 'single', showCheckBoxesMode: 'none' }), []);
     const gridEditing = useMemo(() => ({
-        mode: 'cell', allowUpdating: true, allowAdding: false, allowDeleting: false, startEditAction: 'click', selectTextOnEditStart: true,
+        mode: 'cell',
+        allowUpdating: true,
+        allowAdding: false,
+        allowDeleting: false,
+        startEditAction: 'click',
+        selectTextOnEditStart: false,
     }), []);
     const handleSelectionChanged = useCallback((e) => {
         setSelectedRowKeys(e.selectedRowKeys ?? []);

package/lib/hooks/useMultiMasterDetailDcmts.js

@@ -61,6 +61,9 @@ export const useMultiMasterDetailDcmts = ({ dtd, masterDcmt, onClose }) => {
     const [masterInfoReloadToken, setMasterInfoReloadToken] = useState(0);
     // Righe del riepilogo
     const [archiveRows, setArchiveRows] = useState([]);
+    // La modifica in cella aggiorna la riga fuori dallo stato (vedi updateRowValues)
+    const archiveRowsRef = useRef(archiveRows);
+    archiveRowsRef.current = archiveRows;
     const rowIdRef = useRef(0);
     const [focusedRowKey, setFocusedRowKey] = useState(undefined);
     const [selectedRowKeys, setSelectedRowKeys] = useState([]);
@@ -354,36 +357,35 @@ export const useMultiMasterDetailDcmts = ({ dtd, masterDcmt, onClose }) => {
     };
     /**
      * Modifica diretta in griglia di uno o più metadati testuali o numerici (changes: mid -> valore).
-     * La riga e i suoi __values sono ricreati: il duplicato, che condivide __values per
-     * riferimento, resta invariato
+     * La riga è aggiornata sul posto e non con setArchiveRows: un nuovo riferimento all'array
+     * ricaricherebbe la griglia a ogni cella confermata, perdendo il clic sulla cella successiva.
+     * L'array __values è sostituito, non modificato: il duplicato, che lo condivide, resta invariato
      */
     const updateRowValues = useCallback((rowId, changes) => {
         const changedMids = Object.keys(changes);
         if (changedMids.length === 0)
             return;
+        const row = archiveRowsRef.current.find(r => r.id === rowId);
+        if (!row)
+            return;
         // I __values vanno a sistema come stringhe, anche quelli che in griglia sono numeri. La griglia
         // restituisce null sull'editor svuotato: a sistema il metadato va vuoto, non nullo
         const normalized = {};
         changedMids.forEach(mid => { normalized[mid] = changes[mid] === null || changes[mid] === undefined ? '' : String(changes[mid]); });
-        setArchiveRows(prev => prev.map(r => {
-            if (r.id !== rowId)
-                return r;
-            const values = r.__values.map(v => (v.mid !== undefined && normalized[String(v.mid)] !== undefined
-                ? { ...v, value: normalized[String(v.mid)] }
-                : v));
-            // Un metadato assente dai valori del form non arriverebbe all'archiviazione
-            changedMids.filter(mid => !values.some(v => String(v.mid) === mid)).forEach(mid => {
-                const added = new MetadataValueDescriptorEx();
-                added.tid = r.__tid;
-                added.mid = Number(mid);
-                added.value = normalized[mid];
-                values.push(added);
-            });
-            const updated = { ...r, __values: values };
-            // In cella il numero resta un numero, come lo vuole il suo editor
-            changedMids.forEach(mid => { updated[mid] = numericMidsRef.current.has(Number(mid)) ? toGridNumber(normalized[mid]) : normalized[mid]; });
-            return updated;
-        }));
+        const values = row.__values.map(v => (v.mid !== undefined && normalized[String(v.mid)] !== undefined
+            ? { ...v, value: normalized[String(v.mid)] }
+            : v));
+        // Un metadato assente dai valori del form non arriverebbe all'archiviazione
+        changedMids.filter(mid => !values.some(v => String(v.mid) === mid)).forEach(mid => {
+            const added = new MetadataValueDescriptorEx();
+            added.tid = row.__tid;
+            added.mid = Number(mid);
+            added.value = normalized[mid];
+            values.push(added);
+        });
+        row.__values = values;
+        // In cella il numero resta un numero, come lo vuole il suo editor
+        changedMids.forEach(mid => { row[mid] = numericMidsRef.current.has(Number(mid)) ? toGridNumber(normalized[mid]) : normalized[mid]; });
     }, []);
     // I __values sono condivisi per riferimento: la modifica sostituisce l'intero array, quindi non intacca il duplicato
     const duplicateRow = useCallback((row) => {

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@topconsultnpm/sdkui-react",
-    "version": "6.22.0-dev2.25",
+    "version": "6.22.0-dev2.27",
     "description": "",
     "scripts": {
         "test": "echo \"Error: no test specified\" && exit 1",