To recreate: 1 - Search for C++ 2 - Sort your results 3 - Note search is now for "C "
Created attachment 95826 [details] [review] Bug 24120: URI filter search terms in sort dropdowns To test: 1 - Search for C++ 2 - Sort your results 3 - Note search is now for "C " 4 - Apply patch 5 - Search for C++ 6 - Reorder results 7 - Still searching for C++
Created attachment 95828 [details] [review] Bug 24120: URI filter search terms in sort dropdowns To test: 1 - Search for C++ 2 - Sort your results 3 - Note search is now for "C " 4 - Apply patch 5 - Search for C++ 6 - Reorder results 7 - Still searching for C++ Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Why not QUERY_INPUT.input_name as well?
Created attachment 95830 [details] [review] Bug 24120: (follow-up) input_name too
Created attachment 95831 [details] [review] Bug 24120: (follow-up) input_name too Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Created attachment 95832 [details] [review] Bug 24120: URI filter search terms in sort dropdowns To test: 1 - Search for C++ 2 - Sort your results 3 - Note search is now for "C " 4 - Apply patch 5 - Search for C++ 6 - Reorder results 7 - Still searching for C++ Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Created attachment 95833 [details] [review] Bug 24120: (follow-up) input_name too Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Nice work! Pushed to master for 19.11.00
backported to 19.05.x for 19.05.06
Backported to 18.11.x for 18.11.12