search display options are not quoted #78

Open
cburroughs opened this Issue Mar 18, 2013 · 0 comments

Projects

None yet

1 participant

@cburroughs

I'm not sure of this is a problem with html form or solr, or with spaces or other special characters. If optionDisplay is used for search field and the value selcted has some combination of the above, then the search will return no results. Since the dropdown values are expected to be exact I believe they should probably be quoted.

Does not work:

2013-03-18 08:57:30,063 - [DEBUG] - CollinsSearchQuery - p.a.LoggerLike$class:debug:70 - SOLR: (BASE_PRODUCT_meta_s:*ProLiant SL335s G7 (random-string)*) AND DOC_TYPE:ASSET| sort: TAG_SORT

By hand CQL query, does work:

2013-03-18 08:58:25,650 - [DEBUG] - CollinsSearchQuery - p.a.LoggerLike$class:debug:70 - SOLR: (BASE_PRODUCT_meta_s:"ProLiant SL335s G7 (random-string)") AND DOC_TYPE:ASSET| sort: TAG_SORT
@cburroughs cburroughs added a commit to cburroughs/collins that referenced this issue Apr 10, 2013
@cburroughs cburroughs If we have a dropdown in the UI, do an exact search
Theoretically the previous behavior would not work if one value was a
substring of the other.  In practice it also would fail to find any
results for some odd combination of edge cases, for example "Dell
Inc."

should handle #78
a004e4f
@dalehamel dalehamel pushed a commit to Shopify/collins that referenced this issue Jun 10, 2014
@cburroughs cburroughs If we have a dropdown in the UI, do an exact search
Theoretically the previous behavior would not work if one value was a
substring of the other.  In practice it also would fail to find any
results for some odd combination of edge cases, for example "Dell
Inc."

should handle #78
9607b66
@dalehamel dalehamel pushed a commit to Shopify/collins that referenced this issue Jun 10, 2014
@cburroughs cburroughs If we have a dropdown in the UI, do an exact search
Theoretically the previous behavior would not work if one value was a
substring of the other.  In practice it also would fail to find any
results for some odd combination of edge cases, for example "Dell
Inc."

should handle #78
06aeb90
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment