From 98f8972f91d946a1c614ccb5d8475cee1b4882fb Mon Sep 17 00:00:00 2001 From: kentsanggds Date: Thu, 16 Jan 2025 17:56:57 +0000 Subject: [PATCH] Split filter query rather than have queries in one string --- app/services/search/solr.rb | 2 +- spec/services/search/solr_spec.rb | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/app/services/search/solr.rb b/app/services/search/solr.rb index 5bec8e8de..1f64a94a2 100644 --- a/app/services/search/solr.rb +++ b/app/services/search/solr.rb @@ -33,7 +33,7 @@ def self.build_filter_query(params) format_param = params.dig(:filters, :format) licence_param = params.dig(:filters, :licence_code) - filter_query = ["state:active"] + filter_query = ["state:active", "type:dataset"] filter_query << publisher_filter(publisher_param) if publisher_param.present? filter_query << topic_filter(topic_param) if topic_param.present? filter_query << format_filter(format_param) if format_param.present? diff --git a/spec/services/search/solr_spec.rb b/spec/services/search/solr_spec.rb index 007fb975f..771c8ff6d 100644 --- a/spec/services/search/solr_spec.rb +++ b/spec/services/search/solr_spec.rb @@ -341,7 +341,7 @@ it "includes active datasets filter" do filter_query = described_class.build_filter_query({ filters: {} }) - expect(filter_query).to eq(["state:active"]) + expect(filter_query).to eq(["state:active", "type:dataset"]) end end