The suggestions filter at the OPAC does not work at all. There is an input box but it does nothing https://snipboard.io/tsGM8h.jpg
Created attachment 109748 [details] [review] Bug 26406: Fix suggestions filter at the OPAC There is an input filter but it does nothing Test plan: You need 2 differents users, U1 and U2 Create several (at least one) suggestions for each of them Test the following with and without OPACViewOthersSuggestions: With U1 logged in, go to the suggestions tab and use the input to filter the suggestions. It should work correctly. Also play with the "Suggested by" options
Created attachment 109762 [details] [review] Bug 26406: Fix suggestions filter at the OPAC There is an input filter but it does nothing Test plan: You need 2 differents users, U1 and U2 Create several (at least one) suggestions for each of them Test the following with and without OPACViewOthersSuggestions: With U1 logged in, go to the suggestions tab and use the input to filter the suggestions. It should work correctly. Also play with the "Suggested by" options Signed-off-by: David Nind <david@davidnind.com>
Patch doesn't apply because of bug 20168
Created attachment 116630 [details] [review] Bug 26406: Fix suggestions filter at the OPAC There is an input filter but it does nothing Test plan: You need 2 differents users, U1 and U2 Create several (at least one) suggestions for each of them Test the following with and without OPACViewOthersSuggestions: With U1 logged in, go to the suggestions tab and use the input to filter the suggestions. It should work correctly. Also play with the "Suggested by" options Signed-off-by: David Nind <david@davidnind.com>
I completely forgot about this patch and, meanwhile, I wrote bug 27047.
Do we still need this patch? The search box on the opac suggestions page seems to work now ? Changing status to reflect need for feedback.
Yes, I think it is. You can filter server-side, better than the DT global filter IMO.
Created attachment 116862 [details] [review] Bug 26406: Fix suggestions filter at the OPAC There is an input filter but it does nothing Test plan: You need 2 differents users, U1 and U2 Create several (at least one) suggestions for each of them Test the following with and without OPACViewOthersSuggestions: With U1 logged in, go to the suggestions tab and use the input to filter the suggestions. It should work correctly. Also play with the "Suggested by" options Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Pushed to master for 21.05, thanks to everybody involved!
Pushed to 20.11.x for 20.11.03
Missing dependency from bug 27047, which needs a rebase. Please rebase that for backport if this is needed.