dd36a1ffa1
QueryParser was loading redundant search aliases, which in turn resulted in certain queries involving two or more clauses joined by a Boolean operator getting parsed in correctly. To test: [1] Enable QueryParser. [2] Arrange your database so that some some bib records contain the keyword 'history' while a smaller subset contain both 'history' and 'earth'. (The exact words used don't matter.) [3] Run the following search from the OPAC global search bar: kw:history && kw:earth [4] Observe that the records that are returned include *all* bibs in the database with the keyword 'history'. In other words, the restriction that the records must also contain 'earth' is not observed. [5] Apply the patch and rerun the search. [6] This time, the results that are returned should include only those that contain both 'history' and 'earth'. [7] Verify that prove -v t/QueryParser.t passes. Signed-off-by: Galen Charlton <gmc@esilibrary.com> Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> Passes test plans, all tests and QA script. Also checked the db_dependent/Search.t tests. Tried some other searches, all seems to be working correctly. Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com> I reproduce the bug and I confirm this patch fixes it. Without the patch, kw:history && kw:earth, I got: Search biblios OK 4 1 1+0 RPN @attrset Bib-1 @attr 4=6 @attr 1=1016 history With the path, I get: Search biblios OK 2 1 1+0 RPN @attrset Bib-1 @and @attr 4=6 @attr 1=1016 history @attr 4=6 @attr 1=1016 earth Signed-off-by: Galen Charlton <gmc@esilibrary.com> |
||
---|---|---|
.. | ||
Acquisition | ||
Circulation | ||
data | ||
Labels | ||
LDAP | ||
Members | ||
Record | ||
Reports | ||
Serials | ||
www | ||
00-strict.t | ||
Accounts.t | ||
Acquisition.t | ||
Amazon.t | ||
Auth.t | ||
Auth_ParseSearchHistoryCookie.t | ||
Auth_with_cas.t | ||
AuthoritiesMarc.t | ||
BackgroundJob.t | ||
Barcodes.t | ||
Biblio.t | ||
Bookseller.t | ||
Borrower_Debarments.t | ||
Branch.t | ||
Budgets.t | ||
check_sysprefs.t | ||
Circulation.t | ||
Circulation_Branch.t | ||
Circulation_issue.t | ||
Circulation_Issuingrule.t | ||
Circulation_issuingrules.t | ||
Circulation_OfflineOperation.t | ||
Circulation_transfers.t | ||
ClassSource.t | ||
Context.t | ||
CourseReserves.t | ||
Csv.t | ||
External_BakerTaylor.t | ||
Heading_MARC21.t | ||
Holds.t | ||
HoldsQueue.t | ||
Holidays.t | ||
ILSDI_Services.t | ||
Items.t | ||
KitchenSinkPlugin.kpz | ||
Koha.t | ||
Koha_Authority.t | ||
Koha_borrower_modifications.t | ||
Koha_Database.t | ||
Koha_template_plugin_KohaBranchName.t | ||
Koha_template_plugin_KohaDates.t | ||
Linker_FirstMatch.t | ||
Log.t | ||
MarcModificationTemplates.t | ||
Members.t | ||
Members_columns.t | ||
PatronLists.t | ||
Plugins.t | ||
Ratings.t | ||
Record.t | ||
RecordProcessor_EmbedSeeFromHeadings.t | ||
Reports.t | ||
Reports_Guided.t | ||
ReportsGuided.t | ||
Reserves.t | ||
rollingloans.t | ||
RotatingCollections.t | ||
Search.t | ||
Search_SearchHistory.t | ||
Serials.t | ||
Serials_2.t | ||
Service.t | ||
ShelfBrowser.t | ||
SIP_ILS.t | ||
SQLHelper.t | ||
SuggestionEngine_ExplodedTerms.t | ||
Suggestions.t | ||
sysprefs.t | ||
Tags.t | ||
UploadedFile.t | ||
VirtualShelves.t | ||
VirtualShelves_Page.t | ||
XISBN.t | ||
zebra_config.pl |