When search with an authtype in ES we are using the term filter to limit to a type. This filter is analyzed and the forward slash in genre/form seems to be problematic bug 20589 fixed this incidentally to other changes, however, 19.05 still has this bug.
Created attachment 96421 [details] [review] Bug 24264: Use raw field when limiting by authtype To test: 1 - Have a genre form record in your authorities file 2 - Search authorities for 'Default' 3 - The result is returned 4 - Set the dropdown to 'Genre/Form' and search matchign term 5 - No results 6 - Apply patch 7 - Restart all the things 8 - Repeat search for 'Genre/Form' and search matching term 9 - Record is returned
I'm sorry, Nick, but I can't get the patch to apply. I've tried on two different systems (Mac and Linux) using docker. Results: Setup: 19.05.x branch (19.05.06.000 on last attempt); ES enabled, query auto truncate turned off 1. There are 13 authority records in the test database containing "fiction" 2. 'Default' authority search for "fiction" 3. Yields 13 results - 3 topical and 10 genre/form 4. 'Genre/Form Term' authority search for "fiction" 5. Yields 0 results 6. Apply patch 7. restart_all, updatedatabase, dbic, reindex ES 8. 'Genre/Form Term' authority search for "fiction" 9. Yields 0 results Plack error log doesn't show anything. If there's another place to check for errors to troubleshoot the patch, please let me know.
Created attachment 96682 [details] [review] Bug 24264: Use raw field when limiting by authtype To test: 1 - Have a genre form record in your authorities file 2 - Search authorities for 'Default' 3 - The result is returned 4 - Set the dropdown to 'Genre/Form' and search matchign term 5 - No results 6 - Apply patch 7 - Restart all the things 8 - Repeat search for 'Genre/Form' and search matching term 9 - Record is returned
(In reply to Myka Kennedy Stephens from comment #2) > I'm sorry, Nick, but I can't get the patch to apply. I've tried on two > different systems (Mac and Linux) using docker. > Hi Myka, 'Patch does not apply' is for when there are errors when trying to apply the patch 'Failed QA' is for when a patch doesn't work after it is applied. That was the case here, I wrote a bad patch :-) When using 'raw' we also need to remove the lowercasing, otherwise it doesn't match. New patch attached, please retest
Created attachment 96694 [details] [review] Bug 24264: Use raw field when limiting by authtype To test: 1 - Have a genre form record in your authorities file 2 - Search authorities for 'Default' 3 - The result is returned 4 - Set the dropdown to 'Genre/Form' and search matchign term 5 - No results 6 - Apply patch 7 - Restart all the things 8 - Repeat search for 'Genre/Form' and search matching term 9 - Record is returned Signed-off-by: Myka Kennedy Stephens <mkstephens@lancasterseminary.edu>
(In reply to Nick Clemens from comment #4) > > New patch attached, please retest Patch worked as expected this time. Thanks, Nick!
Created attachment 96733 [details] [review] Bug 24264: Use raw field when limiting by authtype To test: 1 - Have a genre form record in your authorities file 2 - Search authorities for 'Default' 3 - The result is returned 4 - Set the dropdown to 'Genre/Form' and search matchign term 5 - No results 6 - Apply patch 7 - Restart all the things 8 - Repeat search for 'Genre/Form' and search matching term 9 - Record is returned Signed-off-by: Myka Kennedy Stephens <mkstephens@lancasterseminary.edu> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
backported to 19.05.x for 19.05.07