SyntaxError: Cannot parse '(())'
./solr_log_20150619_1509:ERROR - 2015-06-19 09:50:55.986; [ ] org.apache.solr.common.SolrException; org.apache.solr.common.SolrException: org.apache.solr.search.SyntaxError: Cannot parse '(())': Encountered " ")" ") "" at line 1, column 2.
./solr_log_20150619_1509:Caused by: org.apache.solr.search.SyntaxError: Cannot parse '(())': Encountered " ")" ") "" at line 1, column 2.
./solr_log_20150619_1509:INFO - 2015-06-19 09:50:55.987; [ ] org.apache.solr.core.SolrCore; [] webapp=/solr path=/select params={sort=bs_search_api_aggregation_13+desc,+score+desc,+fs_search_api_aggregation_3+desc,+bs_sticky+desc&fl=item_id,score&start=0&q=house&qf=tm_search_api_aggregation_1^3.0&json.nl=map&wt=json&fq=(())&fq=index_id:"product_display"&fq=hash:jw4ziu&rows=30} status=400 QTime=4
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | 2513314-5--empty_filter_query.patch | 863 bytes | drunken monkey |
| #1 | search_api_solr-fix_empty_filter_query-2513314-1.patch | 1.12 KB | fortis |
Comments
Comment #1
fortis commentedthere is problem
$fq[] = count($q) == 1 ? reset($q) : '((' . implode(')' . $sep . '(', $q) . '))';we need check that $q not empty
quickfix attached
Comment #2
fortis commentedComment #3
fortis commentedComment #4
thepanz commentedWorks fine, thanks!
Comment #5
drunken monkeyThanks a lot for reporting this! You're right, we should definitely check for this (even though there's gotta be something strange about the incoming query then, too).
Your patch looks also good and seems to work fine!
However, I think this one is a bit more elegant, does it also work for you?
Comment #6
fortis commentedYes, it works, thank you!
Comment #7
drunken monkeyOK, good to hear. Thanks for testing, and thanks again for your work before!
Committed!
Comment #9
fortis commentedComment #11
c-logemannHere is a patch for 8.x-1.x-dev branch: #2625932: Fix "SyntaxError: Cannot parse '(())'" also in 8.x-1.x-dev