Before bug 21246 Koha only stored a single last patron - when loading a new patron we would preserve the previous patron during the initial page load. After loading we would replace the stored patron so that the next page load would display the most recent patron. After bug 21246 we began to update the list of patrons to display, add the new one, then display. This means as soon as you load a patron you cannot get back to the previous patron. To test: 1 - Enable showLastPatron system preference and set showLastPatronCount to 1 2 - Go to a patron, say Ronnie Ballard 3 - Note you have a link to 'Last patron' - hover or click split dropdown and see it is the patron you are viewing 4 - Search for another patron, Edna Acosta 5 - Note the 'Last patron' links point to Edna, and not Ronnie
Created attachment 177476 [details] [review] Bug 39022: Display the current previous patron list before updating stored values This patch just reverses the order so we display the list, then update it. To test: 1 - Enable showLastPatron system preference and set showLastPatronCount to 1 2 - Go to a patron, say Ronnie Ballard 3 - Note you have a link to 'Last patron' - hover or click split dropdown and see it is the patron you are viewing 4 - Search for another patron, Edna Acosta 5 - Note the 'Last patron' links point to Edna, and not Ronnie 6 - Apply patch 7 - Reload Edna, note last patron lists Edna 8 - Search for Ronnie 9 - Confirm previous patron is linked to Edna 10 - Click the link to go to Edna 11 - Confirm last patron now links to Ronnie 12 - Instead of following link, click on another tab for patron 13 - Now the link is updated to current patron, this was behaviour before 21246 as well 14 - Sign off.
Created attachment 177491 [details] [review] Bug 39022: Display the current previous patron list before updating stored values This patch just reverses the order so we display the list, then update it. To test: 1 - Enable showLastPatron system preference and set showLastPatronCount to 1 2 - Go to a patron, say Ronnie Ballard 3 - Note you have a link to 'Last patron' - hover or click split dropdown and see it is the patron you are viewing 4 - Search for another patron, Edna Acosta 5 - Note the 'Last patron' links point to Edna, and not Ronnie 6 - Apply patch 7 - Reload Edna, note last patron lists Edna 8 - Search for Ronnie 9 - Confirm previous patron is linked to Edna 10 - Click the link to go to Edna 11 - Confirm last patron now links to Ronnie 12 - Instead of following link, click on another tab for patron 13 - Now the link is updated to current patron, this was behaviour before 21246 as well 14 - Sign off. Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl>
Created attachment 177639 [details] [review] Bug 39022: Display the current previous patron list before updating stored values This patch just reverses the order so we display the list, then update it. To test: 1 - Enable showLastPatron system preference and set showLastPatronCount to 1 2 - Go to a patron, say Ronnie Ballard 3 - Note you have a link to 'Last patron' - hover or click split dropdown and see it is the patron you are viewing 4 - Search for another patron, Edna Acosta 5 - Note the 'Last patron' links point to Edna, and not Ronnie 6 - Apply patch 7 - Reload Edna, note last patron lists Edna 8 - Search for Ronnie 9 - Confirm previous patron is linked to Edna 10 - Click the link to go to Edna 11 - Confirm last patron now links to Ronnie 12 - Instead of following link, click on another tab for patron 13 - Now the link is updated to current patron, this was behaviour before 21246 as well 14 - Sign off. Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl> Signed-off-by: Emily Lamancusa <emily.lamancusa@montgomerycountymd.gov>