Update SolrKeywordSearchHandler.java#2378
Update SolrKeywordSearchHandler.java#2378sammyjava wants to merge 4 commits intointermine:devfrom sammyjava:patch-13
Conversation
Wrap faceted search term in quotes so it works on things like "C. arietinum".
Shortened the line to less than 100 chars (stupid requirement!)
"+ should be on a new line." You realize that the American taxpayer is paying me to update this commit for each one of these stupid checkstyle fails.
|
the change is fine, but i am not sure about the intended behaviour. |
|
Yes, I'm not totally sure this was what fixed the problem. If you like, I can retest with this update backed out. |
|
hi @sammyjava , sorry for the long time it took us to check this! (it is again release time..) |
|
Sounds fine, I think I'll remove this update in my fork and see what happens that I didn't like before, if it happens. :) |
|
i think we can close this, please reopen @sammyjava if needed. thanks! |
|
I agree. :) |
|
So, @danielabutano and @sergiocontrino a user noticed that a recent mine of mine doesn't properly facet on organism (built after I'd removed this change). I can confirm that this bug is reproduced in the current FlyMine and that this PR does, in fact, fix it on my mines.
This is, in fact our bug. We have several species within the same genus, and the Organism facet filter always returns all organisms within the chosen genus, because of this issue with the space in G. species that is fixed with this PR. So, I'm going to add a PR again for a future update (the repo for this one is gone), I'll reference it below. |
Wrap faceted search term in quotes so it works on things like "C. arietinum".
Details
Summary of pull request, including references to relevant tickets (if applicable).
Testing
Besides running unit tests, how can the reviewer test your feature / bug fix? Are there edge cases to be aware of?
Checklist
Before your pull request can be approved, be sure to check all boxes: