@esfaenza/es-table 20.3.11 → 20.3.14

package/fesm2022/esfaenza-es-table.mjs

@@ -1263,8 +1263,15 @@ class ColumnMetadataHandler {
         let newHeaders = [];
         // Se devo finalizzare aggancio tutte le variabili di bind
         if (finalize) {
+            // Mappa ID -> header costruita una sola volta (evita il .filter() lineare per ogni OrderDirective)
+            let headersById = new Map();
+            for (let h = 0; h < this.est.TMPRowTHs.length; h++) {
+                let header = this.est.TMPRowTHs[h];
+                if (!headersById.has(header.ID))
+                    headersById.set(header.ID, header);
+            }
             this.est.OrderDirectives.forEach(o => {
-                let heaedrItem = this.est.TMPRowTHs.filter(t => t.ID == o)[0];
+                let heaedrItem = headersById.get(o);
                 if (heaedrItem)
                     newHeaders.push(heaedrItem);
             });
@@ -1273,8 +1280,16 @@ class ColumnMetadataHandler {
             for (let i = 0; i < this.est.RowContexts.length; i++) {
                 let context = this.est.TMPRowTDs[this.est.RowContexts[i]];
                 let newContext = [];
+                // Mappa ID -> signal costruita una sola volta per riga: il getter del signal viene invocato
+                // una volta per cella invece di una volta per ogni (colonna × OrderDirective)
+                let contextById = new Map();
+                for (let c = 0; c < context.length; c++) {
+                    let id = context[c]().ID;
+                    if (!contextById.has(id))
+                        contextById.set(id, context[c]);
+                }
                 this.est.OrderDirectives.forEach(o => {
-                    let contextItem = context.filter(t => t().ID == o)[0];
+                    let contextItem = contextById.get(o);
                     // potrebbe esser undefined nel caso ci fossero colonne sotto thIf
                     if (contextItem)
                         newContext.push(contextItem);
@@ -1669,7 +1684,6 @@ class HierarchyModeHandler {
             this.orderItemsInPlace(objs);
     }
     orderItemsInPlace(items) {
-        debugger;
         let ownKey = this.est.OwnKey();
         let parentKey = this.est.ParentKey();
         const childrenMap = new Map();
@@ -3529,9 +3543,15 @@ class Logger {
     constructor(debugMode) {
         this.debugMode = debugMode;
     }
+    /**
+     * Emette un log a console solo se la modalità debug è attiva.
+     *
+     * Il testo può essere passato come funzione (**lazy**): in tal caso viene valutato solo quando il debug è attivo,
+     * evitando di costruire stringhe costose (es. interpolazioni per-cella) quando i log sono disabilitati.
+     */
     log(text) {
         if (this.debugMode)
-            console.log("[@esfaenza/es-table @ " + (new Date()).toISOString() + "] " + text);
+            console.log("[@esfaenza/es-table @ " + (new Date()).toISOString() + "] " + (typeof text === 'function' ? text() : text));
     }
     static { this.ɵfac = i0.ɵɵngDeclareFactory({ minVersion: "12.0.0", version: "20.3.28", ngImport: i0, type: Logger, deps: [{ token: EST_DEBUG, optional: true }], target: i0.ɵɵFactoryTarget.Injectable }); }
     static { this.ɵprov = i0.ɵɵngDeclareInjectable({ minVersion: "12.0.0", version: "20.3.28", ngImport: i0, type: Logger, providedIn: "root" }); }
@@ -3861,7 +3881,7 @@ class EstRendererPipe {
             this.logger.log(`Renderer retrieved in a non performant way`);
             EsTd = params.src.find((item) => item.tdForProperty === propName);
         }
-        this.logger.log(`Renderer ${EsTd ? 'found' : 'NOT found'} for '${propName}${params.name ? '/' + params.name : ''}'${!EsTd ? '. Using fallback renderer' : ''}`);
+        this.logger.log(() => `Renderer ${EsTd ? 'found' : 'NOT found'} for '${propName}${params.name ? '/' + params.name : ''}'${!EsTd ? '. Using fallback renderer' : ''}`);
         var ret = EsTd ? EsTd.Template : params.fallback;
         return ret;
     }
@@ -3898,7 +3918,7 @@ class EstEditorPipe {
             this.logger.log(`Editor retrieved in a non performant way`);
             Editor = params.src.find((item) => item.editorForProperty === propName);
         }
-        this.logger.log(`Editor ${Editor ? 'found' : 'NOT found'} for '${propName}${params.name ? '/' + params.name : ''}'${!Editor ? '. Using fallback editor' : ''}`);
+        this.logger.log(() => `Editor ${Editor ? 'found' : 'NOT found'} for '${propName}${params.name ? '/' + params.name : ''}'${!Editor ? '. Using fallback editor' : ''}`);
         var ret = Editor ? Editor.Template : params.fallback;
         return ret;
     }
@@ -4957,69 +4977,77 @@ class EsTableComponent {
         let contextFixeds = {};
         for (let i = 0; i < tds.length; i++) {
             let td = tds[i];
+            // Valori costanti per colonna: calcolati una sola volta invece che per ogni riga
+            let key = td.td;
+            let th = this.thsCache[td.td];
+            if (!!th)
+                th._referenced = true;
+            // Skippare senza th (casistica tabella dinamica)
+            if (!(th && th.thIf && (td.tdIf || th.thGroup)))
+                continue;
+            let thCasted = th;
+            let colDef = columnsCache[td.td];
+            let isFixed = th.thFixed || th.thGroup || th.thPinned;
+            let multiProp = td._multi ? td.td.split('_')[0] : null;
+            let multiIndex = td._multi ? td.td.split('_')[1] : null;
+            let alignment = (td.tdAlignment || th.thAlignment || defaultAlignment).toLowerCase();
+            let format = td.tdFormat || th.thFormat;
+            let template = thCasted.thGroup && !this.ShowItemGroupsColumns ? null : td.Template;
+            let groupClass = thCasted.thGroupClass || '';
             for (let i = 0; i < bs.length; i++) {
                 let item = bs[i];
                 let ctx = item._hash;
-                let key = td.td;
-                let th = this.thsCache[td.td];
-                if (!!th)
-                    th._referenced = true;
-                // Skippare senza th (casistica tabella dinamica)
-                if (th && th.thIf && (td.tdIf || th.thGroup)) {
-                    let thCasted = th;
-                    let isFixed = th.thFixed || th.thGroup || th.thPinned;
-                    let itemToAdd = item._item ?
-                        {
-                            LeftBorder: thCasted._lbord,
-                            RightBorder: thCasted._rbord,
-                            ID: key,
-                            Pinned: columnsCache[td.td].pinned,
-                            PinnedWidth: thCasted.thPinnedWidth,
-                            GroupHeader: false,
-                            GroupAggregation: false,
-                            Visible: columnsCache[td.td].visible,
-                            Wraps: false,
-                            Class: td.tdClass,
-                            IconClass: td.tdIconClass,
-                            ConditionField: td.tdConditionField,
-                            ConditionValue: td.tdConditionValue,
-                            Title: td.tdTitle,
-                            Content: td.tdContent,
-                            Template: thCasted.thGroup && !this.ShowItemGroupsColumns ? null : td.Template,
-                            Multi: td._multi,
-                            MultiProp: td._multi ? td.td.split('_')[0] : null,
-                            MultiIndex: td._multi ? td.td.split('_')[1] : null,
-                            Alignment: (td.tdAlignment || th.thAlignment || defaultAlignment).toLowerCase(),
-                            Format: td.tdFormat || th.thFormat,
-                            PropertyName: th.thProperty,
-                            PropertyType: th.thType,
-                            PropertySource: th.thSource,
-                            EditorOptions: th.thEditorOptions,
-                            PropAccessor: td.tdPropertyAccessor,
-                            RoutePath: thCasted.thRoutePath,
-                            RouteParameterProperties: thCasted.thRouteParameterProperties
-                        } :
-                        {
-                            LeftBorder: thCasted._lbord,
-                            RightBorder: thCasted._rbord,
-                            ID: key,
-                            Pinned: columnsCache[td.td].pinned,
-                            PinnedWidth: thCasted.thPinnedWidth,
-                            GroupHeader: thCasted.thGroup,
-                            GroupAggregation: !!thCasted.thAggregation,
-                            Visible: columnsCache[td.td].visible,
-                            Wraps: false,
-                            Class: thCasted.thGroupClass || '',
-                            Content: this.rowGrouping_H.getGroupCellDisplay(item, key, th)
-                        };
-                    if (this.TMPRowTDs[ctx]) {
-                        this.TMPRowTDs[ctx].push(signal(itemToAdd));
-                    }
-                    else {
-                        this.RowContexts.push(ctx);
-                        this.TMPRowTDs[ctx] = [signal(itemToAdd)];
-                        contextFixeds[ctx] = isFixed ? 1 : 0;
-                    }
+                let itemToAdd = item._item ?
+                    {
+                        LeftBorder: thCasted._lbord,
+                        RightBorder: thCasted._rbord,
+                        ID: key,
+                        Pinned: colDef.pinned,
+                        PinnedWidth: thCasted.thPinnedWidth,
+                        GroupHeader: false,
+                        GroupAggregation: false,
+                        Visible: colDef.visible,
+                        Wraps: false,
+                        Class: td.tdClass,
+                        IconClass: td.tdIconClass,
+                        ConditionField: td.tdConditionField,
+                        ConditionValue: td.tdConditionValue,
+                        Title: td.tdTitle,
+                        Content: td.tdContent,
+                        Template: template,
+                        Multi: td._multi,
+                        MultiProp: multiProp,
+                        MultiIndex: multiIndex,
+                        Alignment: alignment,
+                        Format: format,
+                        PropertyName: th.thProperty,
+                        PropertyType: th.thType,
+                        PropertySource: th.thSource,
+                        EditorOptions: th.thEditorOptions,
+                        PropAccessor: td.tdPropertyAccessor,
+                        RoutePath: thCasted.thRoutePath,
+                        RouteParameterProperties: thCasted.thRouteParameterProperties
+                    } :
+                    {
+                        LeftBorder: thCasted._lbord,
+                        RightBorder: thCasted._rbord,
+                        ID: key,
+                        Pinned: colDef.pinned,
+                        PinnedWidth: thCasted.thPinnedWidth,
+                        GroupHeader: thCasted.thGroup,
+                        GroupAggregation: !!thCasted.thAggregation,
+                        Visible: colDef.visible,
+                        Wraps: false,
+                        Class: groupClass,
+                        Content: this.rowGrouping_H.getGroupCellDisplay(item, key, th)
+                    };
+                if (this.TMPRowTDs[ctx]) {
+                    this.TMPRowTDs[ctx].push(signal(itemToAdd));
+                }
+                else {
+                    this.RowContexts.push(ctx);
+                    this.TMPRowTDs[ctx] = [signal(itemToAdd)];
+                    contextFixeds[ctx] = isFixed ? 1 : 0;
                 }
             }
         }
@@ -5196,7 +5224,6 @@ class EsTableComponent {
      * @returns {boolean} **true** se l'operaizone è andata a buon fine, **false** altrimenti
      */
     handleClick(event, item) {
-        debugger;
         this.selection_H.clearSelectionStatusExcluding(undefined);
         let selecteditems = [];
         if (event.shiftKey && this.ShiftClick() && !this.SingleSelection()) {
@@ -5636,7 +5663,6 @@ class EsTableComponent {
     //* Piango ancora
     // Gestione Embedding
     showDialog(dlg, elementRef) {
-        debugger;
         dlg.show();
         this.retries = 0;
         if (this.emb.Embedded)