ngx-eiffage-material 0.0.47 → 0.0.48

package/fesm2022/ngx-eiffage-material.mjs

@@ -3018,19 +3018,34 @@ class NgxPaginatedSelectBottomSheetComponent {
     }, ...(ngDevMode ? [{ debugName: "someSelected" }] : []));
     // ─── Accumulate pages ──────────────────────────────────────────────────────
     /**
-     * Only reads `resource.value()` — never `data.filters()`.
+     * Reactively depends ONLY on `resource.value()`. The filter snapshot is read
+     * via `untracked` so external filter changes (parent-driven, e.g. an extra
+     * filters panel) do NOT create a reactive dependency here — that would make
+     * the effect re-run on scroll page increments with stale resource data,
+     * duplicating items and breaking `onScroll` (see original limitation).
      *
-     * When a query change or a new open resets to page 1 we call
-     * `accumulatedItems.set([])` synchronously *before* updating the filter,
-     * so by the time this effect fires with new data it always appends onto an
-     * already-clean array.  Reading `data.filters()` here would cause the effect
-     * to run a second time (with stale resource data) the moment the page number
-     * is incremented on scroll, duplicating items and making
-     * `options().length >= total()` so `onScroll` would never fire again.
+     * Reset logic:
+     *  - Fingerprint the current filter identity (everything except `pageNumber`).
+     *  - If the fingerprint changed since the previous fetch, this fetch belongs
+     *    to a different filter set → replace the accumulated array.
+     *  - If only `pageNumber` moved, this is a scroll-load-more → append.
+     *
+     * This is what allows callers to mutate `data.filters` from outside (extra
+     * filter selects, checkboxes, etc.) and have the list refresh cleanly
+     * without the previous page's items lingering.
      */
+    _lastFilterFingerprint = null;
     _accumEffect = effect(() => {
         const value = this.resource.value();
-        this.accumulatedItems.update(acc => [...acc, ...value.items]);
+        const { pageNumber: _pn, ...rest } = untracked(() => this.data.filters());
+        const fingerprint = JSON.stringify(rest);
+        if (fingerprint !== this._lastFilterFingerprint) {
+            this._lastFilterFingerprint = fingerprint;
+            this.accumulatedItems.set([...value.items]);
+        }
+        else {
+            this.accumulatedItems.update(acc => [...acc, ...value.items]);
+        }
     }, ...(ngDevMode ? [{ debugName: "_accumEffect" }] : []));
     constructor() {
         // Reset to page 1 when the sheet opens