Summary: | Update patron account templates to use old_issue_id to display circ info | ||
---|---|---|---|
Product: | Koha | Reporter: | Andrew Fuerste-Henry <andrew> |
Component: | Fines and fees | Assignee: | Andrew Fuerste-Henry <andrew> |
Status: | Pushed to stable --- | QA Contact: | Marcel de Rooy <m.de.rooy> |
Severity: | major | ||
Priority: | P5 - low | CC: | danielle.elder, jonathan.druart, laura, lucas, m.de.rooy, paul.derscheid |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
GIT URL: | Change sponsored?: | --- | |
Patch complexity: | Small patch | Documentation contact: | |
Documentation submission: | Text to go in the release notes: | ||
Version(s) released in: |
25.05.00,24.11.03
|
Circulation function: | |
Bug Depends on: | 22421 | ||
Bug Blocks: | |||
Attachments: |
Bug 39025: Use old_issue_id for issue details on patron account screens
Bug 39025: Use old_issue_id for issue details on patron account screens Bug 39025: Use old_issue_id for issue details on patron account screens Bug 39025: Use old_issue_id for issue details on patron account screens |
Description
Andrew Fuerste-Henry
2025-02-03 21:28:18 UTC
Created attachment 177506 [details] [review] Bug 39025: Use old_issue_id for issue details on patron account screens To test: 1: Have an item that is checked out, overdue, and showing an accruing fine 2: On the patron account, check both Make A Payment and Transactions tabs, confirm that the fine shows Due Date, Checkout Date, and Checked Out From values 3: Check your item in 4: Reload the patron account, confirm that the fine no longer shows checkout data on either Make A Payment or Transactions 5: Apply patch 6: Reload pages, confirm Make A Payment and Transactions tabs now show values in Due Date, Checkin Date, Checkout Date, and Checked Out From Created attachment 177513 [details] [review] Bug 39025: Use old_issue_id for issue details on patron account screens To test: 1: Have an item that is checked out, overdue, and showing an accruing fine 2: On the patron account, check both Make A Payment and Transactions tabs, confirm that the fine shows Due Date, Checkout Date, and Checked Out From values 3: Check your item in 4: Reload the patron account, confirm that the fine no longer shows checkout data on either Make A Payment or Transactions 5: Apply patch 6: Reload pages, confirm Make A Payment and Transactions tabs now show values in Due Date, Checkin Date, Checkout Date, and Checked Out From Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl> Created attachment 177856 [details] [review] Bug 39025: Use old_issue_id for issue details on patron account screens To test: 1: Have an item that is checked out, overdue, and showing an accruing fine 2: On the patron account, check both Make A Payment and Transactions tabs, confirm that the fine shows Due Date, Checkout Date, and Checked Out From values 3: Check your item in 4: Reload the patron account, confirm that the fine no longer shows checkout data on either Make A Payment or Transactions 5: Apply patch 6: Reload pages, confirm Make A Payment and Transactions tabs now show values in Due Date, Checkin Date, Checkout Date, and Checked Out From Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl> Patch rebased using the auto-rebase script. Comment on attachment 177856 [details] [review] Bug 39025: Use old_issue_id for issue details on patron account screens Review of attachment 177856 [details] [review]: ----------------------------------------------------------------- The lines modified by the patch check for account.issue_id and account.old_issue_id and then use account.checkout Can't we simply check if account.checkout is defined ? (which should return undef if issue_id and old_issue_id are both null) ? Also, storing the result of account.checkout in a TT variable probably avoids several duplicate SQL queries as it's used multiple times. Created attachment 178077 [details] [review] Bug 39025: Use old_issue_id for issue details on patron account screens To test: 1: Have an item that is checked out, overdue, and showing an accruing fine 2: On the patron account, check both Make A Payment and Transactions tabs, confirm that the fine shows Due Date, Checkout Date, and Checked Out From values 3: Check your item in 4: Reload the patron account, confirm that the fine no longer shows checkout data on either Make A Payment or Transactions 5: Apply patch 6: Reload pages, confirm Make A Payment and Transactions tabs now show values in Due Date, Checkin Date, Checkout Date, and Checked Out From Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Pushed for 25.05! Well done everyone, thank you! Will it be possible to backport this to 24.11, please? Another request for backport. This affects our billing documentation, and when money is involved it can be contentious. Nice work everyone! Pushed to 24.11.x for 24.11.03 |