support legacy string with parenthesis

algolia · Feb 1, 2022 · 6015b3e · 6015b3e
1 parent 99a40c3
commit 6015b3e
Show file tree

Hide file tree

Showing 2 changed files with 24 additions and 5 deletions.
diff --git a/...search-core/src/main/java/com/algolia/search/models/indexing/FiltersJsonDeserializer.java b/...search-core/src/main/java/com/algolia/search/models/indexing/FiltersJsonDeserializer.java
@@ -33,10 +33,8 @@ public List<List<String>> deserialize(JsonParser p, DeserializationContext ctxt)
         result = buildFilters(list);
         break;
       case VALUE_STRING:
-        result =
-            Arrays.stream(p.getValueAsString().split(","))
-                .map(Collections::singletonList)
-                .collect(Collectors.toList());
+        String string = p.getValueAsString();
+        result = buildFilters(string);
         break;
       case VALUE_NULL:
         break;
@@ -48,6 +46,7 @@ public List<List<String>> deserialize(JsonParser p, DeserializationContext ctxt)
     return result;
   }
 
+  /** Build filters from a list */
   @SuppressWarnings("unchecked")
   private List<List<String>> buildFilters(List list) {
     return (List<List<String>>)
@@ -62,4 +61,16 @@ private List<List<String>> buildFilters(List list) {
                 })
             .collect(Collectors.toList());
   }
+
+  /** Build filters from (legacy) string */
+  private List<List<String>> buildFilters(String string) {
+    if (string.startsWith("(") && string.endsWith(")")) {
+      String input = string.substring(1, string.length() - 1);
+      return Collections.singletonList(Arrays.asList(input.split(",")));
+    } else {
+      return Arrays.stream(string.split(","))
+          .map(Collections::singletonList)
+          .collect(Collectors.toList());
+    }
+  }
 }
diff --git a/algoliasearch-core/src/test/java/com/algolia/search/JacksonParserTest.java b/algoliasearch-core/src/test/java/com/algolia/search/JacksonParserTest.java
@@ -174,7 +174,6 @@ void testLegacyFiltersFormat(String input) throws IOException {
     // Testing "one string" legacy filters => should be converted to "ANDED" filters
     // [["color:green"],["color:yellow"]]
     String stringFilters = String.format("{\"%s\":\"color:green,color:yellow\"}", input);
-
     assertANDEDListResult(
         extractFilters(
             Defaults.getObjectMapper().readValue(stringFilters, ConsequenceParams.class), input));
@@ -193,6 +192,15 @@ void testLegacyFiltersFormat(String input) throws IOException {
             Defaults.getObjectMapper().readValue(nestedArrayFilters, ConsequenceParams.class),
             input));
 
+    // Testing "one string with parenthesis" legacy filters => should be converted to "ORED" filters
+    // [["color:green", "color:yellow"]]
+    String stringParenthesisFilters =
+        String.format("{\"%s\":\"(color:green,color:yellow)\"}", input);
+    assertOREDResult(
+        extractFilters(
+            Defaults.getObjectMapper().readValue(stringParenthesisFilters, ConsequenceParams.class),
+            input));
+
     // Testing mixed case with array and string
     // [["color:green","color:yellow"],"color:blue"]
     String stringAndArrayFilters =