Bug 12775 added a link "Show only mine" in newordersuggestion.pl. This does not work, no results.
Created attachment 58957 [details] [review] Bug 17899 - Show only mine does not work in newordersuggestion.pl Bug 12775 added a link "Show only mine" in newordersuggestion.pl. This does not work, no results. Also corrects the fact that click must not do default action by adding e.preventDefault(). Test plan : - You must have suggestions you have accepted - Create a new order from suggestion : /cgi-bin/koha/acqui/newordersuggestion.pl - Click on Show only mine => Without patch the table is empty showing "No matching records found" => With patch you see only suggestions you have accpeted
Created attachment 59136 [details] [review] Bug 17899 - Show only mine does not work in newordersuggestion.pl Bug 12775 added a link "Show only mine" in newordersuggestion.pl. This does not work, no results. Also corrects the fact that click must not do default action by adding e.preventDefault(). Test plan : - You must have suggestions you have accepted - Create a new order from suggestion : /cgi-bin/koha/acqui/newordersuggestion.pl - Click on Show only mine => Without patch the table is empty showing "No matching records found" => With patch you see only suggestions you have accpeted Signed-off-by: Zoe Schoeler <crazy.mental.onion@gmail.com>
Created attachment 59251 [details] [review] Bug 17899 - Show only mine does not work in newordersuggestion.pl Bug 12775 added a link "Show only mine" in newordersuggestion.pl. This does not work, no results. Also corrects the fact that click must not do default action by adding e.preventDefault(). Test plan : - You must have suggestions you have accepted - Create a new order from suggestion : /cgi-bin/koha/acqui/newordersuggestion.pl - Click on Show only mine => Without patch the table is empty showing "No matching records found" => With patch you see only suggestions you have accpeted Signed-off-by: Zoe Schoeler <crazy.mental.onion@gmail.com> Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
I am sure the link used to work. I don't know for how long it has been broken.
Pushed to master for 17.05, thanks Frido!
This definitely works better than before, but there is an oddness. For testing this, I set 'acceptedby' to another use. This made no difference. I had to change 'managedby' as well, then it worked. It appears that the 'order by suggestion' page uses managedby instead of acceptedby for determining if you can see the suggestion? Anyway, improvement! This patch has been pushed to 16.11.x and will be in 16.11.02.
Pushed to 16.05.x, for 16.05.08 release