Comment 1 for bug 1669522

Revision history for this message
Kathy Lussier (klussier) wrote :

It looks like there was a change to the normalize_qterm block that caused this error. There is no mention in the commit message about changes being made to the qterm, but, from what I can tell, it was an attempt to clean up the subsequent search to remove additional spaces between the search terms. However, instead of replacing the already normalized qterm, this regex seemed to add a string to the qterm, which resulted in duplicate author names and dates being sent in the query.

Since this was an aesthetic concern rather than a functional concern and was not a primary component of the feature that was added, I removed the regex. It seems to be working now.

Working branch at:
http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/kmlussier/lp1669522-fix-duplicate-author-query

Test plan:

Pre-patch: Retrieve a bib record in the catalog and click one of the author links at the top of the page. The search box on the subsequent search results page will show that the author names and dates were duplicated in the search. After loading the patch, clicking the link should only add each of these search terms once.