Summary: | Last patron is replaced by current patron on page load | ||
---|---|---|---|
Product: | Koha | Reporter: | Nick Clemens (kidclamp) <nick> |
Component: | Staff interface | Assignee: | Nick Clemens (kidclamp) <nick> |
Status: | Pushed to main --- | QA Contact: | Emily Lamancusa (emlam) <emily.lamancusa> |
Severity: | normal | ||
Priority: | P5 - low | CC: | emily.lamancusa, gmcharlt |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
GIT URL: | Change sponsored?: | --- | |
Patch complexity: | --- | Documentation contact: | |
Documentation submission: | Text to go in the release notes: | ||
Version(s) released in: |
25.05.00
|
Circulation function: | |
Attachments: |
Bug 39022: Display the current previous patron list before updating stored values
Bug 39022: Display the current previous patron list before updating stored values Bug 39022: Display the current previous patron list before updating stored values |
Description
Nick Clemens (kidclamp)
2025-02-03 19:25:17 UTC
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> Pushed for 25.05! Well done everyone, thank you! |