From a487f50a30dcd6824788ecb26537cc6b39845502 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Youn=C3=A8s=20Adem?= <younes.adem@lnds.lu>
Date: Wed, 28 Aug 2024 15:09:43 +0200
Subject: [PATCH] fix: remove fl attribute from ckan

---
 .../ckan/persistence/CkanDatasetIdsCollector.java           | 1 -
 .../ckan/persistence/CkanDatasetsRepository.java            | 1 -
 .../infrastructure/ckan/persistence/CkanFacetsBuilder.java  | 1 -
 src/main/openapi/ckan.yaml                                  | 6 ------
 4 files changed, 9 deletions(-)

diff --git a/src/main/java/io/github/genomicdatainfrastructure/discovery/datasets/infrastructure/ckan/persistence/CkanDatasetIdsCollector.java b/src/main/java/io/github/genomicdatainfrastructure/discovery/datasets/infrastructure/ckan/persistence/CkanDatasetIdsCollector.java
index bf49f62..1b03064 100644
--- a/src/main/java/io/github/genomicdatainfrastructure/discovery/datasets/infrastructure/ckan/persistence/CkanDatasetIdsCollector.java
+++ b/src/main/java/io/github/genomicdatainfrastructure/discovery/datasets/infrastructure/ckan/persistence/CkanDatasetIdsCollector.java
@@ -37,7 +37,6 @@ public List<String> collect(DatasetSearchQuery query, String accessToken) {
         var response = ckanQueryApi.packageSearch(
                 query.getQuery(),
                 facetsQuery,
-                CKAN_IDENTIFIER_FIELD,
                 null,
                 CKAN_PAGINATION_MAX_SIZE,
                 0,
diff --git a/src/main/java/io/github/genomicdatainfrastructure/discovery/datasets/infrastructure/ckan/persistence/CkanDatasetsRepository.java b/src/main/java/io/github/genomicdatainfrastructure/discovery/datasets/infrastructure/ckan/persistence/CkanDatasetsRepository.java
index 3311cd5..42c457e 100644
--- a/src/main/java/io/github/genomicdatainfrastructure/discovery/datasets/infrastructure/ckan/persistence/CkanDatasetsRepository.java
+++ b/src/main/java/io/github/genomicdatainfrastructure/discovery/datasets/infrastructure/ckan/persistence/CkanDatasetsRepository.java
@@ -72,7 +72,6 @@ public List<SearchedDataset> search(List<String> datasetIds,
         var response = ckanQueryApi.packageSearch(
                 null,
                 facetsQuery,
-                null,
                 sort,
                 rows,
                 start,
diff --git a/src/main/java/io/github/genomicdatainfrastructure/discovery/datasets/infrastructure/ckan/persistence/CkanFacetsBuilder.java b/src/main/java/io/github/genomicdatainfrastructure/discovery/datasets/infrastructure/ckan/persistence/CkanFacetsBuilder.java
index 63fa51a..723c6f5 100644
--- a/src/main/java/io/github/genomicdatainfrastructure/discovery/datasets/infrastructure/ckan/persistence/CkanFacetsBuilder.java
+++ b/src/main/java/io/github/genomicdatainfrastructure/discovery/datasets/infrastructure/ckan/persistence/CkanFacetsBuilder.java
@@ -44,7 +44,6 @@ public FacetGroup build(DatasetSearchQuery query, String accessToken) {
                 null,
                 null,
                 null,
-                null,
                 selectedFacets,
                 accessToken
         );
diff --git a/src/main/openapi/ckan.yaml b/src/main/openapi/ckan.yaml
index 05ebc73..a87f02a 100644
--- a/src/main/openapi/ckan.yaml
+++ b/src/main/openapi/ckan.yaml
@@ -30,12 +30,6 @@ paths:
           required: false
           schema:
             type: string
-        - name: fl
-          in: query
-          description: dataset fields to be returned
-          required: false
-          schema:
-            type: string
         - name: sort
           in: query
           description: Sorting of search results