Fix bug when combining filters

netarchivesuite · Aug 9, 2023 · 529d2cb · 529d2cb
1 parent 0bea2c4
commit 529d2cb
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/src/main/java/dk/kb/netarchivesuite/solrwayback/util/SolrUtils.java b/src/main/java/dk/kb/netarchivesuite/solrwayback/util/SolrUtils.java
@@ -430,8 +430,8 @@ public static String fieldValueToString(Object value) {
      */
     public static String combineFilterQueries(String predefinedFilterField, String predefinedFilterValue, String[] filterQueries) {
         Stream<String> filtersStream = Stream.of(filterQueries);
-        Stream<String> fullFiltersStream =Stream.concat(Stream.of(predefinedFilterField + ":(" + predefinedFilterValue + ")"), filtersStream);
+        Stream<String> fullFiltersStream =Stream.concat(Stream.of(predefinedFilterField + ":" + predefinedFilterValue), filtersStream);
         return fullFiltersStream
-                .collect(Collectors.joining(" AND ", "(", ")"));
+                .collect(Collectors.joining(") AND (", "(", ")"));
     }
 }
diff --git a/src/test/java/dk/kb/netarchivesuite/solrwayback/util/SolrUtilsTest.java b/src/test/java/dk/kb/netarchivesuite/solrwayback/util/SolrUtilsTest.java
@@ -2,13 +2,15 @@
 
 import org.junit.Test;
 
+import static org.junit.Assert.assertEquals;
+
 public class SolrUtilsTest {
 
     @Test
     public void combineFilterQueriesTest(){
-        String[] filtersFromFrontend = new String[]{"filter1:value1", "filter2:value2"};
+        String[] filtersFromFrontend = new String[]{"filter1:value1", "filter2:value2", "foo:bar OR bar:zoo"};
         String filterquery = SolrUtils.combineFilterQueries("content_type", "text/html", filtersFromFrontend);
 
-        System.out.println(filterquery);
+        assertEquals("(content_type:text/html) AND (filter1:value1) AND (filter2:value2) AND (foo:bar OR bar:zoo)", filterquery);
     }
 }