https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41074

--- Comment #1 from Nick Clemens (kidclamp) <[email protected]> ---
Created attachment 188301
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=188301&action=edit
Bug 41074: Don't reverse previous_patrons arrary

This patch alters the code to append instead of prepend, removing the need to
reverse the list

To test:
1 - Write a report to get some patron
SELECT cardnumber,surname FROM borrowers WHERE cardnumber <>'' ORDER BY rand()
LIMit 6
2 - Enabled 'showLastPatron' and set count to 3
   
http://localhost:8081/cgi-bin/koha/admin/preferences.pl?op=search&searchfield=showLastPatron
3 - Copy the barcodes in order from the report and search for them in patron
search, you'll want two tabs so you can keep report open
4 - Notice at 4th patron that list order has changed
5 - When 5th patron is loaded, the 4th is removed from the list
6 - Apply patch
7 - Clear the list
8 - Repeat the test plan, but note that list order does not change
9 - Confirm the patrons are removed from the list in expected order when adding
new
10 - Change the count to 1 and search for several patrons, ensuring the last
patron link is always correct

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to