Skip to content

Commit 0f09920

Browse files
authored
[Port dspace-7_x] site search should not adopt existing URL query string parameters (DSpace#2824)
Do not merge existing query params when using site search from MyDSpace workflow.
1 parent 41faa5e commit 0f09920

2 files changed

Lines changed: 3 additions & 4 deletions

File tree

src/app/search-navbar/search-navbar.component.spec.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -88,7 +88,7 @@ describe('SearchNavbarComponent', () => {
8888
fixture.detectChanges();
8989
}));
9090
it('to search page with empty query', () => {
91-
const extras: NavigationExtras = { queryParams: { query: '' }, queryParamsHandling: 'merge' };
91+
const extras: NavigationExtras = { queryParams: { query: '' } };
9292
expect(component.onSubmit).toHaveBeenCalledWith({ query: '' });
9393
expect(router.navigate).toHaveBeenCalledWith(['search'], extras);
9494
});
@@ -113,7 +113,7 @@ describe('SearchNavbarComponent', () => {
113113
fixture.detectChanges();
114114
}));
115115
it('to search page with query', async () => {
116-
const extras: NavigationExtras = { queryParams: { query: 'test' }, queryParamsHandling: 'merge' };
116+
const extras: NavigationExtras = { queryParams: { query: 'test' } };
117117
expect(component.onSubmit).toHaveBeenCalledWith({ query: 'test' });
118118

119119
expect(router.navigate).toHaveBeenCalledWith(['search'], extras);

src/app/search-navbar/search-navbar.component.ts

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -66,8 +66,7 @@ export class SearchNavbarComponent {
6666
this.searchForm.reset();
6767

6868
this.router.navigate(linkToNavigateTo, {
69-
queryParams: queryParams,
70-
queryParamsHandling: 'merge'
69+
queryParams: queryParams
7170
});
7271
}
7372
}

0 commit comments

Comments
 (0)