@topconsultnpm/sdkui-react 6.22.0-dev2.21 → 6.22.0-dev2.22

package/lib/components/features/documents/TMMultiMasterDetailDcmtsUtils.d.ts

@@ -46,13 +46,14 @@ export interface ArchiveRow {
  * stringhe, quindi il `value` generico di DevExtreme si restringe
  */
 export type ArchiveRowCell = Omit<DataGridTypes.ColumnCellTemplateData<ArchiveRow, number>, 'value'> & {
-    readonly value?: string;
+    readonly value?: string | number;
 };
 /**
  * Modifiche di una riga in arrivo dalla griglia (mid -> nuovo valore): in modifica diretta ci sono
- * solo i metadati testuali (vedi isInlineEditableMetadata) e l'editor svuotato restituisce null
+ * i metadati testuali e numerici (vedi isInlineEditableMetadata), quindi stringhe e numeri,
+ * e l'editor svuotato restituisce null
  */
-export type ArchiveRowChanges = Record<string, string | null | undefined>;
+export type ArchiveRowChanges = Record<string, string | number | null | undefined>;
 /** Aggiornamento di una riga della griglia, con le sole modifiche possibili in linea */
 export type ArchiveRowUpdatingEvent = Omit<DataGridTypes.RowUpdatingEvent<ArchiveRow, number>, 'newData'> & {
     newData: ArchiveRowChanges;
@@ -70,10 +71,15 @@ export declare const canArchive: (dtd?: DcmtTypeDescriptor) => boolean;
  */
 export declare const INLINE_EDITABLE_CELL_CLASS = "tm-cell-inline-editable";
 /**
- * Modifica diretta in griglia: solo il testo libero. Liste dati, calcolati, numeratori e speciali
- * hanno editor e vincoli propri, date e numeri una formattazione dedicata: restano al form
+ * Modifica diretta in griglia: testo libero e numeri. Liste dati, calcolati, numeratori e speciali
+ * hanno editor e vincoli propri, le date una formattazione dedicata: restano al form
  */
 export declare const isInlineEditableMetadata: (md: MetadataDescriptor) => boolean;
+/**
+ * Valore di una cella numerica: l'editor della griglia vuole un numero, non la stringa che
+ * la riga porta in `__values` per l'archiviazione. Vuoto o non numerico: cella vuota
+ */
+export declare const toGridNumber: (value: unknown) => number | undefined;
 /** Colonne della griglia di riepilogo dai metadati (utente) del tipo documento */
 export declare const buildColumnsFromDtd: (dtd: DcmtTypeDescriptor | undefined, renderDataListCell: (value: string | Date | number | undefined, dataListID: number, viewMode: DataListViewModes) => React.ReactElement) => Array<IColumnProps>;
 export declare const getDetailDtdAsync: (detailTID: number | undefined) => Promise<DcmtTypeDescriptor | undefined>;

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

@@ -27,12 +27,22 @@ export const canArchive = (dtd) => dtd?.perm?.canArchive === AccessLevelsEx.Yes
  */
 export const INLINE_EDITABLE_CELL_CLASS = 'tm-cell-inline-editable';
 /**
- * Modifica diretta in griglia: solo il testo libero. Liste dati, calcolati, numeratori e speciali
- * hanno editor e vincoli propri, date e numeri una formattazione dedicata: restano al form
+ * Modifica diretta in griglia: testo libero e numeri. Liste dati, calcolati, numeratori e speciali
+ * hanno editor e vincoli propri, le date una formattazione dedicata: restano al form
  */
-export const isInlineEditableMetadata = (md) => md.dataType === MetadataDataTypes.Varchar
+export const isInlineEditableMetadata = (md) => (md.dataType === MetadataDataTypes.Varchar || md.dataType === MetadataDataTypes.Number)
     && (md.dataDomain === undefined || md.dataDomain === MetadataDataDomains.None)
     && md.isSystem !== 1 && md.isSystemDerived !== 1;
+/**
+ * Valore di una cella numerica: l'editor della griglia vuole un numero, non la stringa che
+ * la riga porta in `__values` per l'archiviazione. Vuoto o non numerico: cella vuota
+ */
+export const toGridNumber = (value) => {
+    if (value === undefined || value === null || value === '')
+        return undefined;
+    const parsed = typeof value === 'number' ? value : Number(value);
+    return Number.isFinite(parsed) ? parsed : undefined;
+};
 /** Colonne della griglia di riepilogo dai metadati (utente) del tipo documento */
 export const buildColumnsFromDtd = (dtd, renderDataListCell) => getUserMetadataSorted(dtd).map(md => {
     const dataListID = md.dataListID ?? 0;
@@ -41,21 +51,25 @@ export const buildColumnsFromDtd = (dtd, renderDataListCell) => getUserMetadataS
     const dataType = md.dataType;
     const format = md.format?.format;
     const formatCulture = md.format?.formatCulture;
-    // Date e numeri usano la formattazione scalare; le liste dati il render comune
-    const needsScalarFormat = !isDataList && (dataType === MetadataDataTypes.DateTime || dataType === MetadataDataTypes.Number);
     const isInlineEditable = isInlineEditableMetadata(md);
+    const isNumber = dataType === MetadataDataTypes.Number;
+    // Date e numeri usano la formattazione scalare; le liste dati il render comune. La cella
+    // modificabile mostra sempre l'editor, quindi non ha un proprio render
+    const needsScalarFormat = !isDataList && !isInlineEditable && (dataType === MetadataDataTypes.DateTime || isNumber);
     return {
         dataField: String(md.id),
         caption: (SDK_Globals.useLocalizedName ? md.nameLoc : md.name) ?? md.name ?? '',
         // La colonna modificabile dichiara il tipo: senza valori DevExtreme non saprebbe che editor usare
         ...(isInlineEditable
             ? {
-                dataType: 'string',
+                // I numeri arrivano in cella già come numeri (vedi buildArchiveRow): l'editor è la casella numerica
+                dataType: isNumber ? 'number' : 'string',
                 allowEditing: true,
                 showEditorAlways: true,
                 cssClass: INLINE_EDITABLE_CELL_CLASS,
-                // Lunghezza e obbligatorietà del metadato valgono anche in griglia
-                editorOptions: md.length ? { maxLength: md.length } : undefined,
+                // Lunghezza e obbligatorietà del metadato valgono anche in griglia; sui numeri
+                // i limiti restano all'API, in cella basta che sia un numero
+                editorOptions: !isNumber && md.length ? { maxLength: md.length } : undefined,
                 validationRules: String(md.isRequired) === '1' ? [{ type: 'required', message: SDKUI_Localizator.RequiredField }] : undefined,
             }
             : { allowEditing: false }),

package/lib/hooks/useMultiMasterDetailDcmts.js

@@ -8,7 +8,7 @@ import TMSpinner from '../components/base/TMSpinner';
 import ShowAlert from '../components/base/TMAlert';
 import { TMResultManager } from '../components/forms/TMResultDialog';
 import { useDataListItem } from './useDataListItem';
-import { buildColumnsFromDtd, canArchive, canUpdateMasterCallback, getDetailDtdAsync } from '../components/features/documents/TMMultiMasterDetailDcmtsUtils';
+import { buildColumnsFromDtd, canArchive, canUpdateMasterCallback, getDetailDtdAsync, toGridNumber } from '../components/features/documents/TMMultiMasterDetailDcmtsUtils';
 /** Fasi del flusso di archiviazione multipla dei dettagli */
 export var MultiDetailArchivePhase;
 (function (MultiDetailArchivePhase) {
@@ -44,6 +44,8 @@ export const useMultiMasterDetailDcmts = ({ dtd, masterDcmt, onClose }) => {
     // Struttura del tipo documento selezionato
     const [detailColumns, setDetailColumns] = useState([]);
     const [detailSupportsFile, setDetailSupportsFile] = useState(false);
+    // Mid numerici del tipo di dettaglio: in griglia le loro celle portano il numero (vedi buildArchiveRow)
+    const numericMidsRef = useRef(new Set());
     // Dettagli già archiviati del tipo selezionato, collegati al master
     const [archivedDetailResults, setArchivedDetailResults] = useState([]);
     const [isLoadingArchivedDetails, setIsLoadingArchivedDetails] = useState(false);
@@ -146,6 +148,7 @@ export const useMultiMasterDetailDcmts = ({ dtd, masterDcmt, onClose }) => {
             if (!selectedRelation?.detailTID) {
                 setDetailColumns([]);
                 setDetailSupportsFile(false);
+                numericMidsRef.current = new Set();
                 return;
             }
             try {
@@ -153,16 +156,21 @@ export const useMultiMasterDetailDcmts = ({ dtd, masterDcmt, onClose }) => {
                 setDetailSupportsFile(detailDtd?.archiveConstraint !== ArchiveConstraints.OnlyMetadata);
                 // Liste dati referenziate dai metadati, servono al render delle celle
                 const dataListIDs = new Set();
+                const numericMids = new Set();
                 (detailDtd?.metadata ?? []).forEach(md => {
                     if (md.dataDomain === MetadataDataDomains.DataList && md.dataListID)
                         dataListIDs.add(md.dataListID);
+                    if (md.dataType === MetadataDataTypes.Number && md.id !== undefined)
+                        numericMids.add(md.id);
                 });
+                numericMidsRef.current = numericMids;
                 await loadDataListsAsync(dataListIDs);
                 setDetailColumns(buildColumnsFromDtd(detailDtd, renderDataListCell));
             }
             catch {
                 setDetailColumns([]);
                 setDetailSupportsFile(false);
+                numericMidsRef.current = new Set();
             }
         };
         loadDetailStructureAsync();
@@ -315,11 +323,13 @@ export const useMultiMasterDetailDcmts = ({ dtd, masterDcmt, onClose }) => {
             },
         });
     }, []);
+    // I valori del form sono stringhe: le celle numeriche vogliono il numero, richiesto dal loro editor
     const buildArchiveRow = useCallback((id, tid, values, file) => {
         const row = { id, __tid: tid, __values: values, __file: file };
         for (const v of values) {
-            if (v.mid !== undefined)
-                row[String(v.mid)] = v.value ?? '';
+            if (v.mid === undefined)
+                continue;
+            row[String(v.mid)] = numericMidsRef.current.has(v.mid) ? toGridNumber(v.value) : (v.value ?? '');
         }
         return row;
     }, []);
@@ -338,7 +348,7 @@ export const useMultiMasterDetailDcmts = ({ dtd, masterDcmt, onClose }) => {
         closeAddForm();
     };
     /**
-     * Modifica diretta in griglia di uno o più metadati testuali (changes: mid -> valore).
+     * 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
      */
@@ -346,9 +356,10 @@ export const useMultiMasterDetailDcmts = ({ dtd, masterDcmt, onClose }) => {
         const changedMids = Object.keys(changes);
         if (changedMids.length === 0)
             return;
-        // La griglia restituisce null sull'editor svuotato: a sistema il metadato va vuoto, non nullo
+        // 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] ?? ''; });
+        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;
@@ -364,7 +375,8 @@ export const useMultiMasterDetailDcmts = ({ dtd, masterDcmt, onClose }) => {
                 values.push(added);
             });
             const updated = { ...r, __values: values };
-            changedMids.forEach(mid => { updated[mid] = normalized[mid]; });
+            // 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;
         }));
     }, []);

package/package.json

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