Bug #15363 [Search + Collect] – When saving a search with multiple filters, only one filter is retained.

Salim Terres · Salim Terres · commit c25c081fce5d · 2025-11-05T16:18:52.000+01:00
diff --git a/ui/ui-frontend/projects/archive-search/src/app/archive/archive-search/archive-search.component.ts b/ui/ui-frontend/projects/archive-search/src/app/archive/archive-search/archive-search.component.ts
@@ -789,7 +789,7 @@ export class ArchiveSearchComponent implements OnInit, OnChanges, OnDestroy, Aft
           category,
           criteria.valueTranslated,
           criteria.dataType,
-          true,
+          false,
         );
       });
     });
diff --git a/ui/ui-frontend/projects/archive-search/src/app/core/archive-shared-data.service.ts b/ui/ui-frontend/projects/archive-search/src/app/core/archive-shared-data.service.ts
@@ -237,7 +237,10 @@ export class ArchiveSharedDataService {
       }
       this.simpleSearchCriteriaAddSubject.next(searchCriteria);
     });
-    builder.navigate();
+    // NOTE: Navigation disabled to prevent overwriting searchCriterias loaded from saved filters
+    // The local state is already updated via simpleSearchCriteriaAddSubject.next()
+    // Query params will be synced when performing a search
+    // builder.navigate();
   }
 
   addSimpleSearchCriteriaSubject(searchCriteria: SearchCriteriaAddAction) {
diff --git a/ui/ui-frontend/projects/collect/src/app/collect/archive-search-collect/archive-search-collect.component.ts b/ui/ui-frontend/projects/collect/src/app/collect/archive-search-collect/archive-search-collect.component.ts
@@ -1182,7 +1182,7 @@ export class ArchiveSearchCollectComponent extends SidenavPage<any> implements O
           category,
           criteria.valueTranslated,
           criteria.dataType,
-          true,
+          false,
         );
       });
     });
diff --git a/ui/ui-frontend/projects/collect/src/app/collect/core/archive-shared-data.service.ts b/ui/ui-frontend/projects/collect/src/app/collect/core/archive-shared-data.service.ts
@@ -232,7 +232,10 @@ export class ArchiveSharedDataService {
       }
       this.simpleSearchCriteriaAddSubject.next(searchCriteria);
     });
-    builder.navigate();
+    // NOTE: Navigation disabled to prevent overwriting searchCriterias loaded from saved filters
+    // The local state is already updated via simpleSearchCriteriaAddSubject.next()
+    // Query params will be synced when performing a search
+    // builder.navigate();
   }
 
   addSimpleSearchCriteriaSubject(searchCriteria: SearchCriteriaAddAction) {

Original file line number	Diff line number	Diff line change
`@@ -237,7 +237,10 @@ export class ArchiveSharedDataService {`
`237`	`237`	`}`
`238`	`238`	`this.simpleSearchCriteriaAddSubject.next(searchCriteria);`
`239`	`239`	`});`
`240`		`- builder.navigate();`
	`240`	`+ // NOTE: Navigation disabled to prevent overwriting searchCriterias loaded from saved filters`
	`241`	`+ // The local state is already updated via simpleSearchCriteriaAddSubject.next()`
	`242`	`+ // Query params will be synced when performing a search`
	`243`	`+ // builder.navigate();`
`241`	`244`	`}`
`242`	`245`
`243`	`246`	`addSimpleSearchCriteriaSubject(searchCriteria: SearchCriteriaAddAction) {`
Original file line number	Diff line number	Diff line change
`@@ -232,7 +232,10 @@ export class ArchiveSharedDataService {`
`232`	`232`	`}`
`233`	`233`	`this.simpleSearchCriteriaAddSubject.next(searchCriteria);`
`234`	`234`	`});`
`235`		`- builder.navigate();`
	`235`	`+ // NOTE: Navigation disabled to prevent overwriting searchCriterias loaded from saved filters`
	`236`	`+ // The local state is already updated via simpleSearchCriteriaAddSubject.next()`
	`237`	`+ // Query params will be synced when performing a search`
	`238`	`+ // builder.navigate();`
`236`	`239`	`}`
`237`	`240`
`238`	`241`	`addSimpleSearchCriteriaSubject(searchCriteria: SearchCriteriaAddAction) {`