Apply suggestions from code review

Co-authored-by: Domenico Gemoli <dgemoli@redcross.nl>
global-121 · Jan 20, 2025 · ae6e925 · ae6e925
1 parent 07f4586
commit ae6e925
Showing 1 changed file with 2 additions and 7 deletions.
diff --git a/interfaces/Portalicious/src/app/components/query-table/query-table.component.ts b/interfaces/Portalicious/src/app/components/query-table/query-table.component.ts
@@ -148,7 +148,7 @@ export class QueryTableComponent<TData extends { id: PropertyKey }, TContext> {
 
   selectedColumnsStateKey = computed(() => {
     const key = this.localStorageKey();
-    return key ? `${key}-selected-columns` : 'selected-columns';
+    return key ? `${key}-selected-columns` : undefined;
   });
 
   /**
@@ -501,13 +501,8 @@ export class QueryTableComponent<TData extends { id: PropertyKey }, TContext> {
       const interpretedColumns = JSON.parse(
         storedSelectedColumns,
       ) as QueryTableColumn<TData>[];
-      console.log(
-        '🚀 ~ QueryTableComponent<TData ~ resetColumnVisibility ~ interpretedColumns:',
-        interpretedColumns,
-      );
       const matchedColumns = interpretedColumns
-        .map((column) => this.columns().find((c) => c.field === column.field))
-        .filter(Boolean) as QueryTableColumn<TData>[];
+        .filter((column) => !!this.columns().find((c) => c.field === column.field))  as QueryTableColumn<TData>[];
       this.visibleColumns.set(matchedColumns);
     } else {
       this.visibleColumns.set(