impresso / impresso-middle-layer

Middle layer API
GNU Affero General Public License v3.0
0 stars 1 forks source link

[chore] Do not double-validate filters and construct the Solr query correctly #402

Closed theorm closed 4 months ago

theorm commented 4 months ago

This PR contains two fixes:

  1. Filters were double parsed resulting in an array of arrays structure. Fixed to have a single top level array.
  2. Text reuse passages queries with text and filters were not built correctly:
    • Before: content_txt_fr:"Plus" OR content_txt_de:"Plus" OR content_txt_en:"Plus" AND filter(cluster_size_l:[50 TO 100])
    • After: (content_txt_fr:"Plus" OR content_txt_de:"Plus" OR content_txt_en:"Plus") AND filter(cluster_size_l:[50 TO 100])