@elasticias/screens 1.0.41 → 1.0.42

package/fesm2022/elasticias-screens.mjs

@@ -458,6 +458,13 @@ class AbstractScreenComponent extends AbstractComponent {
         const v = raw;
         if (!v || typeof v !== 'object' || !v.presetKey)
             return null;
+        // An open range stores null bounds. `new Date(null)` is 1970, not
+        // NaN, so it must be handled before parsing, not after.
+        if (v.start == null && v.end == null) {
+            return { start: null, end: null, presetKey: v.presetKey, label: v.label ?? '', labelKey: v.labelKey };
+        }
+        if (v.start == null || v.end == null)
+            return null;
         const start = new Date(v.start);
         const end = new Date(v.end);
         if (isNaN(start.getTime()) || isNaN(end.getTime()))
@@ -1075,14 +1082,27 @@ class AbstractSearchScreenV2 extends AbstractScreenComponent {
     criteria = signal(new SearchEntity({}), ...(ngDevMode ? [{ debugName: "criteria" }] : /* istanbul ignore next */ []));
     /**
      * Active date-range filter shown in `ef-datepicker-advanced`'s
-     * trigger. Display-only at construction time — actually filters
-     * the search once `onDateRangeChange()` fires (or a screen wires
-     * one in `ngOnInit`).
+     * trigger, and the single source of the criteria's `start`/`end`.
+     * The first search, `clear()` and a picked range all send exactly
+     * what this holds, and a cache restore rebuilds it from the restored
+     * dates, so the chip never reads one period while the list shows
+     * another.
      *
-     * Override `buildDefaultDateRange()` per screen to ship a
-     * different starting preset.
+     * Defaults to `'all_time'` (no dates sent). Override
+     * `buildDefaultDateRange()` per screen to open on a window instead.
      */
     dateRange = signal(this.buildDefaultDateRange(), ...(ngDevMode ? [{ debugName: "dateRange" }] : /* istanbul ignore next */ []));
+    /**
+     * Whether this screen offers the date chip at all. Set it to `false`
+     * (`override readonly showDateFilter = false;`) on a screen whose
+     * backend query has no date field, and guard the template's
+     * `<ef-datepicker-advanced>` with `@if (showDateFilter)` (or drop it).
+     * The base then never sends `start`/`end`: not on first load, not
+     * after `clear()`, not from criteria cached by an older session, and
+     * not from `onDateRangeChange()` / `setDateRange()`, which become
+     * no-ops, so a picker left in the template cannot filter either.
+     */
+    showDateFilter = true;
     /* ── Filter-bar UI state (DRY) ──────────────────────────────────
          Identical across every list screen — lifted up so subclasses
          don't re-declare. `clear()` resets all four signals. */
@@ -1196,14 +1216,16 @@ class AbstractSearchScreenV2 extends AbstractScreenComponent {
     }
     /* ── Date-range filter (DRY) ────────────────────────────────────
          Wired to ef-datepicker-advanced. The default range is
-         `last_30_days` — override `buildDefaultDateRange()` per screen
-         if a different starting preset is needed. */
+         `all_time`, which sends no dates. Override
+         `buildDefaultDateRange()` per screen if a window is needed. */
     /**
      * Wired to `<ef-datepicker-advanced (rangeChange)>` — stores the
      * range for trigger display and pushes it into the search criteria
      * so the next `search()` filters by it.
      */
     onDateRangeChange(range) {
+        if (!this.showDateFilter)
+            return;
         this.dateRange.set(range);
         this.setDateRange(range.start, range.end);
     }
@@ -1216,19 +1238,52 @@ class AbstractSearchScreenV2 extends AbstractScreenComponent {
         this.dateRange.set(this.buildDefaultDateRange());
     }
     /**
-     * Build the default `EfDateRange`. Override per screen to ship a
-     * different default — e.g., a 90-day window for low-velocity
-     * catalogues. Default: last 30 days.
+     * Build the default `EfDateRange`. Default: `all_time`, no bounds,
+     * so a typed search finds an old record instead of silently missing
+     * it behind a window nobody chose. Override per screen to open on a
+     * window (e.g. `daysAgo(89)` to `startOfToday()` for `last_90_days`);
+     * the first search and `clear()` then send that window.
      */
     buildDefaultDateRange() {
         return {
-            start: this.daysAgo(29),
-            end: this.startOfToday(),
-            presetKey: 'last_30_days',
-            labelKey: 'date_preset_last_30_days',
-            label: '30 derniers jours',
+            start: null,
+            end: null,
+            presetKey: 'all_time',
+            labelKey: 'date_preset_all_time',
+            label: '',
         };
     }
+    /** Copy the chip's range onto fresh criteria, so what is shown is
+     *  what is sent. Sends nothing when the screen has no date chip. */
+    seedDateCriteria(criteria) {
+        const range = this.showDateFilter ? this.dateRange() : null;
+        criteria.start = range?.start ?? null;
+        criteria.end = range?.end ?? null;
+    }
+    /** Rebuild the chip from restored criteria: both dates are shown as
+     *  the custom range they filter on; anything less is `all_time`, and
+     *  `bootstrapInitialSearch()` drops a lone bound so nothing is sent.
+     *  No literal label: the picker formats a custom range itself. */
+    dateRangeFromCriteria(startRaw, endRaw) {
+        const start = this.toDateOrNull(startRaw);
+        const end = this.toDateOrNull(endRaw);
+        if (!start || !end) {
+            return {
+                start: null,
+                end: null,
+                presetKey: 'all_time',
+                labelKey: 'date_preset_all_time',
+                label: '',
+            };
+        }
+        return { start, end, presetKey: 'custom', label: '' };
+    }
+    toDateOrNull(raw) {
+        if (raw == null || raw === '')
+            return null;
+        const d = raw instanceof Date ? raw : new Date(raw);
+        return isNaN(d.getTime()) ? null : d;
+    }
     /** Today at 00:00 local time. */
     startOfToday() {
         const d = new Date();
@@ -1430,7 +1485,17 @@ class AbstractSearchScreenV2 extends AbstractScreenComponent {
     bootstrapInitialSearch() {
         const cached = this.cacheService.getCache(this.screenStateKey);
         if (cached) {
-            this.criteria.set(new SearchEntity(cached));
+            const restored = new SearchEntity(cached);
+            const bothBounds = this.toDateOrNull(cached['start']) && this.toDateOrNull(cached['end']);
+            if (!this.showDateFilter || !bothBounds) {
+                // Criteria cached while this screen still had a chip may carry
+                // dates its backend ignores at best; and a lone bound would be
+                // sent under a chip reading "all time". Drop them.
+                restored.start = null;
+                restored.end = null;
+                restored.startEndDateRanges = null;
+            }
+            this.criteria.set(restored);
             this.restoreFilterUiFromCriteria(cached);
         }
         else {
@@ -1448,6 +1513,7 @@ class AbstractSearchScreenV2 extends AbstractScreenComponent {
                 pageNumber: PaginationEnum.DEFAULT_PAGE,
                 pageSize: PaginationEnum.DEFAULT_PAGE_SIZE,
             };
+            this.seedDateCriteria(fresh);
             this.criteria.set(fresh);
         }
         this.search();
@@ -1458,11 +1524,15 @@ class AbstractSearchScreenV2 extends AbstractScreenComponent {
      * what the drawer / search box displays matches what the search will
      * actually send (otherwise a stale criteria filter keeps applying
      * while every control reads "Tous"). Base handles the free-text
-     * input and the declared advanced filters; override (calling super)
-     * to sync screen-specific state — status pill, custom switches, ….
+     * input, the date chip and the declared advanced filters; override
+     * (calling super) to sync screen-specific state, such as a status pill
+     * or custom switches.
      */
     restoreFilterUiFromCriteria(cached) {
         this.searchQuery.set(cached['searchText'] ?? '');
+        if (this.showDateFilter) {
+            this.dateRange.set(this.dateRangeFromCriteria(cached['start'], cached['end']));
+        }
         const next = {};
         for (const f of this.advancedFilters) {
             const value = cached[f.key];
@@ -1535,6 +1605,8 @@ class AbstractSearchScreenV2 extends AbstractScreenComponent {
         this.search();
     }
     setDateRange(start, end) {
+        if (!this.showDateFilter)
+            return;
         this.criteria.update((c) => this.cloneCriteria(c, { start, end, page: 1 }));
         this.search();
     }
@@ -1557,6 +1629,8 @@ class AbstractSearchScreenV2 extends AbstractScreenComponent {
                 },
             ];
         }
+        this.resetDateRange();
+        this.seedDateCriteria(fresh);
         this.criteria.set(fresh);
         // Reset all the shared filter-bar / selection signals so
         // subclasses don't have to track each one individually.
@@ -1566,7 +1640,6 @@ class AbstractSearchScreenV2 extends AbstractScreenComponent {
         this.activeFilters.set([]);
         this.advancedValues.set({});
         this.selected.set(new Set());
-        this.resetDateRange();
         this.cacheService.setCache(this.screenStateKey, fresh);
         this.search();
     }
@@ -2401,7 +2474,7 @@ class AbstractReportScreenV2 extends AbstractScreenComponent {
         this.reportsService = this.injector.get(this.getConfig().SERVICE);
         // Reinstate the period the user last chose before anything loads, so the
         // widgets fetch it once instead of fetching the default and refetching.
-        this.dateRange.set(this.restorePersistedDateRange(this.dateRange(), key => this.rangeFromPreset(key)));
+        this.dateRange.set(this.boundedOrDefault(this.restorePersistedDateRange(this.dateRange(), key => this.rangeFromPreset(key))));
         const lists = this.getConfig()?.REPORT_STATIC_LISTS ?? [];
         if (lists.length > 0) {
             this.initializeStaticLists(lists);
@@ -2415,10 +2488,22 @@ class AbstractReportScreenV2 extends AbstractScreenComponent {
     /* ── Period ─────────────────────────────────────────────────── */
     /** Wired to `<ef-datepicker-advanced (rangeChange)>`. */
     onDateRangeChange(range) {
-        this.dateRange.set(range);
-        this.persistDateRange(range);
+        const bounded = this.boundedOrDefault(range);
+        this.dateRange.set(bounded);
+        this.persistDateRange(bounded);
         this.loadAll();
     }
+    /**
+     * Every widget query is a period, so an open range (`all_time`) has no
+     * meaning here. The picker only offers it with `allowAllTime`, which a
+     * report should not set; if one arrives anyway, fall back to the
+     * default period rather than send a null bound.
+     */
+    boundedOrDefault(range) {
+        return range.start && range.end
+            ? range
+            : this.buildDefaultDateRange();
+    }
     /** Default period from the config's `DEFAULT_PERIOD` preset. */
     buildDefaultDateRange() {
         return this.rangeFromPreset(this.getConfig()?.DEFAULT_PERIOD ?? 'last_30_days');