New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add configurable search parameters to search options and remove unused method (get_full_url) #1228
Conversation
…d method (get_full_url)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ran several tests against the test db using test.archivesspace.org and local build from this branch. Great improvements noted for:
- keyword search for 'MS Am'
Collections with identifiers starting 'MS Am' pulled to top.
- keyword search for 'additional papers'
Results containing just 'papers' now lower than previously.
- keyword search for 'Company D'
Result containing full phrase ranked highest.
@lorawoodford @lmcglohon I still cannot see evidence of the solr parameters set in the AppConfig file having any effect on the PUI. Can you test by simply adding "AND" as the default operator, and then performing a search with at least two terms? For example, if you search for James Gunn in the test site, like this: 818 results are returned right now. That's because the PUI is doing a search for James OR Gunn (at least, an equivalent search). I cannot change anything locally to impact that result (although I can for the staff interface!). And, as you'll see, adding AND to the search string does not have any impact, either: It's only when the solr op parameter is placed elsewhere, which is triggered by splitting the two terms on the homepage, that the AND parameter works here: All that said, it sounds like if the PUI search was harmonized with the SUI search, then things would work as expected here (although the SUI would need to be updated to be able to search for begin/end descriptive dates, since the PUI has that option as of 2.1). I've also tested passing the "mm" parameter to the SUI (changing it from 1 to 2 and back again, for example), and that works perfectly. But just like trying to change the default operator in the PUI, it does not have any effect in the PUI right now even though the param is being picked up via this latest pull request. |
Also, I just performed the exact same search in the SUI and the PUI, and looked at the backend logs. Those are startling. I'll copy a bit of the SUI log here, which makes sense to me for the most part, and shows that the mm=2 value is not only passed but respected:
|
And when I look at the logs for a PUI search, I'm quite confused because it seems to be repeating the repository OR searches over and over again. Anyhow, here's just a small bit that shows that the mm=2 param is passed but not utilized -- the number of search results, at 178, is because these two terms have been ORed together essentially, rather than both matching (as they both match in the SUI logs above); the number of results should actually be in the 20s, though, just like with the SUI:
And I can also get the expected results by going to the solr console and performing the search directly there. So, something in the PUI code needs to be pared away it seems. |
Also removed commented out code