Datahub : Custom data toggle broken

apps/datahub/src/app/home/home-header/home-header.component.spec.ts

@@ -34,7 +34,6 @@
           filters: { publisher: ['DREAL'] },
         },
         {
-          sort: 'title',
           name: 'filterCarto',
           filters: { q: 'Cartographie' },
         },
@@ -61,13 +60,15 @@
 class searchFacadeMock {
   setFavoritesOnly = jest.fn()
   setSortBy = jest.fn()
+  sortBy$ = new BehaviorSubject(['desc', 'createDate'])
 }
 
 class searchServiceMock {
   updateSearchFilters = jest.fn()
   setSearch = jest.fn()
   setSortBy = jest.fn()
   setSortAndFilters = jest.fn()
+  setFilters = jest.fn()
 }
 
 class AuthServiceMock {
@@ -139,7 +140,7 @@
     describe('isAuthenticated$', () => {
       describe('user is authenticated', () => {
         beforeEach(() => {
           ;(authService as any)._authSubject$.next({
             id: 'user-id',
             name: 'testuser',
           })
@@ -153,7 +154,7 @@
       })
       describe('user is NOT authenticated', () => {
         beforeEach(() => {
           ;(authService as any)._authSubject$.next(null)
         })
         it('does NOT display favoriteBadge when NOT authenticated', async () => {
           const isAuthenticated = await firstValueFrom(
@@ -175,7 +176,7 @@
   describe('sort badges', () => {
     describe('navigate to search route', () => {
       beforeEach(() => {
         ;(routerFacade.currentRoute$ as any).next({
           url: [{ path: ROUTER_ROUTE_SEARCH }],
         })
       })
@@ -188,7 +189,7 @@
     })
     describe('navigate to news route', () => {
       beforeEach(() => {
         ;(routerFacade.currentRoute$ as any).next({
           url: [{ path: ROUTER_ROUTE_NEWS }],
         })
       })
@@ -233,17 +234,32 @@
           const allBadges = fixture.debugElement.queryAll(By.css('.badge-btn'))
           expect(allBadges.length).toBe(4)
         })
-        beforeEach(() => {
-          const firstCustomBadge = fixture.debugElement.queryAll(
-            By.css('.badge-btn')
-          )[2]
-          firstCustomBadge.nativeElement.click()
+        describe('when sort is defined', () => {
+          beforeEach(() => {
+            const firstCustomBadge = fixture.debugElement.queryAll(
+              By.css('.badge-btn')
+            )[2]
+            firstCustomBadge.nativeElement.click()
+          })
+          it('should redirect correctly', () => {
+            expect(searchService.setSortAndFilters).toHaveBeenCalledWith(
+              { thisIs: 'a fake filter' },
+              SortByEnum.CREATE_DATE
+            )
+          })
         })
-        it('should redirect correctly', () => {
-          expect(searchService.setSortAndFilters).toHaveBeenCalledWith(
-            { thisIs: 'a fake filter' },
-            SortByEnum.CREATE_DATE
-          )
+        describe('when sort is not defined', () => {
+          beforeEach(() => {
+            const secondCustomBadge = fixture.debugElement.queryAll(
+              By.css('.badge-btn')
+            )[3]
+            secondCustomBadge.nativeElement.click()
+          })
+          it('should redirect correctly', () => {
+            expect(searchService.setFilters).toHaveBeenCalledWith({
+              thisIs: 'a fake filter',
+            })
+          })
         })
       })
 

apps/datahub/src/app/home/home-header/home-header.component.ts

@@ -19,8 +19,8 @@
 } from '@geonetwork-ui/util/app-config'
 import { SortByEnum, SortByField } from '@geonetwork-ui/common/domain/search'
 import { map } from 'rxjs/operators'
 import { ROUTER_ROUTE_NEWS } from '../../router/constants'
-import { lastValueFrom } from 'rxjs'
+import { firstValueFrom, lastValueFrom } from 'rxjs'
 import { CatalogRecord } from '@geonetwork-ui/common/domain/record'
 import { sortByFromString } from '@geonetwork-ui/util/shared'
 
@@ -84,9 +84,11 @@
         customSearchParameters.filters
       )
     )
-    this.searchService.setSortAndFilters(
-      searchFilters,
-      sortByFromString(customSearchParameters.sort)
-    )
+    if (customSearchParameters.sort) {
+      const sortBy = sortByFromString(customSearchParameters.sort)
+      this.searchService.setSortAndFilters(searchFilters, sortBy)
+    } else {
+      this.searchService.setFilters(searchFilters)
+    }
   }
 }

conf/default.toml

@@ -75,7 +75,7 @@ background_color = "#fdfbff"
 
 # One or several search presets can be defined here; every search preset is composed of:
 # - a name (which can be a translation key)
-# - a sort criteria: either `createDate`, `userSavedCount` or `_score` (prepend with `-` for descending sort)
+# - a sort criteria: either `createDate`, `userSavedCount` or `_score` (prepend with `-` for descending sort) (optionnal)
 # - filters which can be expressed like so:
 # [[search_preset]]
 #     name = 'filterByName'
@@ -88,6 +88,7 @@ background_color = "#fdfbff"
 #     filters.publicationYear = ['2023', '2022']
 #     filters.isSpatial = ['yes']
 #     filters.license = ['unknown']
+#     sort = 'createDate'
 # [[search_preset]]
 #     name = 'otherFilterName'
 #     filters.q = 'Other Full text search'