@one-grid-core/angular 0.5.1 → 0.5.2

package/fesm2022/one-grid-core-angular.mjs

@@ -4722,6 +4722,24 @@ class OneGridBodyComponent {
          * MAX_BUFFER_PX keeps roughly twenty rows of overscan either side instead.
          */
         this.MAX_BUFFER_PX = 600;
+        /**
+         * Params for a component cell renderer.
+         *
+         * The cell params, plus whatever the column declared in
+         * `cellRendererParams` — a field that exists precisely so a renderer can be
+         * configured per column, and that nothing had ever forwarded to it. Column
+         * config goes first so the grid's own fields win a name collision.
+         */
+        /**
+         * Cached per cell, keyed on the identities that mean "this cell changed":
+         * its cellStates object (replaced on every edit and state refresh) and its
+         * data object. Building a fresh object per call — which a template method
+         * does per change-detection pass — made the renderer directive see "params
+         * changed" every pass and re-run the component's oneInit each time. A
+         * renderer whose oneInit touched grid state (a custom tree expander is the
+         * classic case) then re-triggered change detection and looped forever.
+         */
+        this.rendererParamsCache = new Map();
         // ---------------------------------------------------------------------------
         // Cell navigation
         //
@@ -4791,6 +4809,8 @@ class OneGridBodyComponent {
          */
         this.subscriptions.add(this.eventService.addEventListener('columnChanged', (payload) => {
             this.columnDefs = [...payload.columns];
+            // Params carry the columnDef, so a column change invalidates them all.
+            this.rendererParamsCache.clear();
             this.cellRenderer = this.columnDefs.some(col => col.cellRendererComponent);
             this.rowWidth = this.columnDefs.reduce((sum, col) => sum + resolveColumnWidth(col), 0);
             // A column change can alter the width left for flex columns (a hide, a
@@ -4806,6 +4826,9 @@ class OneGridBodyComponent {
             // The full post-filter count, reported before the page slice is taken,
             // so the host's paging totals see everything.
             this.rowCountChanged.emit(allNodes?.length ?? 0);
+            // New rows, new renderer params — and removed rows leave no stale
+            // cache entries behind.
+            this.rendererParamsCache.clear();
             const nodes = this.api.pageNodes(allNodes);
             setTimeout(() => {
                 this.viewport?.checkViewportSize();
@@ -4834,6 +4857,7 @@ class OneGridBodyComponent {
          * Listen Pinned Row Data Changed Event
          */
         this.subscriptions.add(this.eventService.addEventListener('pinnedRowDataChanged', ({ side, nodes }) => {
+            this.rendererParamsCache.clear();
             if (side === 'top') {
                 this.pinnedTopNodes = nodes;
             }
@@ -5416,16 +5440,14 @@ class OneGridBodyComponent {
     cellParams(node, column) {
         return this.cellService.cellParams(column, node);
     }
-    /**
-     * Params for a component cell renderer.
-     *
-     * The cell params, plus whatever the column declared in
-     * `cellRendererParams` — a field that exists precisely so a renderer can be
-     * configured per column, and that nothing had ever forwarded to it. Column
-     * config goes first so the grid's own fields win a name collision.
-     */
     rendererParams(node, column) {
-        return { ...(column?.cellRendererParams ?? {}), ...this.cellParams(node, column) };
+        const key = getCellKey(node.rowId, column.field ?? column.id ?? '');
+        const cached = this.rendererParamsCache.get(key);
+        if (cached && cached.states === node.cellStates && cached.data === node.data)
+            return cached.params;
+        const params = { ...(column?.cellRendererParams ?? {}), ...this.cellParams(node, column) };
+        this.rendererParamsCache.set(key, { states: node.cellStates, data: node.data, params });
+        return params;
     }
     /**
      * `aria-rowindex` is 1-based over the whole grid, not over what is rendered.