@fuentis/phoenix-ui 0.0.9-alpha.636 → 0.0.9-alpha.637

package/fesm2022/fuentis-phoenix-ui.mjs

@@ -2023,7 +2023,8 @@ class TableCaptionComponent {
      */
     lastSignature = '';
     ngOnChanges(changes) {
-        if (changes['filterValidValues'] && !changes['filterValidValues'].firstChange) {
+        if (changes['filterValidValues'] &&
+            !changes['filterValidValues'].firstChange) {
             const validMap = changes['filterValidValues'].currentValue;
             if (validMap)
                 this.applyValidValuesPatch(validMap);
@@ -2075,7 +2076,7 @@ class TableCaptionComponent {
                         ? !!f.selected
                         : t === 'range'
                             ? this.normalizeRangeInitial(f.selected, f)
-                            : f.selected ?? '';
+                            : (f.selected ?? '');
             fg.addControl(f.key, this.fb.control(initialValue));
         }
         this.filtersForm = fg;
@@ -2263,7 +2264,9 @@ class TableCaptionComponent {
                 continue;
             }
             if (t === 'multiselect') {
-                const arr = Array.isArray(v) ? v.filter((x) => x != null && x !== '') : [];
+                const arr = Array.isArray(v)
+                    ? v.filter((x) => x != null && x !== '')
+                    : [];
                 if (arr.length)
                     out[key] = arr;
                 continue;
@@ -2293,6 +2296,11 @@ class TableCaptionComponent {
      * NOTE:
      * - boolean-multiselect is restored into OPTION OBJECTS (for UI),
      *   even though persisted payload is boolean[].
+     * - Since `options` are rebuilt from the currently loaded table data (see
+     *   createDynamicFilters), a value selected in one context (e.g. a different
+     *   Entity in "Add link") may no longer exist in the current options. We
+     *   revalidate multiselect/dropdown selections against the current options
+     *   on every restore, so stale values don't render as broken/empty entries.
      */
     restoreFilterState(cfg) {
         const stored = localStorage.getItem(this.FILTER_KEY);
@@ -2308,25 +2316,39 @@ class TableCaptionComponent {
                 if (known.has(k))
                     patch[k] = parsed[k];
             }
+            let changed = false;
             for (const f of cfg) {
                 const t = f.type ?? 'text';
+                const options = Array.isArray(f.options) ? f.options : [];
+                const ov = f.optionValue ?? 'key';
                 if (t === 'boolean-multiselect') {
-                    // parsed payload is boolean[]; map -> option objects
+                    // parsed payload is boolean[]; map -> option objects (already validated against options)
                     patch[f.key] = this.normalizeBooleanMultiSelection(patch[f.key], f);
                     continue;
                 }
                 if (t === 'multiselect') {
                     const val = patch[f.key];
-                    if (val == null)
-                        patch[f.key] = [];
-                    else if (!Array.isArray(val))
-                        patch[f.key] = [val];
+                    const arr = val == null ? [] : Array.isArray(val) ? val : [val];
+                    const validValues = new Set(options.map((o) => o?.[ov]));
+                    const filtered = arr.filter((x) => validValues.has(x));
+                    if (filtered.length !== arr.length)
+                        changed = true;
+                    patch[f.key] = filtered;
                     continue;
                 }
                 if (t === 'checkbox') {
                     patch[f.key] = !!patch[f.key];
                     continue;
                 }
+                if (t === 'dropdown' || t === 'person-dropdown') {
+                    const val = patch[f.key];
+                    const isEmpty = val == null || val === '';
+                    const isValid = isEmpty || options.some((o) => o?.[ov] === val);
+                    if (!isValid) {
+                        patch[f.key] = '';
+                        changed = true;
+                    }
+                }
                 if (t === 'range') {
                     // persisted payload is { type:'range', min, max }; map -> [from, to]
                     const stored = patch[f.key];
@@ -2340,6 +2362,11 @@ class TableCaptionComponent {
                 }
             }
             this.filtersForm.patchValue(patch, { emitEvent: false });
+            // Persist the pruned state so stale (non-existent) values don't linger in storage.
+            if (changed) {
+                const payload = this.normalizeOutgoingPayload(this.filtersForm.getRawValue(), cfg);
+                this.persistState(payload, this.FILTER_KEY);
+            }
         }
         catch {
             // ignore invalid state
@@ -2433,7 +2460,9 @@ class TableCaptionComponent {
     /** Reset a range filter back to its full [min, max] bounds. */
     resetRangeFilter(control) {
         const [min, max] = this.rangeBounds(control);
-        this.filtersForm.get(control.key)?.setValue([min, max], { emitEvent: true });
+        this.filtersForm
+            .get(control.key)
+            ?.setValue([min, max], { emitEvent: true });
     }
     resetFilters() {
         this.filtersForm.reset({}, { emitEvent: true });
@@ -2443,7 +2472,9 @@ class TableCaptionComponent {
     persistState(state, key) {
         const isEmpty = state == null ||
             (Array.isArray(state) && state.length === 0) ||
-            (typeof state === 'object' && !Array.isArray(state) && Object.keys(state).length === 0);
+            (typeof state === 'object' &&
+                !Array.isArray(state) &&
+                Object.keys(state).length === 0);
         if (isEmpty)
             localStorage.removeItem(key);
         else
@@ -2456,7 +2487,7 @@ class TableCaptionComponent {
         const arr = Array.isArray(value) ? value : [];
         const labelField = this.getOptionLabelField(control);
         return arr
-            .map((x) => (x && typeof x === 'object' ? x[labelField] : String(x ?? '')))
+            .map((x) => x && typeof x === 'object' ? x[labelField] : String(x ?? ''))
             .filter((s) => !!s)
             .join(', ');
     }